Date: Wed, 23 May 2001 10:26:41 -0700 (PDT) From: John Baldwin <jhb@FreeBSD.org> To: Alfred Perlstein <alfred@FreeBSD.org> Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Subject: RE: cvs commit: src/sys/vm swap_pager.c Message-ID: <XFMail.010523102641.jhb@FreeBSD.org> In-Reply-To: <200105231028.f4NASEh62711@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 23-May-01 Alfred Perlstein wrote: > alfred 2001/05/23 03:28:14 PDT > > Modified files: > sys/vm swap_pager.c > Log: > aquire Giant when playing with the buffercache and doing IO. > use msleep against the vm mutex while waiting for a page IO to complete. Erm, have you looked at my vm.patch at all? You no longer need to acquire Giant here. Giant is acquired hirer up in the tree to guarantee that we don't manipulate vnodes without it w/o having to do ugly things like release the vm lock just to acquire Giant adn then acquire the vm lock. It also will make it much easier to remove Giant when vnodes and buf/bio stuff is locked. -- John Baldwin <jhb@FreeBSD.org> -- http://www.FreeBSD.org/~jhb/ PGP Key: http://www.Baldwin.cx/~john/pgpkey.asc "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?XFMail.010523102641.jhb>