WARNING - OLD ARCHIVES

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/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] dom0 pvops crash

On Sun, 2010-02-07 at 19:35 +0000, Pasi Kärkkäinen wrote: 
> On Wed, Jan 27, 2010 at 05:26:13PM +0000, Ian Campbell wrote:
> > diff --git a/arch/x86/mm/pgtable.c b/arch/x86/mm/pgtable.c
> > index 65215ab..49f8e83 100644
> > --- a/arch/x86/mm/pgtable.c
> > +++ b/arch/x86/mm/pgtable.c
> > @@ -28,7 +28,10 @@ pgtable_t pte_alloc_one(struct mm_struct *mm, unsigned 
> > long address)
> >     struct page *pte;
> >  
> >  #ifdef CONFIG_HIGHPTE
> > -   pte = alloc_pages(PGALLOC_GFP | __GFP_HIGHMEM, 0);
> > +   if (is_xen_domain())
> > +           pte = alloc_pages(PGALLOC_GFP, 0);
> > +   else
> > +           pte = alloc_pages(PGALLOC_GFP | __GFP_HIGHMEM, 0);
> >  #else
> >     pte = alloc_pages(PGALLOC_GFP, 0);
> >  #endif
> > 
> 
> I just tried this patch, but it fails to compile:
> 
> arch/x86/mm/pgtable.c: In function 'pte_alloc_one':
> arch/x86/mm/pgtable.c:19: error: implicit declaration of function 
> 'is_xen_domain'
> make[2]: *** [arch/x86/mm/pgtable.o] Error 1
> make[1]: *** [arch/x86/mm] Error 2
> make: *** [arch/x86] Error 2
> 
> I tried grepping around for that function but didn't find it from any header..

IIRC on some kernels it was called just xen_domain(), I'm not sure but I
think my patch was against plain 2.6.32. I think the function (whatever
it is called) also moved around in the headers recently.

For the purposes of testing you could just ignore the conditional part
of the patch and simply drop the __GFP_HIGHMEM altogether.

Ian.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel