[ltt-dev] [PATCH 07/11] add uatomic_gcc.h, use it for default definitions
pbonzini at redhat.com
Mon Feb 15 11:55:54 EST 2010
On 02/15/2010 05:51 PM, Mathieu Desnoyers wrote:
>>> I was thinking of doing (3) with the additional twist that S390
>>> would count as a "new architecture". On it, GCC is providing
>>> really everything that is needed and furthermore, unlike PPC,
>>> your testers surely have a new-enough GCC because they're using
> Hrm. As things stand, I would recommend plainly going with (3),
> using gcc builtins only for the new architectures which are not
> supported at the moment, being ready to merge compatibility code for
> these architectures if we see the need for it at some point.
> From this point of view, moving s390 from the custom operations to
> gcc builtins seems like a step backward.
Fine, will adjust.
> I'd prefer to look into the api_gcc.h issues and fix those so we can
> ensure that sparc64 and s390 are OK with older gcc.
Requiring GCC for testing purposes doesn't seem too bad, but I guess
this latest paragraph was a kind of "patches are welcome" thing. :-)
More information about the lttng-dev