diff options
| author | Ben Dooks <ben.dooks@codethink.co.uk> | 2025-07-04 13:29:38 +0100 |
|---|---|---|
| committer | Anup Patel <anup@brainfault.org> | 2025-07-20 21:15:10 +0530 |
| commit | 663b05a5f745f6b1360f073587bb57fe9af6a40a (patch) | |
| tree | 752b5828f654624ce72ef272adb36d42f76af137 | |
| parent | edfbc1285dd9ac624665fe9fa5de26437d61c1eb (diff) | |
| download | opensbi-663b05a5f745f6b1360f073587bb57fe9af6a40a.zip opensbi-663b05a5f745f6b1360f073587bb57fe9af6a40a.tar.gz opensbi-663b05a5f745f6b1360f073587bb57fe9af6a40a.tar.bz2 | |
include: sbi: fix swap errors with newer gcc -Werror=sequence-point
The BSWAPxx() macros are now throwing the following warnings with
newer gcc versions. This is due to throwing an argument in that may
be evaluated more than one (I think) and therefore things like the
example below should be avoided.
Fix by making a set of BSWAPxx() wrappers which specifically only
evaluate 'x' once.
In file included lib/sbi/sbi_mpxy.c:21:
lib/sbi/sbi_mpxy.c: In function ‘sbi_mpxy_write_attrs’:
ib/sbi/sbi_mpxy.c:632:63: error: operation on ‘mem_idx’ may be undefined [-Werror=sequence-point]
632 | attr_val = le32_to_cpu(mem_ptr[mem_idx++]);
| ~~~~~~~^~
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
Reviewed-by: Rahul Pathak <rahul@summations.net>
Reviewed-by: Xiang W <wxjstz@126.com>
Link: https://lore.kernel.org/r/20250704122938.897832-1-ben.dooks@codethink.co.uk
Signed-off-by: Anup Patel <anup@brainfault.org>
| -rw-r--r-- | include/sbi/sbi_byteorder.h | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/include/sbi/sbi_byteorder.h b/include/sbi/sbi_byteorder.h index 2b4981e..ed7cad1 100644 --- a/include/sbi/sbi_byteorder.h +++ b/include/sbi/sbi_byteorder.h @@ -14,13 +14,13 @@ # define _conv_cast(type, val) ((type)(val)) #endif -#define BSWAP16(x) ((((x) & 0x00ff) << 8) | \ +#define __BSWAP16(x) ((((x) & 0x00ff) << 8) | \ (((x) & 0xff00) >> 8)) -#define BSWAP32(x) ((((x) & 0x000000ff) << 24) | \ +#define __BSWAP32(x) ((((x) & 0x000000ff) << 24) | \ (((x) & 0x0000ff00) << 8) | \ (((x) & 0x00ff0000) >> 8) | \ (((x) & 0xff000000) >> 24)) -#define BSWAP64(x) ((((x) & 0x00000000000000ffULL) << 56) | \ +#define __BSWAP64(x) ((((x) & 0x00000000000000ffULL) << 56) | \ (((x) & 0x000000000000ff00ULL) << 40) | \ (((x) & 0x0000000000ff0000ULL) << 24) | \ (((x) & 0x00000000ff000000ULL) << 8) | \ @@ -29,6 +29,10 @@ (((x) & 0x00ff000000000000ULL) >> 40) | \ (((x) & 0xff00000000000000ULL) >> 56)) +#define BSWAP64(x) ({ uint64_t _sv = (x); __BSWAP64(_sv); }) +#define BSWAP32(x) ({ uint32_t _sv = (x); __BSWAP32(_sv); }) +#define BSWAP16(x) ({ uint16_t _sv = (x); __BSWAP16(_sv); }) + #if __BYTE_ORDER__ == __ORDER_LITTLE_ENDIAN__ /* CPU(little-endian) */ #define cpu_to_be16(x) _conv_cast(uint16_t, BSWAP16(x)) #define cpu_to_be32(x) _conv_cast(uint32_t, BSWAP32(x)) |