Lorenzo Bianconi [off-list ref] writes:

quoted

quoted

quoted

+	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")

ack, right, I missed it. I will fix it.

Regards,
Lorenzo

-Toke