Re: [PATCH net-next v3 3/5] tc/act: remove unneeded RCU lock in action callback

2018-07-25 Thread Marcelo Ricardo Leitner
On Wed, Jul 25, 2018 at 07:59:48AM -0400, Jamal Hadi Salim wrote: > On 24/07/18 04:06 PM, Paolo Abeni wrote: > > Each lockless action currently does its own RCU locking in ->act(). > > This is allows using plain RCU accessor, even if the context > > is really RCU BH. > > > > This change drops the

Re: [PATCH net-next v3 3/5] tc/act: remove unneeded RCU lock in action callback

2018-07-25 Thread Jiri Pirko
Tue, Jul 24, 2018 at 10:06:41PM CEST, pab...@redhat.com wrote: >Each lockless action currently does its own RCU locking in ->act(). >This is allows using plain RCU accessor, even if the context >is really RCU BH. > >This change drops the per action RCU lock, replace the accessors >with _bh variant,

Re: [PATCH net-next v3 3/5] tc/act: remove unneeded RCU lock in action callback

2018-07-25 Thread Jamal Hadi Salim
On 24/07/18 04:06 PM, Paolo Abeni wrote: Each lockless action currently does its own RCU locking in ->act(). This is allows using plain RCU accessor, even if the context is really RCU BH. This change drops the per action RCU lock, replace the accessors with _bh variant, cleans up a bit the surro

[PATCH net-next v3 3/5] tc/act: remove unneeded RCU lock in action callback

2018-07-24 Thread Paolo Abeni
Each lockless action currently does its own RCU locking in ->act(). This is allows using plain RCU accessor, even if the context is really RCU BH. This change drops the per action RCU lock, replace the accessors with _bh variant, cleans up a bit the surronding code and documents the RCU status in