PV i386 patch

2011-12-16 Thread Alan Cox
Is anyone here actively working on fixing problems with SMP support under PV i386? While doing some other maintenance on the vm_page_alloc() callers in the source tree, I happened to take a look at cpu_initialize_context() in mp_machdep.c. This function is involved in bringing up the 2nd, 3rd

Re: PV i386 patch

2011-12-18 Thread Alan Cox
On 12/17/2011 18:56, Sean Bruno wrote: On Fri, 2011-12-16 at 11:32 -0800, Alan Cox wrote: Is anyone here actively working on fixing problems with SMP support under PV i386? While doing some other maintenance on the vm_page_alloc() callers in the source tree, I happened to take a look at

Re: PV i386 patch

2011-12-20 Thread Alan Cox
On 12/19/2011 18:09, Sean Bruno wrote: On Sat, 2011-12-17 at 18:01 -0800, Colin Percival wrote: On 12/17/11 16:56, Sean Bruno wrote: This seems happy on our ref9 VMs. I don't suppose this means I can go above 768M of Ram now? Can't hurt to try... whatever the problem is with our code and larg

Re: PV i386 patch

2011-12-20 Thread Alan Cox
On 12/20/2011 07:28, Sean Bruno wrote: The code that panics shouldn't even exist in the Xen pmap. Try the attached patch. Alan Indeed how on earth did we ever use this stuff? :-) Tested to 2G on ref9-xen32.f.o should I go any higher? Sure. Right now, I don't know of any reason

Re: PV i386 patch

2011-12-20 Thread Alan Cox
On 12/20/2011 13:57, Sean Bruno wrote: On Tue, 2011-12-20 at 10:49 -0800, Alan Cox wrote: On 12/20/2011 07:28, Sean Bruno wrote: The code that panics shouldn't even exist in the Xen pmap. Try the attached patch. Alan Indeed how on earth did we ever use this stuff? :-) Tested

Re: PV i386 patch

2011-12-20 Thread Alan Cox
On 12/20/2011 14:12, Colin Percival wrote: On 12/20/11 10:49, Alan Cox wrote: Do either of you know if there is a PR in gnats for this 768 MB limitation bug that I should mention in the commit log? The only one I'm aware of is kern/153789. Oops. I've already committed the chang

Re: PV i386 patch

2011-12-27 Thread Alan Cox
On 12/23/2011 16:25, Sean Bruno wrote: On Wed, 2011-12-21 at 12:47 -0800, Alan Cox wrote: Can you please try the attached patch? I'm trying to reduce the number of differences between the native and Xen pmap implementations. Alan Without really looking at the output, I note that

Re: PV i386 patch

2011-12-29 Thread Alan Cox
Please try this patch. It eliminates a race condition that might actually account for some of the crashes in FreeBSD >= 9 on Xen. Alan Index: i386/xen/pmap.c === --- i386/xen/pmap.c (revision 228935) +++ i386/xen/pmap.c (w

Re: PV i386 patch

2011-12-29 Thread Alan Cox
On 12/29/2011 16:28, Sean Bruno wrote: On Thu, 2011-12-29 at 12:22 -0800, Alan Cox wrote: Please try this patch. It eliminates a race condition that might actually account for some of the crashes in FreeBSD>= 9 on Xen. Alan ref10-xen32.freebsd.org has this applied now. Looks ok to