From owner-freebsd-current@FreeBSD.ORG  Sun Nov 27 01:37:02 2011
Return-Path: <owner-freebsd-current@FreeBSD.ORG>
Delivered-To: freebsd-current@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id C2D22106566C;
	Sun, 27 Nov 2011 01:37:02 +0000 (UTC)
	(envelope-from imb@protected-networks.net)
Received: from sarah.protected-networks.net (sarah.protected-networks.net
	[IPv6:2001:470:1f07:4e1::1])
	by mx1.freebsd.org (Postfix) with ESMTP id 8F3818FC0A;
	Sun, 27 Nov 2011 01:37:02 +0000 (UTC)
Received: from toshi.auburn.protected-networks.net
	(toshi.auburn.protected-networks.net [202.12.127.84])
	(using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits))
	(Client CN "Iain Butler",
	Issuer "RSA Class 2 Personal CA" (verified OK))
	(Authenticated sender: imb@protected-networks.net)
	by sarah.protected-networks.net (Postfix) with ESMTPSA id 35326617F;
	Sat, 26 Nov 2011 20:37:01 -0500 (EST)
DomainKey-Signature: a=rsa-sha1; s=200509; d=protected-networks.net; c=nofws;
	q=dns; 
	h=message-id:date:from:user-agent:mime-version:to:cc:subject:
	references:in-reply-to:content-type:content-transfer-encoding;
	b=l3u5IP+oauRdnVSP2R0jmDlijmiIynSRJeXyBg9uqfnyN/hr87MMboxCIniwNAibr
	mtKSQZQ7ChvChrTWgEzXnPZei3GMRYBSWvZU8h1nMngpIYj9soOs6uf2QHRCaV8
Message-ID: <4ED1943A.90707@protected-networks.net>
Date: Sat, 26 Nov 2011 20:36:58 -0500
From: Michael Butler <imb@protected-networks.net>
User-Agent: Mozilla/5.0 (X11; FreeBSD i386;
	rv:8.0) Gecko/20111111 Thunderbird/8.0
MIME-Version: 1.0
To: Gleb Kurtsou <gleb.kurtsou@gmail.com>
References: <4ECF7440.4070300@entel.upc.edu>
	<4ECF8F05.8000007@protected-networks.net>
	<4ED0C40D.5010307@entel.upc.edu>
	<4ED0D963.1030702@entel.upc.edu> <4ED0DF1F.6090901@FreeBSD.org>
	<20111126163343.GA9150@reks>
In-Reply-To: <20111126163343.GA9150@reks>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Cc: FreeBSD current <freebsd-current@FreeBSD.org>,
	freebsd-emulation@FreeBSD.org,
	=?UTF-8?B?R3VzdGF1IFDDqXJleg==?= <gperez@entel.upc.edu>,
	Andriy Gapon <avg@FreeBSD.org>
Subject: Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}
X-BeenThere: freebsd-current@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussions about the use of FreeBSD-current
	<freebsd-current.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>, 
	<mailto:freebsd-current-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-current>
List-Post: <mailto:freebsd-current@freebsd.org>
List-Help: <mailto:freebsd-current-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
	<mailto:freebsd-current-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 27 Nov 2011 01:37:02 -0000

On 11/26/11 11:33, Gleb Kurtsou wrote:
> On (26/11/2011 14:44), Andriy Gapon wrote:
>> vm_phys_alloc_contig implementation has been recently changed and now it seems
>> to require that vm_page_queue_free_mtx is held.
>
> Using new vm_page_alloc_contig() may be a better option here. Can't help
> with patch, stuck with pre Nov 15 CURRENT myself.

If I understand the change in locking semantics (post SVN r227568?), a 
good number of chunks of 
src/VBox/Runtime/r0drv/freebsd/memobj-r0drv-freebsd.c need updating to 
follow this :-(.

It is now insufficient to hold only the queue lock when calling 
vm_page_unwire or vm_page_free (and maybe others). The page itself must 
now also be locked,

	imb