On Thu, Jan 11, 2018 at 06:25:09PM -0800, Brian Norris wrote: > Anyway, I'll do my own testing and then submit my patch properly.
OK, so I definitely confirmed: if your patch does anything, it introduces a new deadlock possibility. Just trigger a Wifi timeout or reset from within remove(), and you'll see the work event get stuck in pci_reset_function(), while remove() gets stuck at cancel_work_sync(). I also confirmed that my patch resolves this problem. I'll send the revert + my patch now. Brian
