On Thu, Feb 02, 2017 at 03:41:22AM +0000, Jonathan A. Kollasch wrote: > @@ -432,6 +433,10 @@ wpi_detach(device_t self, int flags __un > pci_intr_disestablish(sc->sc_pct, sc->sc_ih); > sc->sc_ih = NULL; > } > + if (sc->sc_pihp != NULL) { > + pci_intr_release(sc->sc_pct, sc->sc_pihp, 1); > + sc->sc_pihp = NULL; > + } > mutex_enter(&sc->sc_rsw_mtx); > sc->sc_dying = 1; > cv_signal(&sc->sc_rsw_cv); >
hmm, not introduced by you now, but is it safe that this function is mutex_entering so late? shouldn't it mutex enter and set sc_dying first thing? thanks