Date: Mon, 1 Mar 2004 14:01:36 -0500 From: John Baldwin <jhb@FreeBSD.org> To: Scott Long <scottl@FreeBSD.org> Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/en midway.c src/sys/dev/hatm if_hatm.c src/sys/dev/patm if_patm.c Message-ID: <200403011401.36326.jhb@FreeBSD.org> In-Reply-To: <200402290926.i1T9Q1mv041844@repoman.freebsd.org> References: <200402290926.i1T9Q1mv041844@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sunday 29 February 2004 04:26 am, Scott Long wrote: > scottl 2004/02/29 01:26:01 PST > > FreeBSD src repository > > Modified files: > sys/dev/en midway.c > sys/dev/hatm if_hatm.c > sys/dev/patm if_patm.c > Log: > All three of these drivers abused cv_waitq_empty in the same way by > spinning on it in hopes of making sure that the waitq was empty before > going on. This wasn't needed and probably never would have worked as > intended. Now that cv_waitq_empty() and friends are gone, the code in > these drivers that spins on it can go away too. This should unbreak LINT. > > Discussed with: kan Thanks. Sorry for breaking the build. :( -- John Baldwin <jhb@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve" = http://www.FreeBSD.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200403011401.36326.jhb>