On Thu, 6 Mar 2025 09:00:02 -0800 Joe Damato wrote: > + * - wrap all of the work in a lock (perhaps > vi->refill_lock?) > + * - check netif_running() and return early to avoid a race > + */
probably netdev instance lock is better here, as it will also protect the return value of netif_running(). IOW we need to base the "is the device up" test on some state that's protected by the lock we take.