Re: [PATCH net] ethernet: alx: fix order of calls on resume

On Fri, 5 Mar 2021 14:17:29 -0800 Jakub Kicinski wrote:

netif_device_attach() will unpause the queues so we can't call it before __alx_open(). This went undetected until commit b0999223f224 ("alx: add ability to allocate and free alx_napi structures") but now if stack tries to xmit immediately on resume before __alx_open() we'll crash on the NAPI being null:

Cc: stable@vger.kernel.org # 4.9+ Fixes: bc2bebe8de8e ("alx: remove WoL support") Reported-by: Zbynek Michl zbynek.michl@gmail.com Link: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=983595 Signed-off-by: Jakub Kicinski kuba@kernel.org

I should have also added:

Tested-by: Zbynek Michl zbynek.michl@gmail.com