Page MenuHomeFreeBSD

iwm(4): Don't panic if firmware load fails

Authored by gavin on Nov 11 2016, 7:16 PM.



If the firmware load fails during init via iwm_preinit(), we bail out before calling ieee80211_ifattach() so the taskqueue hasn't been initialized. Don't try to drain it, we'll panic. Looks like this issue was introduced in r303326.

Diff Detail

rS FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

gavin updated this revision to Diff 22154.Nov 11 2016, 7:16 PM
gavin retitled this revision from to iwm(4): Don't panic if firmware load fails.
gavin updated this object.
gavin edited the test plan for this revision. (Show Details)
gavin added reviewers: adrian,, sbruno.
gavin set the repository for this revision to rS FreeBSD src repository.
gavin updated this object.Nov 11 2016, 7:23 PM edited edge metadata. added inline comments.
6195 ↗(On Diff #22154)

(unrelated; may be done later) 'do_net80211' may be replaced with 'sc->sc_ic.ic_vap_create == iwm_vap_create'

This revision is now accepted and ready to land.Nov 11 2016, 7:38 PM
sbruno accepted this revision.Nov 12 2016, 5:39 PM
sbruno edited edge metadata.
This revision was automatically updated to reflect the committed changes.