From owner-svn-src-head@FreeBSD.ORG Sat Dec 18 07:27:56 2010 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2D6031065672; Sat, 18 Dec 2010 07:27:56 +0000 (UTC) (envelope-from joel@FreeBSD.org) Received: from mail.vnode.se (mail.vnode.se [62.119.52.80]) by mx1.freebsd.org (Postfix) with ESMTP id D40FC8FC08; Sat, 18 Dec 2010 07:27:55 +0000 (UTC) Received: from mail.vnode.se (localhost [127.0.0.1]) by mail.vnode.se (Postfix) with ESMTP id 1D036E3F07B; Sat, 18 Dec 2010 08:08:07 +0100 (CET) X-Virus-Scanned: amavisd-new at vnode.se Received: from mail.vnode.se ([127.0.0.1]) by mail.vnode.se (mail.vnode.se [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id DC7tUWagMsEa; Sat, 18 Dec 2010 08:08:04 +0100 (CET) Received: from pluto.vnode.local (unknown [83.223.1.131]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.vnode.se (Postfix) with ESMTPSA id 7CC2DE3F079; Sat, 18 Dec 2010 08:08:03 +0100 (CET) Date: Sat, 18 Dec 2010 08:08:00 +0100 From: Joel Dahl To: Alan Cox Message-ID: <20101218070800.GA59878@pluto.vnode.local> References: <201012092016.oB9KG05P049565@svn.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201012092016.oB9KG05P049565@svn.freebsd.org> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r216333 - head/sys/i386/i386 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 18 Dec 2010 07:27:56 -0000 On 09-12-2010 20:16, Alan Cox wrote: > Author: alc > Date: Thu Dec 9 20:16:00 2010 > New Revision: 216333 > URL: http://svn.freebsd.org/changeset/base/216333 > > Log: > When r207410 eliminated the acquisition and release of the page queues > lock from pmap_extract_and_hold(), it didn't take into account that > pmap_pte_quick() sometimes requires the page queues lock to be held. > This change reimplements pmap_extract_and_hold() such that it no > longer uses pmap_pte_quick(), and thus never requires the page queues > lock. > > For consistency, adopt the same idiom as used by the new > implementation of pmap_extract_and_hold() in pmap_extract() and > pmap_mincore(). It also happens to make these functions shorter. Hi Alan, This commit makes my laptop hang everytime I quit X. I just get a black screen and the machine won't respond to any keys. Everything works if I go back to r216330. -- Joel