On Tue, Feb 19, 2019 at 2:15 AM Vlad Buslov <vla...@mellanox.com> wrote: > > > On Mon 18 Feb 2019 at 19:27, Cong Wang <xiyou.wangc...@gmail.com> wrote: > > On Wed, Feb 13, 2019 at 11:47 PM Vlad Buslov <vla...@mellanox.com> wrote: > >> > >> Set TCF_PROTO_OPS_DOIT_UNLOCKED for flower classifier to indicate that its > >> ops callbacks don't require caller to hold rtnl lock. > > > > So, if this means RTNL is gone for all cls_flower changes, why > > do I still see rtnl_lock() in cls_flower.c after all your patches in > > this set? > > It doesn't say that rtnl lock is gone, what it says is that caller > doesn't have to obtain rtnl lock before calling flower ops callbacks.
So RTNL lock is still a bottle neck after all of these patches, isn't it? :) Yeah, please kindly add an explanation for why RTNL lock is still here. Thanks.