[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.