diff options
| author | Warner Losh <imp@FreeBSD.org> | 2023-02-16 16:58:33 +0000 |
|---|---|---|
| committer | Warner Losh <imp@FreeBSD.org> | 2023-02-19 03:14:36 +0000 |
| commit | bae5c73a5ffa237e437605bbf2216a574d1dbda2 (patch) | |
| tree | 1a7ee3582ef437126dfb99d99f6cef382f0f36b3 | |
| parent | 0b69409270e47fc90fa7141098c2621e4df1c45c (diff) | |
| download | src-bae5c73a5ffa237e437605bbf2216a574d1dbda2.tar.gz src-bae5c73a5ffa237e437605bbf2216a574d1dbda2.zip | |
sys/_endian.h: Move powerpc workaround into _endian.h
Powerpc compilers often define _BIG_ENDIAN or _LITTLE_ENDIAN depending
on the byte order of the target. This interfers with the endian.h and
sys/endian.h APIs which expect those two to be 4321 or 1234.
Unconditionally undefine these two before we define them to work around
this issue. By including endian.h, the programmer is saying they want
this API not the native one. This matches historic practice as well.
Fixes: e35448a901aa
Sponsored by: Netflix
Noticed by: powerpc, powerpc64 CI jobs
Approved by: re (cperciva)
(cherry picked from commit 6f23df1ae7f6ec3cd06a15c87f0aaaa770b2941b)
(cherry picked from commit 07e39710b170859ef6a57185f9a775973f3eeffe)
| -rw-r--r-- | sys/sys/_endian.h | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sys/sys/_endian.h b/sys/sys/_endian.h index ff909f532663..7ac39386e2e1 100644 --- a/sys/sys/_endian.h +++ b/sys/sys/_endian.h @@ -41,9 +41,13 @@ /* * Definitions for byte order, according to byte significance from low - * address to high. + * address to high. We undefine any prior definition of them because + * powerpc compilers define _LITTLE_ENDIAN and _BIG_ENDIAN to mean + * something else. */ +#undef _LITTLE_ENDIAN #define _LITTLE_ENDIAN __ORDER_LITTLE_ENDIAN__ /* LSB first: 1234 */ +#undef _BIG_ENDIAN #define _BIG_ENDIAN __ORDER_BIG_ENDIAN__ /* MSB first: 4321 */ #define _PDP_ENDIAN __ORDER_PDP_ENDIAN__ /* LSB first in word, * MSW first in long: 3412 */ |