Date: Thu, 27 Sep 2007 12:18:10 -0700 From: "Kip Macy" <kip.macy@gmail.com> To: "John Baldwin" <jhb@freebsd.org> Cc: Perforce Change Reviews <perforce@freebsd.org>, Kip Macy <kmacy@freebsd.org> Subject: Re: PERFORCE change 126835 for review Message-ID: <b1fa29170709271218s19fbb775hdf4da10d8cfe6d80@mail.gmail.com> In-Reply-To: <200709261742.04531.jhb@freebsd.org> References: <200709262040.l8QKe8FL074297@repoman.freebsd.org> <200709261742.04531.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
It is in there, it just doesn't seem to do the trick :-/.
-Kip
On 9/26/07, John Baldwin <jhb@freebsd.org> wrote:
> On Wednesday 26 September 2007 04:40:08 pm Kip Macy wrote:
> > http://perforce.freebsd.org/chv.cgi?CH=126835
> >
> > Change 126835 by kmacy@kmacy_home:ethng on 2007/09/26 20:40:03
> >
> > increase delay in shutdown to avoid destroying the port lock
> > before the last call to the tick handler
> >
> > Affected files ...
> >
> > .. //depot/projects/ethng/src/sys/dev/cxgb/cxgb_main.c#21 edit
> >
> > Differences ...
> >
> > ==== //depot/projects/ethng/src/sys/dev/cxgb/cxgb_main.c#21 (text+ko) ====
> >
> > @@ -1023,7 +1023,7 @@
> >
> > ether_ifdetach(p->ifp);
> > printf("waiting for callout to stop ...");
> > - DELAY(10000);
> > + DELAY(100000);
> > printf("done\n");
>
> Umm, what about callout_drain()? That's its purpose.
>
> --
> John Baldwin
>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?b1fa29170709271218s19fbb775hdf4da10d8cfe6d80>