Re: [PATCH v6 01/10] xen: vnuma topology and subop hypercalls

[Dario Faggioli <dario.faggioli@citrix.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1624 bytes --]

On dom, 2014-07-20 at 16:59 +0100, Wei Liu wrote:
> On Sun, Jul 20, 2014 at 09:16:11AM -0400, Elena Ufimtseva wrote:

> > >> +struct vnuma_topology_info {
> > >> +    /* IN */
> > >> +    domid_t domid;
> > >> +    /* IN/OUT */
> > >> +    unsigned int nr_vnodes;
> > >> +    unsigned int nr_vcpus;
> > >> +    /* OUT */
> > >> +    union {
> > >> +        XEN_GUEST_HANDLE(uint) h;
> > >> +        uint64_t pad;
> > >> +    } vdistance;
> > >> +    union {
> > >> +        XEN_GUEST_HANDLE(uint) h;
> > >> +        uint64_t pad;
> > >> +    } vcpu_to_vnode;
> > >> +    union {
> > >> +        XEN_GUEST_HANDLE(vmemrange_t) h;
> > >> +        uint64_t pad;
> > >> +    } vmemrange;
> > >
> > > Why do you need to use union? The other interface you introduce in this
> > > patch doesn't use union.
> > 
> > This is one is for making sure on 32 and 64 bits the structures are of
> > the same size.
> > 
> 
> I can see other similiar occurences of XEN_GUEST_HANDLE don't need
> padding. Did I miss something?
> 
I remember this coming up during review of an earlier version of Elena's
series, and I think I also remember the union with padding solution
being suggested, but I don't remember which round it was, and who
suggested it... Elena, up for some digging in your inbox (or xen-devel
archives)? :-P

Regards,
Dario

-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

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