You said suspend/resume Do you mean for DVACT_WAKEUP? That's run with sleep allowed.
Some drivers fix their hardware without sleeps in DVACT_RESUME. Others, do this async with sleeps in DVACT_WAKEUP. If you can gaurantee that no io transactions happen async to you, then what you do now is fine. And I see the same structure in other net80211 drivers. Stefan Sperling <[email protected]> wrote: > CVSROOT: /cvs > Module name: src > Changes by: [email protected] 2024/08/17 02:12:46 > > Modified files: > sys/dev/ic : qwx.c > > Log message: > enable qwx(4) ext irqs earlier for suspend/resume > > Enable ext IRQs like the Linux driver does it, not later. I did this > differently due to problems in early development that have since > been fixed. > > Fixes an interrupt storm during resume which prevents things like > key installation from working, leaving the network link dead. >
