Re: [Xen-ia64-devel] PATCH [resend]: more case handled in mmio.c

[Tristan Gingold <Tristan.Gingold@xxxxxxxx>]

Le Vendredi 21 Avril 2006 10:01, Xu, Anthony a écrit :
> From: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
>
> >[mailto:xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Xu,
> > Anthony Sent: 2006?4?21? 15:55
> >To: Tristan Gingold; xen-ia64-devel@xxxxxxxxxxxxxxxxxxx; Alex Williamson
> >Subject: RE: [Xen-ia64-devel] PATCH [resend]: more case handled in mmio.c
> >
> >>From: Tristan Gingold [mailto:Tristan.Gingold@xxxxxxxx]
> >>Sent: 2006?4?21? 15:53
> >>To: Xu, Anthony; xen-ia64-devel@xxxxxxxxxxxxxxxxxxx; Alex Williamson
> >>Subject: Re: [Xen-ia64-devel] PATCH [resend]: more case handled in mmio.c
> >>
> >>Le Vendredi 21 Avril 2006 09:45, Xu, Anthony a écrit :
> >>> One more comment.
> >>>         /* Write high word.
> >>>                 FIXME: this is a kludge!  */
> >>>              v.u.bits[1] &= 0x3ffff;
> >>> This seems not be necessary.
> >>> v is got by stf.spill,
> >>> stf.spill makes sure the high 47 bits of v.u.bits[1] is 0.
> >>
> >>Yes, this is correct.
> >>In fact the comment is not correctly placed: the kludge is to do two
> >>mmio_access.
> >
> >The largest size of mmio is 8 byte, if not using pointer in ioreq
> > structure. And I don't know whether pointer works.
> >Is that possible to change "uint64_t data" to "long long data"?
> >Long long type is 16 byte, which is largest size of mmio in ia64.
>
> Sorry, long long type is still 8 byte.
> Do you know which type is 16 byte in ia64?
I don't think such a type exists on gcc/ia64.  Maybe a vector type ?
I think it is better to use a struct.

Tristan.



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