Re: virtualbox r0 memory management update [Was: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}]

2011-12-06 Thread Andriy Gapon
on 06/12/2011 20:54 Bernhard Froehlich said the following: > On 06.12.2011 12:14, Andriy Gapon wrote: >> Meanwhile, here is the code that I came up with: >> http://people.freebsd.org/~avg/vbox/ >> For your convenience I have uploaded both the new file and its diff >> against svn >> head. I am test

Re: virtualbox r0 memory management update [Was: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}]

2011-12-06 Thread Bernhard Froehlich
On 06.12.2011 12:14, Andriy Gapon wrote: Meanwhile, here is the code that I came up with: http://people.freebsd.org/~avg/vbox/ For your convenience I have uploaded both the new file and its diff against svn head. I am testing this on FreeBSD head (r228017), so far no breakage observed. I would

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-06 Thread Andriy Gapon
on 06/12/2011 14:51 Bernhard Froehlich said the following: > > No. We currently have the last 2 virtualbox major versions in the ports tree > so > if we decide to do such an incompatible change we always have the older > version > around for 8.2 users for about the next year. Oh, I missed the f

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-06 Thread Bernhard Froehlich
On 06.12.2011 12:00, Andriy Gapon wrote: on 06/12/2011 11:40 Bernhard Froehlich said the following: On 06.12.2011 10:08, Andriy Gapon wrote: on 06/12/2011 02:08 Alan Cox said the following: The right thing to do is to MFC vm_page_alloc_contig(). It shouldn't be that hard to merge it. Ah, b

virtualbox r0 memory management update [Was: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}]

