Messages in this thread | | | Date | Tue, 16 Aug 2022 16:44:35 -0700 | From | Jakub Kicinski <> | Subject | Re: [PATCH] net: Fix suspicious RCU usage in bpf_sk_reuseport_detach() |
| |
On Tue, 16 Aug 2022 22:16:46 +0100 David Howells wrote: > So either __rcu_dereference_sk_user_data_with_flags_check() has to be a macro, > or we need to go with something like the first version of my patch where I > don't pass the condition through. Do you have a preference?
I like your version because it documents what the lock protecting this field is.
In fact should we also add && sock_owned_by_user(). Martin, WDYT? Would that work for reuseport? Jakub S is fixing l2tp to hold the socket lock while setting this field, yet most places take the callback lock...
One the naming - maybe just drop the _with_flags() ? There's no version of locked helper which does not take the flags. And not underscores?
| |