Date: Thu, 01 Dec 2011 09:37:12 +0100
From: Bernhard Froehlich <decke@FreeBSD.org>
To: Jung-uk Kim <jkim@freebsd.org>
Cc: Alan Cox <alc@freebsd.org>, freebsd-emulation@freebsd.org, FreeBSD current <freebsd-current@freebsd.org>, Gleb Kurtsou <gleb.kurtsou@gmail.com>, Andriy Gapon <avg@freebsd.org>
Subject: Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}
Message-ID: <60ea779052f025798cf65e18c24b7b31@bluelife.at>
In-Reply-To: <201111301807.21351.jkim@FreeBSD.org>
References: <4ECF7440.4070300@entel.upc.edu> <20111126163343.GA9150@reks> <4ED6AEFE.4010106@FreeBSD.org> <201111301807.21351.jkim@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 01.12.2011 00:07, Jung-uk Kim wrote: > On Wednesday 30 November 2011 05:32 pm, Andriy Gapon wrote: >> on 26/11/2011 18:33 Gleb Kurtsou said the following: >> > Using new vm_page_alloc_contig() may be a better option here. >> > Can't help with patch, stuck with pre Nov 15 CURRENT myself. >> >> on 27/11/2011 19:09 Alan Cox said the following: >> > vm_page_alloc_contig() should be used instead. >> >> My take on the patch: >> http://people.freebsd.org/~avg/vbox-10.patch >> This is for head only, no check for FreeBSD version. > > Actually, I did the same thing last night: > > > http://people.freebsd.org/~jkim/patch-src-VBox-Runtime-r0drv-freebsd-memobj-r0drv-freebsd.c > > This is a drop-in replacement for the patch. The only practical > difference I see from yours is I used VM_ALLOC_INTERRUPT instead of > VM_ALLOC_NORMAL. I believe this function may be used in interrupt > context. FYI, I tried FreeBSD 9 and Fedora 10 without problem. Thanks a lot for both patches! Could you please as usual reply and tell me if it is okay to send this patch upstream under MIT license? Once there is some positive feedback I will commit both patches to the ports tree. -- Bernhard Froehlich http://www.bluelife.at/
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?60ea779052f025798cf65e18c24b7b31>