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

Re: [Xen-devel] dom0 pvops crash



On Sun, 2010-02-07 at 16:42 -0500, Ian Campbell wrote:
> 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.

is_running_on_xen()

Daniel




_______________________________________________
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®.