From owner-cvs-all Wed May 23 10:26:50 2001 Delivered-To: cvs-all@freebsd.org Received: from pike.osd.bsdi.com (pike.osd.bsdi.com [204.216.28.222]) by hub.freebsd.org (Postfix) with ESMTP id ED54637B42C; Wed, 23 May 2001 10:26:43 -0700 (PDT) (envelope-from jhb@foo.osd.bsdi.com) Received: from foo.osd.bsdi.com (root@foo.osd.bsdi.com [204.216.28.137]) by pike.osd.bsdi.com (8.11.1/8.9.3) with ESMTP id f4NHQgK60013; Wed, 23 May 2001 10:26:42 -0700 (PDT) (envelope-from jhb@foo.osd.bsdi.com) Received: (from jhb@localhost) by foo.osd.bsdi.com (8.11.1/8.11.1) id f4NHQf244588; Wed, 23 May 2001 10:26:41 -0700 (PDT) (envelope-from jhb) Message-ID: X-Mailer: XFMail 1.4.0 on FreeBSD X-Priority: 3 (Normal) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8bit MIME-Version: 1.0 In-Reply-To: <200105231028.f4NASEh62711@freefall.freebsd.org> Date: Wed, 23 May 2001 10:26:41 -0700 (PDT) From: John Baldwin To: Alfred Perlstein Subject: RE: cvs commit: src/sys/vm swap_pager.c Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG 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 -- 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