diff options
author | kfraser@localhost.localdomain <kfraser@localhost.localdomain> | 2007-09-11 12:14:38 +0100 |
---|---|---|
committer | kfraser@localhost.localdomain <kfraser@localhost.localdomain> | 2007-09-11 12:14:38 +0100 |
commit | 9caf62ac1390ad6fc8895ae442023dedd2c3597b (patch) | |
tree | 5c31e5f0416a2f740a7ce97fa7422d0adc55388e /xen/include/asm-x86/byteorder.h | |
parent | f7f39e3ac3944a3a56cfc57696347c485aea941c (diff) | |
download | xen-9caf62ac1390ad6fc8895ae442023dedd2c3597b.tar.gz xen-9caf62ac1390ad6fc8895ae442023dedd2c3597b.tar.bz2 xen-9caf62ac1390ad6fc8895ae442023dedd2c3597b.zip |
x86: Clean up asm keyword usage (asm volatile rather than __asm__
__volatile__ in most places) and ensure we use volatile keyword
wherever we have an asm stmt that produces outputs but has other
unspecified side effects or dependencies other than the
explicitly-stated inputs.
Also added volatile in a few places where its not strictly necessary
but where it's unlikely to produce worse code and it makes our
intentions perfectly clear.
The original problem this patch fixes was tracked down by Joseph
Cihula <joseph.cihula@intel.com>.
Signed-off-by: Keir Fraser <keir@xensource.com>
Diffstat (limited to 'xen/include/asm-x86/byteorder.h')
-rw-r--r-- | xen/include/asm-x86/byteorder.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xen/include/asm-x86/byteorder.h b/xen/include/asm-x86/byteorder.h index eb52ae2f11..1f77e502a5 100644 --- a/xen/include/asm-x86/byteorder.h +++ b/xen/include/asm-x86/byteorder.h @@ -20,7 +20,7 @@ static inline __attribute_const__ __u64 ___arch__swab64(__u64 val) asm("bswapl %0 ; bswapl %1 ; xchgl %0,%1" : "=r" (v.s.a), "=r" (v.s.b) : "0" (v.s.a), "1" (v.s.b)); - return v.u; + return v.u; } /* Do not define swab16. Gcc is smart enough to recognize "C" version and |