Lorenzo Bianconi <lorenzo@xxxxxxxxxx> writes: >> > + rcu_read_lock(); >> >> This is not needed when the function is only being called from XDP... > > don't we need it since we do not hold the rtnl here? No. XDP programs always run under local_bh_disable() which "counts" as an rcu_read_lock(); I did some cleanup around this a while ago, see this commit for a longer explanation: 782347b6bcad ("xdp: Add proper __rcu annotations to redirect map entries") -Toke