On Tue, 17 Jun 2025 06:24:23 -0700 Breno Leitao wrote: > > here we are in process context and debug checks complain about the use > > of this_cpu_ptr(). Let's wrap this in local_bh_disable() / enable() ? > > Thanks. I was able to reproduce it. Your suggestion avoids the complain. > I suppose we should just wrap dev_dstats_rx_dropped_add(), right?
I think so. I hope that makes the preempt checker happy. The reason for bh_disable rather than preempt_disable() is that we should only update the stats from one context, the NAPI poll context. So we also need to prevent races with another NAPI running on the same core while we free an old queue.
