This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH] Speed up superpage mapping, take 2

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Speed up superpage mapping, take 2
From: Dave McCracken <dcm@xxxxxxxx>
Date: Wed, 26 May 2010 07:20:16 -0500
Cc: Xen Developers List <xen-devel@xxxxxxxxxxxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxxxx>
Delivery-date: Wed, 26 May 2010 05:21:24 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C822903F.15E62%keir.fraser@xxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <C822903F.15E62%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: KMail/1.13.3 (Linux/2.6.32; KDE/4.4.3; x86_64; ; )
On Wednesday, May 26, 2010, Keir Fraser wrote:
> On 26/05/2010 08:36, "Jan Beulich" <JBeulich@xxxxxxxxxx> wrote:
> >> -#define __HYPERVISOR_VIRT_START_PAE    0xF5800000
> >> -#define __MACH2PHYS_VIRT_START_PAE     0xF5800000
> >> -#define __MACH2PHYS_VIRT_END_PAE       0xF6800000
> >> +#define __HYPERVISOR_VIRT_START_PAE    0xF5600000
> >> +#define __MACH2PHYS_VIRT_START_PAE     0xF5600000
> >> +#define __MACH2PHYS_VIRT_END_PAE       0xF6600000
> >> 
> >>  #define HYPERVISOR_VIRT_START_PAE      \
> >>  
> >>      mk_unsigned_long(__HYPERVISOR_VIRT_START_PAE)
> >>  
> >>  #define MACH2PHYS_VIRT_START_PAE       \
> > 
> > is absolutely forbidden: You cannot lower the start address of
> > the hypervisor address space hole - no kernel can be expected
> > to be able to deal with that. I think you will need to make the
> > feature 64-bit specific, unless you can find a 2Mb range within
> > the existing hole (but I doubt there is any room left), or unless
> > you map the table dynamically as needed for 32-bits.
> Sixty-four-bit-only is *fine* by me. But yes, the patch will need another
> rev for this.

Sorry, I should have picked up on this.  64 bit only is fine by me too.  I'll 
respin the patch.

Dave McCracken

Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>