Re: [Xen-devel] uint64_aligned_t not compatible across gcc versions

[Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>]

On 28/8/06 3:46 pm, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:

> gcc up to 3.3.x doesn't honor __attribute__((__aligned(..))) on typedef-s (and
> in my opinion this was the correct behavior, as a typedef and its original
> type
> must be fully exchangeable in all their uses). For this reason, I'd recommend
> changing the definition of this type on x86_32 with below patch.

Thanks, I did wonder.

> Further looking into how it is being used and therefore into the uses of
> XEN_GUEST_HANDLE_64 I am not really clear about the intentions of that
> macro and its companions: Reserving a 64-bit slot for a pointer would seem
> nice for doing the compatibility layer, but turns out useless if the 32-bit
> native implementation doesn't check the upper half of passed in values, as
> that implies that the 64-bit compatibility layer must not assume these upper
> bits are all zero or else it wouldn't be binary compatible.

Yes, the aim is to avoid the need for a 32-on-64 shim for the domctl and
sysctl hypercalls. I guess we'll end up with a semi-automated scheme for the
other hypercalls anyway, but since we don't guarantee compatibility on those
interfaces we may as well make our lives as easy as possible.

Guest handle fields are *only* written by the set_xen_guest_handle() macro
in the tools -- you'll see that I modified that macro to zap the high bits
if the field is 8 bytes. This should be sufficient?

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel