Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 05 Dec 2011 13:57:15 +0100
From:      Bernhard Froehlich <decke@FreeBSD.org>
To:        Bernhard Froehlich <decke@freebsd.org>
Cc:        Gleb Kurtsou <gleb.kurtsou@gmail.com>, Alan Cox <alc@freebsd.org>, Gapon <avg@freebsd.org>, freebsd-emulation@freebsd.org, FreeBSD current <freebsd-current@freebsd.org>, Andriy, Jung-uk Kim <jkim@freebsd.org>
Subject:   Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}
Message-ID:  <7c3c9505867f4528af276a571077b9ce@bluelife.at>
In-Reply-To: <47eb9f9b139dd8c59b050f1670a5f18d@bluelife.at>
References:  <4ECF7440.4070300@entel.upc.edu> <20111126163343.GA9150@reks> <4ED6AEFE.4010106@FreeBSD.org> <201111301807.21351.jkim@FreeBSD.org> <60ea779052f025798cf65e18c24b7b31@bluelife.at> <47eb9f9b139dd8c59b050f1670a5f18d@bluelife.at>

next in thread | previous in thread | raw e-mail | index | archive | help
On 02.12.2011 12:55, Bernhard Froehlich wrote:
> On 01.12.2011 09:37, Bernhard Froehlich wrote:
>> 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.
>
> Patch has been send upstream:
>
> 
> https://www.virtualbox.org/pipermail/vbox-dev/2011-December/004842.html

Patches have been committed upstream. Thanks a lot guys!

https://www.virtualbox.org/changeset/39521

-- 
Bernhard Froehlich
http://www.bluelife.at/



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7c3c9505867f4528af276a571077b9ce>