Paul Irofti <[email protected]> wrote:

> Any reason to just sleep here besides anoying the user? OK?
> 
> 
> diff --git sys/dev/ic/pgt.c sys/dev/ic/pgt.c
> index 45a85503221..477fda34c40 100644
> --- sys/dev/ic/pgt.c
> +++ sys/dev/ic/pgt.c
> @@ -2250,13 +2250,6 @@ pgt_ioctl(struct ifnet *ifp, u_long cmd, caddr_t req)
>               /*
>                * This chip scans always as soon as it gets initialized.
>                */
> -
> -             /*
> -              * Give us a bit time to scan in case we were not
> -              * initialized before and let the userland process wait.
> -              */
> -             tsleep(&sc->sc_flags, 0, "pgtsca", hz * SCAN_TIMEOUT);
> -
>               break;
>       case SIOCG80211ALLNODES: {
>               struct ieee80211_nodereq *nr = NULL;
> 

Little hazard to this.  pgt use is very rare.  Take a shot at it, and
pay attention to reports for a few months.  Make your commit message
clear so that people who hit problesm can identify this change
as the likely cause.

Reply via email to