On Mon, Nov 13, 2023 at 02:05:36PM +0100, Daniel Borkmann wrote: > On 11/13/23 11:03 AM, Simon Horman wrote: > > On Sun, Nov 12, 2023 at 09:30:03PM +0100, Daniel Borkmann wrote: ... > > > @@ -10469,6 +10513,7 @@ void netdev_run_todo(void) > > > WARN_ON(rcu_access_pointer(dev->ip_ptr)); > > > WARN_ON(rcu_access_pointer(dev->ip6_ptr)); > > > + netdev_do_free_pcpu_stats(dev); > > > if (dev->priv_destructor) > > > dev->priv_destructor(dev); > > > if (dev->needs_free_netdev) > > > > nit: the hunk above seems unnecessary; one blank line is enough. > > I'm not sure which one you mean? It seems that I was confused for some reason, please ignore my previous comment.