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

Reply via email to