[PATCH net-next 2/4] tc/act: remove unneeded RCU lock in action callback

2018-07-19 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

Re: [PATCH net-next 2/4] tc/act: remove unneeded RCU lock in action callback

2018-07-13 Thread Paolo Abeni
On Fri, 2018-07-13 at 16:41 +0200, Daniel Borkmann wrote: > On 07/13/2018 04:26 PM, Paolo Abeni wrote: > > On Fri, 2018-07-13 at 16:08 +0200, Daniel Borkmann wrote: > > > On 07/13/2018 11:55 AM, Paolo Abeni wrote: > > > > Each lockless action currently does its own RCU locking in ->act(). > > > > T

Re: [PATCH net-next 2/4] tc/act: remove unneeded RCU lock in action callback

2018-07-13 Thread Daniel Borkmann
On 07/13/2018 04:26 PM, Paolo Abeni wrote: > On Fri, 2018-07-13 at 16:08 +0200, Daniel Borkmann wrote: >> On 07/13/2018 11:55 AM, 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

Re: [PATCH net-next 2/4] tc/act: remove unneeded RCU lock in action callback

2018-07-13 Thread Paolo Abeni
On Fri, 2018-07-13 at 16:08 +0200, Daniel Borkmann wrote: > Hi Paolo, > > On 07/13/2018 11:55 AM, 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 d

Re: [PATCH net-next 2/4] tc/act: remove unneeded RCU lock in action callback

2018-07-13 Thread Daniel Borkmann
Hi Paolo, On 07/13/2018 11:55 AM, 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, cl

[PATCH net-next 2/4] tc/act: remove unneeded RCU lock in action callback

2018-07-13 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