On Sun, 2017-04-09 at 08:25 +0200, Johannes Berg wrote: > That would also let you use rcu_assign_pointer() which seems like the > right thing to do here, along with marking the xdp_prog pointer as > __rcu? That'd also let you use rcu_dereference() instead of > READ_ONCE() which seems like the better annotation? Looks like drivers do it exactly this way too though. What I forgot: I guess we could make drivers use dev->xdp_prog after this, instead of having their own? johannes
|