Brian Norris <briannor...@chromium.org> writes: > 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.
Great, thanks. I didn't had a chance to do the revert yet but I'll now apply your revert instead. -- Kalle Valo
