From owner-freebsd-emulation@FreeBSD.ORG Sun Nov 27 01:37:02 2011 Return-Path: Delivered-To: freebsd-emulation@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 User-Agent: Mozilla/5.0 (X11; FreeBSD i386; rv:8.0) Gecko/20111111 Thunderbird/8.0 MIME-Version: 1.0 To: Gleb Kurtsou 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-emulation@FreeBSD.org, Andriy Gapon Subject: Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464} X-BeenThere: freebsd-emulation@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Development of Emulators of other operating systems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-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