On 2025-01-16 17:52, Jakub Kicinski wrote: > On Thu, 16 Jan 2025 15:16:52 -0800 David Wei wrote: >> + ret = netdev_rx_queue_restart(dev, ifq_idx); >> + if (ret) >> + pr_devel("Could not restart queue %u after removing memory provider.\n", >> + ifq_idx); > > devmem does a WARN_ON() in this case, probably better to keep it > consistent? In case you need to respin Yeah I see it, thanks. It looks like I need to respin so I'll fix it up.