Re: [Xen-devel] [PATCH v4 1/6] xen: improve changes to xen_add_to_physmap

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2012-09-14 at 14:47 +0100, Ian Campbell wrote:
> On Fri, 2012-09-14 at 14:17 +0100, Ian Campbell wrote:
> > (forgot to CC Mukesh on y last mail like I said it would, I've done it
> > now and will bounce the last one to you Mukesh)
> 
> &%$Â% I've done it again. Actually CCing Mukesh this time!

OK, it's definitely not me, I'm certain I CCd him this time. Where on
earth are they going!

> 
> > 
> > On Fri, 2012-09-14 at 13:59 +0100, Ian Campbell wrote:
> > > 
> > > Might we prefer to have a batched version of this call? I don't think
> > > we can shoehorn the necessary fields into xen_add_to_physmap_t though.
> > 
> > Actually, talking about this we Stefano we think we can. Since both idx
> > and gpfn are unsigned longs they can become unions with a GUEST_HANDLE
> > without changing the ABI on x86_32 or x86_64.

Except we need both foreign_domid and size, which currently overlap, and
there's no other spare bits. Damn.

Looks like XENMEM_add_to_physmap_range is the only option :-(

Ian.

> > 
> > If we do this then we should do away with the singleton
> > XENMAPSPACE_gmfn_foreign and just make it a batched interface.
> > 
> > Ian.
> > 
> > 
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@xxxxxxxxxxxxx
> > http://lists.xen.org/xen-devel
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel