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

From: David Laight
Date: Wed Feb 04 2015 - 06:55:33 EST


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

N‹§²æ¸›yú²X¬¶ÇvØ–)Þ{.nlj·¥Š{±‘êX§¶›¡Ü}©ž²ÆzÚj:+v‰¨¾«‘êZ+€Êzf£¢·hšˆ§~†­†Ûÿû®w¥¢¸?™¨è&¢)ßf”ùy§m…á«a¶Úÿ 0¶ìå