On Thu, 2016-09-08 at 22:30 -0700, Cong Wang wrote: > On Thu, Sep 8, 2016 at 9:15 AM, John Fastabend <john.fastab...@gmail.com> > wrote: > > > > This should be rtnl_derefence(t->params) and drop the read_lock/unlock > > pair. This is always called with RTNL lock unless you have a path I'm > > not seeing. > > You missed the previous discussion on V6, John. > > BTW, you really should follow the whole discussion instead of > jumping in the middle, like what you did for my patchset. > I understand you are eager to comment, but please don't waste > others' time in this way.... Please.
But John is right, and he definitely is welcome to give his feedback even at V13 if he wants to. tunnel_key_dump() is called with RTNL being held. Take a deep breath, vacations, and come back when you are relaxed. Thanks.