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

Re: [Xen-devel] [patch] make evtchn_upcall_pending arch-specific type




On 13 Jun 2006, at 21:02, Hollis Blanchard wrote:

You'll notice I added an alignment attribute to that field because it
really really needs to be aligned for us. This has only been
build-tested on x86!

If it's acceptable, please apply.

Hide evtchn_upcall_pending test-and-set accesses behind a wrapper.
Signed-off-by: Hollis Blanchard <hollisb@xxxxxxxxxx>

It would probably be better to move the TAS of the upcall_pending flag into evtchn_notify(). Avoids expanding the arch-specific interface further.

We have avoided adding gcc extensions to core public header files as it prevents easy use of the header files by some other projects (e.g., solaris port, I believe). You should instead add a suitable BUILD_BUG_ON() in arch/ppc -- in any case, if field offsets change in future you break backward compatibility, even if new field alignment does happen to still be 'okay' in that one case.

 -- Keir


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


 


Rackspace

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