2011-12-06 Thread Andriy Gapon
Meanwhile, here is the code that I came up with: http://people.freebsd.org/~avg/vbox/ For your convenience I have uploaded both the new file and its diff against svn head. I am testing this on FreeBSD head (r228017), so far no breakage observed. I would appreciate reviews and testing of this cod

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-06 Thread Andriy Gapon
on 06/12/2011 11:40 Bernhard Froehlich said the following: > On 06.12.2011 10:08, Andriy Gapon wrote: >> on 06/12/2011 02:08 Alan Cox said the following: >>> The right thing to do is to MFC vm_page_alloc_contig(). It shouldn't be >>> that >>> hard to merge it. >> >> Ah, but we want to be able to

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-06 Thread Bernhard Froehlich
On 06.12.2011 10:08, Andriy Gapon wrote: on 06/12/2011 02:08 Alan Cox said the following: The right thing to do is to MFC vm_page_alloc_contig(). It shouldn't be that hard to merge it. Ah, but we want to be able to run virtualbox on the past releases too. Which releases are we talking abo

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-06 Thread Andriy Gapon
on 06/12/2011 02:08 Alan Cox said the following: > The right thing to do is to MFC vm_page_alloc_contig(). It shouldn't be that > hard to merge it. Ah, but we want to be able to run virtualbox on the past releases too. -- Andriy Gapon ___ freebsd-curr

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-05 Thread Alan Cox
On 12/5/2011 4:38 PM, Andriy Gapon wrote: on 06/12/2011 00:22 Alan Cox said the following: On 12/5/2011 4:02 PM, Andriy Gapon wrote: on 05/12/2011 19:58 Alan Cox said the following: On 12/05/2011 07:56, Andriy Gapon wrote: Pages should be marked busy only for some special occasions, wired pag

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-05 Thread Andriy Gapon
on 06/12/2011 00:22 Alan Cox said the following: > On 12/5/2011 4:02 PM, Andriy Gapon wrote: >> on 05/12/2011 19:58 Alan Cox said the following: >>> On 12/05/2011 07:56, Andriy Gapon wrote: Pages should be marked busy only for some special occasions, wired pages are not normall

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-05 Thread Alan Cox
On 12/5/2011 4:02 PM, Andriy Gapon wrote: on 05/12/2011 19:58 Alan Cox said the following: On 12/05/2011 07:56, Andriy Gapon wrote: Pages should be marked busy only for some special occasions, wired pages are not normally busy; the correct explanation is quite a bit longer than this, the commen

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-05 Thread Andriy Gapon
on 05/12/2011 19:58 Alan Cox said the following: > On 12/05/2011 07:56, Andriy Gapon wrote: >> Pages should be marked busy only for some special occasions, wired pages are >> not >> normally busy; the correct explanation is quite a bit longer than this, the >> comment in the code explains VPO_BUSY

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-05 Thread Alan Cox
On 12/05/2011 07:56, Andriy Gapon wrote: on 05/12/2011 15:15 Bernhard Froehlich said the following: On 05.12.2011 14:06, Andriy Gapon wrote: on 05/12/2011 14:57 Bernhard Froehlich said the following: On 02.12.2011 12:55, Bernhard Froehlich wrote: Patch has been send upstream: https://www.v

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-05 Thread Andriy Gapon
on 05/12/2011 15:15 Bernhard Froehlich said the following: > On 05.12.2011 14:06, Andriy Gapon wrote: >> on 05/12/2011 14:57 Bernhard Froehlich said the following: >>> On 02.12.2011 12:55, Bernhard Froehlich wrote: Patch has been send upstream: https://www.virtualbox.org/pi

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-05 Thread Bernhard Froehlich
On 05.12.2011 14:06, Andriy Gapon wrote: on 05/12/2011 14:57 Bernhard Froehlich said the following: On 02.12.2011 12:55, Bernhard Froehlich wrote: Patch has been send upstream: https://www.virtualbox.org/pipermail/vbox-dev/2011-December/004842.html Patches have been committed upstream. Tha

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-05 Thread Andriy Gapon
on 05/12/2011 14:57 Bernhard Froehlich said the following: > On 02.12.2011 12:55, Bernhard Froehlich wrote: >> 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://ww

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-05 Thread Bernhard Froehlich
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 bet

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-02 Thread Bernhard Froehlich
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

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-01 Thread Jung-uk Kim
On Thursday 01 December 2011 03:37 am, 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. >

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-01 Thread Bernhard Froehlich
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 1

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-12-01 Thread Gustau Pérez
On 01/12/2011 00:35, Andriy Gapon wrote: on 01/12/2011 01:27 Jung-uk Kim said the following: On Wednesday 30 November 2011 06:07 pm, 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_cont

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-30 Thread Andriy Gapon
on 01/12/2011 01:27 Jung-uk Kim said the following: > On Wednesday 30 November 2011 06:07 pm, 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.

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-30 Thread Jung-uk Kim
On Wednesday 30 November 2011 06:07 pm, 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 CU

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-30 Thread Jung-uk Kim
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:

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-30 Thread Andriy Gapon
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 patc

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-27 Thread Alan Cox
On 11/26/2011 06:44, Andriy Gapon wrote: on 26/11/2011 14:19 Gustau Pérez said the following: Starting Virtualbox in the console in headless mode allows to see what happens and get a dump of the panic. The messages I got were not the cause problem. The panic I was able to get shows this:

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-27 Thread Gustau Pérez
Using new vm_page_alloc_contig() may be a better option here. Can't help with patch, stuck with pre Nov 15 CURRENT myself. Ok. The third parameter of vm_page_alloc_contig says the caller has to specify an allocation class. Which one should we use? Also the vm_object_t and the vm_m

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-27 Thread Andriy Gapon
on 27/11/2011 03:36 Michael Butler said the following: > 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 v

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-26 Thread Michael Butler
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,

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-26 Thread Gleb Kurtsou
On (26/11/2011 14:44), Andriy Gapon wrote: > on 26/11/2011 14:19 Gustau Pérez said the following: > > > > Starting Virtualbox in the console in headless mode allows to see what > > happens > > and get a dump of the panic. > > > > The messages I got were not the cause problem. The panic I was

Re: Freeze with 10.0 and VirtualBox {4.1.4|4.1.6|4.1.51r38464}

2011-11-26 Thread Andriy Gapon
on 26/11/2011 14:19 Gustau Pérez said the following: > > Starting Virtualbox in the console in headless mode allows to see what > happens > and get a dump of the panic. > > The messages I got were not the cause problem. The panic I was able to get > shows this: > > http://pastebin.c