Date: Mon, 05 Mar 2007 19:48:05 -0800 From: Sam Leffler <sam@errno.com> To: Jeff Roberson <jroberson@chesapeake.net> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, "Wojciech A. Koszek" <wkoszek@FreeBSD.org>, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern kern_alq.c Message-ID: <45ECE475.1010405@errno.com> In-Reply-To: <20070306234936.O635@10.0.0.1> References: <200703042340.l24Neanq039195@repoman.freebsd.org> <20070306234936.O635@10.0.0.1>
next in thread | previous in thread | raw e-mail | index | archive | help
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. Sam
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?45ECE475.1010405>