cvs commit: src/sys/kern kern_alq.c

Olivier Houchard cognet at ci0.org
Tue Mar 6 12:42:38 UTC 2007


On Mon, Mar 05, 2007 at 07:48:05PM -0800, Sam Leffler wrote:
> Jeff Roberson wrote:
> > The ALQ lock is a spinlock.  I don't believe this works.  Do I
> > misunderstand something?
> > 
> > On Sun, 4 Mar 2007, Wojciech A. Koszek wrote:
> > 
> >> wkoszek     2007-03-04 23:40:36 UTC
> >>
> >>  FreeBSD src repository
> >>
> >>  Modified files:
> >>    sys/kern             kern_alq.c
> >>  Log:
> >>  Use msleep(9) instead of tsleep(9) surrounded by lock acquisition and
> >>  release.
> >>
> >>  Approved by:    cognet (mentor)
> >>
> >>  Revision  Changes    Path
> >>  1.17      +2 -6      src/sys/kern/kern_alq.c
> >>
> > 
> > 
> 
> Yeah, this is totally busted and could never been tested.
> 

That's my fault, sorry.

Olivier


More information about the cvs-src mailing list