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

Re: [Xen-devel] [PATCH v3 1/8] public / x86: Introduce __HYPERCALL_dm_op...



> -----Original Message-----
> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
> Sent: 13 January 2017 10:31
> To: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>; Paul Durrant
> <Paul.Durrant@xxxxxxxxxx>
> Cc: Ian Jackson <Ian.Jackson@xxxxxxxxxx>; Jennifer Herbert
> <jennifer.herbert@xxxxxxxxxx>; Wei Liu <wei.liu2@xxxxxxxxxx>; xen-
> devel@xxxxxxxxxxxxxxxxxxxx; Daniel DeGraaf <dgdegra@xxxxxxxxxxxxx>
> Subject: RE: [PATCH v3 1/8] public / x86: Introduce __HYPERCALL_dm_op...
> 
> >>> On 13.01.17 at 10:05, <Paul.Durrant@xxxxxxxxxx> wrote:
> >> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
> >> Sent: 12 January 2017 16:29
> >> >>> On 12.01.17 at 17:10, <Paul.Durrant@xxxxxxxxxx> wrote:
> >> >> The userspace side should be
> >> >>
> >> >> struct xen_dm_op_buf {
> >> >>     void *h;
> >> >>     size_t size;
> >> >> }
> >> >>
> >> >> which will work sensibly for 32bit and 64bit userspace, and futureproof
> >> >> (for when 128bit turns up).  Its size is also a power of two which
> >> >> avoids alignment issues in the array.
> >> >>
> >> >> The kernel already has to parse this structure anyway, and will know
> the
> >> >> bitness of its userspace process.  We could easily (at this point)
> >> >> require the kernel to turn it into the kernels bitness for forwarding on
> >> >> to Xen, which covers the 32bit userspace under a 64bit kernel problem,
> >> >> in a way which won't break the hypercall ABI when 128bit comes along.
> >>
> >> But that won't cover a 32-bit kernel.
> >>
> >
> > Do we need to care about a 32-bit kernel for a tools-only hypercall? I
> thought
> > a toolstack already had to be (at least) 64-bit to match Xen.
> 
> Why would that be? Didn't XenServer for a long time run with a
> 32-bit Dom0 on top of a 64-bit Xen? See also domctl/sysctl.
> 

Ok, I thought the landscape had changed since then. If that's the case then a 
64 bit handle seems sensible and I guess we have to deal with 128 bit when that 
happens.

  Paul

> Jan


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

 


Rackspace

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