[PATCH 0/5] powerpc: Get rid of redundant arch specific swab functions

David Laight David.Laight at ACULAB.COM
Wed Feb 4 22:54:39 AEDT 2015


From: David Gibson
> arch/powerpc/include/asm/swab.h includes some powerpc specific
> byteswapping functions, which are implemented in terms of powerpc's
> built in byte reversed load/store instructions.  There are two problems with this:
> 
> 1) They're not necessary - gcc is perfectly capable of generating the
>    byte-reversed load and store instructions when using the normal,
>    generic byteswapping functions (tested with gcc (GCC) 4.8.3
>    20140911 (Red Hat 4.8.3-9))

Should you be worrying about older versions of gcc?
IIRC the internal byteswap 'stuff' is relatively recent (like
the last couple of years) so people building current kernels
on older distributions might have issues.

	David



More information about the Linuxppc-dev mailing list