From owner-cvs-src@FreeBSD.ORG Wed Mar 23 20:03:57 2005 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 211D716A4CE; Wed, 23 Mar 2005 20:03:57 +0000 (GMT) Received: from mail-gw0.york.ac.uk (mail-gw0.york.ac.uk [144.32.128.245]) by mx1.FreeBSD.org (Postfix) with ESMTP id 4D16D43D49; Wed, 23 Mar 2005 20:03:56 +0000 (GMT) (envelope-from gavin.atkinson@ury.york.ac.uk) Received: from ury.york.ac.uk (ury.york.ac.uk [144.32.108.81]) by mail-gw0.york.ac.uk (8.12.10/8.12.10) with ESMTP id j2NK3pW5013876; Wed, 23 Mar 2005 20:03:51 GMT Received: from ury.york.ac.uk (localhost.york.ac.uk [127.0.0.1]) by ury.york.ac.uk (8.12.9p2/8.12.9) with ESMTP id j2NK3oFd059957; Wed, 23 Mar 2005 20:03:51 GMT (envelope-from gavin.atkinson@ury.york.ac.uk) Received: from localhost (gavin@localhost)j2NK3oPX059954; Wed, 23 Mar 2005 20:03:50 GMT (envelope-from gavin.atkinson@ury.york.ac.uk) X-Authentication-Warning: ury.york.ac.uk: gavin owned process doing -bs Date: Wed, 23 Mar 2005 20:03:50 +0000 (GMT) From: Gavin Atkinson X-X-Sender: gavin@ury.york.ac.uk To: Alan Cox In-Reply-To: <200503230602.j2N62TXV016518@repoman.freebsd.org> Message-ID: <20050323200200.E58945@ury.york.ac.uk> References: <200503230602.j2N62TXV016518@repoman.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-York-MailScanner: Found to be clean X-York-MailScanner-From: gavin.atkinson@ury.york.ac.uk cc: cvs-src@freebsd.org cc: src-committers@freebsd.org cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/pci agp.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 Mar 2005 20:03:57 -0000 On Wed, 23 Mar 2005, Alan Cox wrote: > alc 2005-03-23 06:02:29 UTC > Modified files: (Branch: RELENG_5) > sys/pci agp.c > Log: > MFC revision 1.46 > Avoid repeated acquisition and release of the vm object lock inside of > two loops in agp_generic_bind_memory(). As an intended side-effect, all > of the calls to vm_page_wakeup() are now performed with the containing > vm object lock held. Thanks! This fixes the panics I was seeing. Gavin