Re: [Xen-devel] [PATCH] PVH linux: Use ballooning to allocate granttable pages

From: Mukesh Rathor
Date: Fri Feb 01 2013 - 18:51:57 EST


On Fri, 1 Feb 2013 14:00:58 -0800
Mukesh Rathor <mukesh.rathor@xxxxxxxxxx> wrote:

> On Thu, 31 Jan 2013 18:44:46 -0800
> Mukesh Rathor <mukesh.rathor@xxxxxxxxxx> wrote:
>
> > On Thu, 31 Jan 2013 18:30:15 -0800
> > Mukesh Rathor <mukesh.rathor@xxxxxxxxxx> wrote:
> >
> > > This patch fixes a fixme in Linux to use
> > > alloc_xenballooned_pages() to allocate pfns for grant table pages
> > > instead of kmalloc. This also simplifies add to physmap on the
> > > xen side a bit.
> >
> > Looking at it again, I realized rc should be signed in
> > gnttab_resume(). Below again. Thanks.
>
> Konrad, Please hold off on this patch. I discovered an issue on the
> domU side with this change. I'm currently investigating if it's
> related.

Ah right, I forgot the pfn's from balloon may not be always contigous.
Besides these are special pfns so to speak, so in gnttab_map()
virt_to_pfn doesn't work.

I'm gonna have to create a separate gnttab map routine for pvh case, it
appears. Shouldn't be too bad tho.

thanks,
Mukesh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/