* [PATCH] exec: Rename target_words_bigendian() -> target_big_endian()
@ 2025-04-17 21:00 Philippe Mathieu-Daudé
2025-04-17 21:01 ` Michael S. Tsirkin
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2025-04-17 21:00 UTC (permalink / raw)
To: qemu-devel
Cc: David Hildenbrand, Zhao Liu, Paolo Bonzini, Peter Xu,
Gerd Hoffmann, Eduardo Habkost, Marcel Apfelbaum,
Philippe Mathieu-Daudé, Michael S. Tsirkin, Yanan Wang,
Richard Henderson, Laurent Vivier, Fabiano Rosas
In commit 98ed8ecfc9d ("exec: introduce target_words_bigendian()
helper") target_words_bigendian() was matching the definition it
was depending on (TARGET_WORDS_BIGENDIAN). Later in commit
ee3eb3a7ce7 ("Replace TARGET_WORDS_BIGENDIAN") the definition was
renamed as TARGET_BIG_ENDIAN but we didn't update the helper.
Do it now mechanically using:
$ sed -i -e s/target_words_bigendian/target_big_endian/g \
$(git grep -wl target_words_bigendian)
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
include/exec/tswap.h | 12 ++++++------
system/memory-internal.h | 2 +-
cpu-target.c | 4 ++--
hw/core/cpu-system.c | 2 +-
hw/display/vga.c | 2 +-
hw/virtio/virtio.c | 2 +-
system/memory.c | 4 ++--
system/qtest.c | 2 +-
8 files changed, 15 insertions(+), 15 deletions(-)
diff --git a/include/exec/tswap.h b/include/exec/tswap.h
index 84060a49994..49511f26117 100644
--- a/include/exec/tswap.h
+++ b/include/exec/tswap.h
@@ -11,15 +11,15 @@
#include "qemu/bswap.h"
/**
- * target_words_bigendian:
+ * target_big_endian:
* Returns true if the (default) endianness of the target is big endian,
* false otherwise. Common code should normally never need to know about the
* endianness of the target, so please do *not* use this function unless you
* know very well what you are doing!
*/
-bool target_words_bigendian(void);
+bool target_big_endian(void);
#ifdef COMPILING_PER_TARGET
-#define target_words_bigendian() TARGET_BIG_ENDIAN
+#define target_big_endian() TARGET_BIG_ENDIAN
#endif
/*
@@ -29,7 +29,7 @@ bool target_words_bigendian(void);
#ifdef COMPILING_PER_TARGET
#define target_needs_bswap() (HOST_BIG_ENDIAN != TARGET_BIG_ENDIAN)
#else
-#define target_needs_bswap() (HOST_BIG_ENDIAN != target_words_bigendian())
+#define target_needs_bswap() (HOST_BIG_ENDIAN != target_big_endian())
#endif /* COMPILING_PER_TARGET */
static inline uint16_t tswap16(uint16_t s)
@@ -83,7 +83,7 @@ static inline void tswap64s(uint64_t *s)
/* Return ld{word}_{le,be}_p following target endianness. */
#define LOAD_IMPL(word, args...) \
do { \
- if (target_words_bigendian()) { \
+ if (target_big_endian()) { \
return glue(glue(ld, word), _be_p)(args); \
} else { \
return glue(glue(ld, word), _le_p)(args); \
@@ -120,7 +120,7 @@ static inline uint64_t ldn_p(const void *ptr, int sz)
/* Call st{word}_{le,be}_p following target endianness. */
#define STORE_IMPL(word, args...) \
do { \
- if (target_words_bigendian()) { \
+ if (target_big_endian()) { \
glue(glue(st, word), _be_p)(args); \
} else { \
glue(glue(st, word), _le_p)(args); \
diff --git a/system/memory-internal.h b/system/memory-internal.h
index 085e81a9fe4..29717b3c58f 100644
--- a/system/memory-internal.h
+++ b/system/memory-internal.h
@@ -45,7 +45,7 @@ static inline bool devend_big_endian(enum device_endian end)
DEVICE_HOST_ENDIAN != DEVICE_BIG_ENDIAN);
if (end == DEVICE_NATIVE_ENDIAN) {
- return target_words_bigendian();
+ return target_big_endian();
}
return end == DEVICE_BIG_ENDIAN;
}
diff --git a/cpu-target.c b/cpu-target.c
index e018acbf71a..b5645ff0dbb 100644
--- a/cpu-target.c
+++ b/cpu-target.c
@@ -86,8 +86,8 @@ void cpu_abort(CPUState *cpu, const char *fmt, ...)
abort();
}
-#undef target_words_bigendian
-bool target_words_bigendian(void)
+#undef target_big_endian
+bool target_big_endian(void)
{
return TARGET_BIG_ENDIAN;
}
diff --git a/hw/core/cpu-system.c b/hw/core/cpu-system.c
index 82b68b8927d..3c84176a0c5 100644
--- a/hw/core/cpu-system.c
+++ b/hw/core/cpu-system.c
@@ -133,7 +133,7 @@ bool cpu_virtio_is_big_endian(CPUState *cpu)
if (cpu->cc->sysemu_ops->virtio_is_big_endian) {
return cpu->cc->sysemu_ops->virtio_is_big_endian(cpu);
}
- return target_words_bigendian();
+ return target_big_endian();
}
GuestPanicInformation *cpu_get_crash_info(CPUState *cpu)
diff --git a/hw/display/vga.c b/hw/display/vga.c
index b01f67c65fb..20475ebbd31 100644
--- a/hw/display/vga.c
+++ b/hw/display/vga.c
@@ -2264,7 +2264,7 @@ bool vga_common_init(VGACommonState *s, Object *obj, Error **errp)
* into a device attribute set by the machine/platform to remove
* all target endian dependencies from this file.
*/
- s->default_endian_fb = target_words_bigendian();
+ s->default_endian_fb = target_big_endian();
s->big_endian_fb = s->default_endian_fb;
vga_dirty_log_start(s);
diff --git a/hw/virtio/virtio.c b/hw/virtio/virtio.c
index 85110bce374..8fbf1716b88 100644
--- a/hw/virtio/virtio.c
+++ b/hw/virtio/virtio.c
@@ -2248,7 +2248,7 @@ int virtio_set_status(VirtIODevice *vdev, uint8_t val)
static enum virtio_device_endian virtio_default_endian(void)
{
- if (target_words_bigendian()) {
+ if (target_big_endian()) {
return VIRTIO_DEVICE_ENDIAN_BIG;
} else {
return VIRTIO_DEVICE_ENDIAN_LITTLE;
diff --git a/system/memory.c b/system/memory.c
index 7e2f16f4e95..67e433095b4 100644
--- a/system/memory.c
+++ b/system/memory.c
@@ -2575,7 +2575,7 @@ void memory_region_add_eventfd(MemoryRegion *mr,
unsigned i;
if (size) {
- MemOp mop = (target_words_bigendian() ? MO_BE : MO_LE) | size_memop(size);
+ MemOp mop = (target_big_endian() ? MO_BE : MO_LE) | size_memop(size);
adjust_endianness(mr, &mrfd.data, mop);
}
memory_region_transaction_begin();
@@ -2611,7 +2611,7 @@ void memory_region_del_eventfd(MemoryRegion *mr,
unsigned i;
if (size) {
- MemOp mop = (target_words_bigendian() ? MO_BE : MO_LE) | size_memop(size);
+ MemOp mop = (target_big_endian() ? MO_BE : MO_LE) | size_memop(size);
adjust_endianness(mr, &mrfd.data, mop);
}
memory_region_transaction_begin();
diff --git a/system/qtest.c b/system/qtest.c
index 523a0479959..c675fa2cb30 100644
--- a/system/qtest.c
+++ b/system/qtest.c
@@ -693,7 +693,7 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
qtest_send(chr, "OK\n");
} else if (strcmp(words[0], "endianness") == 0) {
- if (target_words_bigendian()) {
+ if (target_big_endian()) {
qtest_sendf(chr, "OK big\n");
} else {
qtest_sendf(chr, "OK little\n");
--
2.47.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] exec: Rename target_words_bigendian() -> target_big_endian()
2025-04-17 21:00 [PATCH] exec: Rename target_words_bigendian() -> target_big_endian() Philippe Mathieu-Daudé
@ 2025-04-17 21:01 ` Michael S. Tsirkin
2025-04-17 21:37 ` Pierrick Bouvier
2025-04-25 10:44 ` Philippe Mathieu-Daudé
2 siblings, 0 replies; 4+ messages in thread
From: Michael S. Tsirkin @ 2025-04-17 21:01 UTC (permalink / raw)
To: Philippe Mathieu-Daudé
Cc: qemu-devel, David Hildenbrand, Zhao Liu, Paolo Bonzini, Peter Xu,
Gerd Hoffmann, Eduardo Habkost, Marcel Apfelbaum, Yanan Wang,
Richard Henderson, Laurent Vivier, Fabiano Rosas
On Thu, Apr 17, 2025 at 11:00:25PM +0200, Philippe Mathieu-Daudé wrote:
> In commit 98ed8ecfc9d ("exec: introduce target_words_bigendian()
> helper") target_words_bigendian() was matching the definition it
> was depending on (TARGET_WORDS_BIGENDIAN). Later in commit
> ee3eb3a7ce7 ("Replace TARGET_WORDS_BIGENDIAN") the definition was
> renamed as TARGET_BIG_ENDIAN but we didn't update the helper.
> Do it now mechanically using:
>
> $ sed -i -e s/target_words_bigendian/target_big_endian/g \
> $(git grep -wl target_words_bigendian)
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
> ---
> include/exec/tswap.h | 12 ++++++------
> system/memory-internal.h | 2 +-
> cpu-target.c | 4 ++--
> hw/core/cpu-system.c | 2 +-
> hw/display/vga.c | 2 +-
> hw/virtio/virtio.c | 2 +-
> system/memory.c | 4 ++--
> system/qtest.c | 2 +-
> 8 files changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/include/exec/tswap.h b/include/exec/tswap.h
> index 84060a49994..49511f26117 100644
> --- a/include/exec/tswap.h
> +++ b/include/exec/tswap.h
> @@ -11,15 +11,15 @@
> #include "qemu/bswap.h"
>
> /**
> - * target_words_bigendian:
> + * target_big_endian:
> * Returns true if the (default) endianness of the target is big endian,
> * false otherwise. Common code should normally never need to know about the
> * endianness of the target, so please do *not* use this function unless you
> * know very well what you are doing!
> */
> -bool target_words_bigendian(void);
> +bool target_big_endian(void);
> #ifdef COMPILING_PER_TARGET
> -#define target_words_bigendian() TARGET_BIG_ENDIAN
> +#define target_big_endian() TARGET_BIG_ENDIAN
> #endif
>
> /*
> @@ -29,7 +29,7 @@ bool target_words_bigendian(void);
> #ifdef COMPILING_PER_TARGET
> #define target_needs_bswap() (HOST_BIG_ENDIAN != TARGET_BIG_ENDIAN)
> #else
> -#define target_needs_bswap() (HOST_BIG_ENDIAN != target_words_bigendian())
> +#define target_needs_bswap() (HOST_BIG_ENDIAN != target_big_endian())
> #endif /* COMPILING_PER_TARGET */
>
> static inline uint16_t tswap16(uint16_t s)
> @@ -83,7 +83,7 @@ static inline void tswap64s(uint64_t *s)
> /* Return ld{word}_{le,be}_p following target endianness. */
> #define LOAD_IMPL(word, args...) \
> do { \
> - if (target_words_bigendian()) { \
> + if (target_big_endian()) { \
> return glue(glue(ld, word), _be_p)(args); \
> } else { \
> return glue(glue(ld, word), _le_p)(args); \
> @@ -120,7 +120,7 @@ static inline uint64_t ldn_p(const void *ptr, int sz)
> /* Call st{word}_{le,be}_p following target endianness. */
> #define STORE_IMPL(word, args...) \
> do { \
> - if (target_words_bigendian()) { \
> + if (target_big_endian()) { \
> glue(glue(st, word), _be_p)(args); \
> } else { \
> glue(glue(st, word), _le_p)(args); \
> diff --git a/system/memory-internal.h b/system/memory-internal.h
> index 085e81a9fe4..29717b3c58f 100644
> --- a/system/memory-internal.h
> +++ b/system/memory-internal.h
> @@ -45,7 +45,7 @@ static inline bool devend_big_endian(enum device_endian end)
> DEVICE_HOST_ENDIAN != DEVICE_BIG_ENDIAN);
>
> if (end == DEVICE_NATIVE_ENDIAN) {
> - return target_words_bigendian();
> + return target_big_endian();
> }
> return end == DEVICE_BIG_ENDIAN;
> }
> diff --git a/cpu-target.c b/cpu-target.c
> index e018acbf71a..b5645ff0dbb 100644
> --- a/cpu-target.c
> +++ b/cpu-target.c
> @@ -86,8 +86,8 @@ void cpu_abort(CPUState *cpu, const char *fmt, ...)
> abort();
> }
>
> -#undef target_words_bigendian
> -bool target_words_bigendian(void)
> +#undef target_big_endian
> +bool target_big_endian(void)
> {
> return TARGET_BIG_ENDIAN;
> }
> diff --git a/hw/core/cpu-system.c b/hw/core/cpu-system.c
> index 82b68b8927d..3c84176a0c5 100644
> --- a/hw/core/cpu-system.c
> +++ b/hw/core/cpu-system.c
> @@ -133,7 +133,7 @@ bool cpu_virtio_is_big_endian(CPUState *cpu)
> if (cpu->cc->sysemu_ops->virtio_is_big_endian) {
> return cpu->cc->sysemu_ops->virtio_is_big_endian(cpu);
> }
> - return target_words_bigendian();
> + return target_big_endian();
> }
>
> GuestPanicInformation *cpu_get_crash_info(CPUState *cpu)
> diff --git a/hw/display/vga.c b/hw/display/vga.c
> index b01f67c65fb..20475ebbd31 100644
> --- a/hw/display/vga.c
> +++ b/hw/display/vga.c
> @@ -2264,7 +2264,7 @@ bool vga_common_init(VGACommonState *s, Object *obj, Error **errp)
> * into a device attribute set by the machine/platform to remove
> * all target endian dependencies from this file.
> */
> - s->default_endian_fb = target_words_bigendian();
> + s->default_endian_fb = target_big_endian();
> s->big_endian_fb = s->default_endian_fb;
>
> vga_dirty_log_start(s);
> diff --git a/hw/virtio/virtio.c b/hw/virtio/virtio.c
> index 85110bce374..8fbf1716b88 100644
> --- a/hw/virtio/virtio.c
> +++ b/hw/virtio/virtio.c
> @@ -2248,7 +2248,7 @@ int virtio_set_status(VirtIODevice *vdev, uint8_t val)
>
> static enum virtio_device_endian virtio_default_endian(void)
> {
> - if (target_words_bigendian()) {
> + if (target_big_endian()) {
> return VIRTIO_DEVICE_ENDIAN_BIG;
> } else {
> return VIRTIO_DEVICE_ENDIAN_LITTLE;
> diff --git a/system/memory.c b/system/memory.c
> index 7e2f16f4e95..67e433095b4 100644
> --- a/system/memory.c
> +++ b/system/memory.c
> @@ -2575,7 +2575,7 @@ void memory_region_add_eventfd(MemoryRegion *mr,
> unsigned i;
>
> if (size) {
> - MemOp mop = (target_words_bigendian() ? MO_BE : MO_LE) | size_memop(size);
> + MemOp mop = (target_big_endian() ? MO_BE : MO_LE) | size_memop(size);
> adjust_endianness(mr, &mrfd.data, mop);
> }
> memory_region_transaction_begin();
> @@ -2611,7 +2611,7 @@ void memory_region_del_eventfd(MemoryRegion *mr,
> unsigned i;
>
> if (size) {
> - MemOp mop = (target_words_bigendian() ? MO_BE : MO_LE) | size_memop(size);
> + MemOp mop = (target_big_endian() ? MO_BE : MO_LE) | size_memop(size);
> adjust_endianness(mr, &mrfd.data, mop);
> }
> memory_region_transaction_begin();
> diff --git a/system/qtest.c b/system/qtest.c
> index 523a0479959..c675fa2cb30 100644
> --- a/system/qtest.c
> +++ b/system/qtest.c
> @@ -693,7 +693,7 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
>
> qtest_send(chr, "OK\n");
> } else if (strcmp(words[0], "endianness") == 0) {
> - if (target_words_bigendian()) {
> + if (target_big_endian()) {
> qtest_sendf(chr, "OK big\n");
> } else {
> qtest_sendf(chr, "OK little\n");
> --
> 2.47.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] exec: Rename target_words_bigendian() -> target_big_endian()
2025-04-17 21:00 [PATCH] exec: Rename target_words_bigendian() -> target_big_endian() Philippe Mathieu-Daudé
2025-04-17 21:01 ` Michael S. Tsirkin
@ 2025-04-17 21:37 ` Pierrick Bouvier
2025-04-25 10:44 ` Philippe Mathieu-Daudé
2 siblings, 0 replies; 4+ messages in thread
From: Pierrick Bouvier @ 2025-04-17 21:37 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel
Cc: David Hildenbrand, Zhao Liu, Paolo Bonzini, Peter Xu,
Gerd Hoffmann, Eduardo Habkost, Marcel Apfelbaum,
Michael S. Tsirkin, Yanan Wang, Richard Henderson, Laurent Vivier,
Fabiano Rosas
On 4/17/25 14:00, Philippe Mathieu-Daudé wrote:
> In commit 98ed8ecfc9d ("exec: introduce target_words_bigendian()
> helper") target_words_bigendian() was matching the definition it
> was depending on (TARGET_WORDS_BIGENDIAN). Later in commit
> ee3eb3a7ce7 ("Replace TARGET_WORDS_BIGENDIAN") the definition was
> renamed as TARGET_BIG_ENDIAN but we didn't update the helper.
> Do it now mechanically using:
>
> $ sed -i -e s/target_words_bigendian/target_big_endian/g \
> $(git grep -wl target_words_bigendian)
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> include/exec/tswap.h | 12 ++++++------
> system/memory-internal.h | 2 +-
> cpu-target.c | 4 ++--
> hw/core/cpu-system.c | 2 +-
> hw/display/vga.c | 2 +-
> hw/virtio/virtio.c | 2 +-
> system/memory.c | 4 ++--
> system/qtest.c | 2 +-
> 8 files changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/include/exec/tswap.h b/include/exec/tswap.h
> index 84060a49994..49511f26117 100644
> --- a/include/exec/tswap.h
> +++ b/include/exec/tswap.h
> @@ -11,15 +11,15 @@
> #include "qemu/bswap.h"
>
> /**
> - * target_words_bigendian:
> + * target_big_endian:
> * Returns true if the (default) endianness of the target is big endian,
> * false otherwise. Common code should normally never need to know about the
> * endianness of the target, so please do *not* use this function unless you
> * know very well what you are doing!
> */
> -bool target_words_bigendian(void);
> +bool target_big_endian(void);
> #ifdef COMPILING_PER_TARGET
> -#define target_words_bigendian() TARGET_BIG_ENDIAN
> +#define target_big_endian() TARGET_BIG_ENDIAN
> #endif
>
> /*
> @@ -29,7 +29,7 @@ bool target_words_bigendian(void);
> #ifdef COMPILING_PER_TARGET
> #define target_needs_bswap() (HOST_BIG_ENDIAN != TARGET_BIG_ENDIAN)
> #else
> -#define target_needs_bswap() (HOST_BIG_ENDIAN != target_words_bigendian())
> +#define target_needs_bswap() (HOST_BIG_ENDIAN != target_big_endian())
> #endif /* COMPILING_PER_TARGET */
>
> static inline uint16_t tswap16(uint16_t s)
> @@ -83,7 +83,7 @@ static inline void tswap64s(uint64_t *s)
> /* Return ld{word}_{le,be}_p following target endianness. */
> #define LOAD_IMPL(word, args...) \
> do { \
> - if (target_words_bigendian()) { \
> + if (target_big_endian()) { \
> return glue(glue(ld, word), _be_p)(args); \
> } else { \
> return glue(glue(ld, word), _le_p)(args); \
> @@ -120,7 +120,7 @@ static inline uint64_t ldn_p(const void *ptr, int sz)
> /* Call st{word}_{le,be}_p following target endianness. */
> #define STORE_IMPL(word, args...) \
> do { \
> - if (target_words_bigendian()) { \
> + if (target_big_endian()) { \
> glue(glue(st, word), _be_p)(args); \
> } else { \
> glue(glue(st, word), _le_p)(args); \
> diff --git a/system/memory-internal.h b/system/memory-internal.h
> index 085e81a9fe4..29717b3c58f 100644
> --- a/system/memory-internal.h
> +++ b/system/memory-internal.h
> @@ -45,7 +45,7 @@ static inline bool devend_big_endian(enum device_endian end)
> DEVICE_HOST_ENDIAN != DEVICE_BIG_ENDIAN);
>
> if (end == DEVICE_NATIVE_ENDIAN) {
> - return target_words_bigendian();
> + return target_big_endian();
> }
> return end == DEVICE_BIG_ENDIAN;
> }
> diff --git a/cpu-target.c b/cpu-target.c
> index e018acbf71a..b5645ff0dbb 100644
> --- a/cpu-target.c
> +++ b/cpu-target.c
> @@ -86,8 +86,8 @@ void cpu_abort(CPUState *cpu, const char *fmt, ...)
> abort();
> }
>
> -#undef target_words_bigendian
> -bool target_words_bigendian(void)
> +#undef target_big_endian
> +bool target_big_endian(void)
> {
> return TARGET_BIG_ENDIAN;
> }
> diff --git a/hw/core/cpu-system.c b/hw/core/cpu-system.c
> index 82b68b8927d..3c84176a0c5 100644
> --- a/hw/core/cpu-system.c
> +++ b/hw/core/cpu-system.c
> @@ -133,7 +133,7 @@ bool cpu_virtio_is_big_endian(CPUState *cpu)
> if (cpu->cc->sysemu_ops->virtio_is_big_endian) {
> return cpu->cc->sysemu_ops->virtio_is_big_endian(cpu);
> }
> - return target_words_bigendian();
> + return target_big_endian();
> }
>
> GuestPanicInformation *cpu_get_crash_info(CPUState *cpu)
> diff --git a/hw/display/vga.c b/hw/display/vga.c
> index b01f67c65fb..20475ebbd31 100644
> --- a/hw/display/vga.c
> +++ b/hw/display/vga.c
> @@ -2264,7 +2264,7 @@ bool vga_common_init(VGACommonState *s, Object *obj, Error **errp)
> * into a device attribute set by the machine/platform to remove
> * all target endian dependencies from this file.
> */
> - s->default_endian_fb = target_words_bigendian();
> + s->default_endian_fb = target_big_endian();
> s->big_endian_fb = s->default_endian_fb;
>
> vga_dirty_log_start(s);
> diff --git a/hw/virtio/virtio.c b/hw/virtio/virtio.c
> index 85110bce374..8fbf1716b88 100644
> --- a/hw/virtio/virtio.c
> +++ b/hw/virtio/virtio.c
> @@ -2248,7 +2248,7 @@ int virtio_set_status(VirtIODevice *vdev, uint8_t val)
>
> static enum virtio_device_endian virtio_default_endian(void)
> {
> - if (target_words_bigendian()) {
> + if (target_big_endian()) {
> return VIRTIO_DEVICE_ENDIAN_BIG;
> } else {
> return VIRTIO_DEVICE_ENDIAN_LITTLE;
> diff --git a/system/memory.c b/system/memory.c
> index 7e2f16f4e95..67e433095b4 100644
> --- a/system/memory.c
> +++ b/system/memory.c
> @@ -2575,7 +2575,7 @@ void memory_region_add_eventfd(MemoryRegion *mr,
> unsigned i;
>
> if (size) {
> - MemOp mop = (target_words_bigendian() ? MO_BE : MO_LE) | size_memop(size);
> + MemOp mop = (target_big_endian() ? MO_BE : MO_LE) | size_memop(size);
> adjust_endianness(mr, &mrfd.data, mop);
> }
> memory_region_transaction_begin();
> @@ -2611,7 +2611,7 @@ void memory_region_del_eventfd(MemoryRegion *mr,
> unsigned i;
>
> if (size) {
> - MemOp mop = (target_words_bigendian() ? MO_BE : MO_LE) | size_memop(size);
> + MemOp mop = (target_big_endian() ? MO_BE : MO_LE) | size_memop(size);
> adjust_endianness(mr, &mrfd.data, mop);
> }
> memory_region_transaction_begin();
> diff --git a/system/qtest.c b/system/qtest.c
> index 523a0479959..c675fa2cb30 100644
> --- a/system/qtest.c
> +++ b/system/qtest.c
> @@ -693,7 +693,7 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
>
> qtest_send(chr, "OK\n");
> } else if (strcmp(words[0], "endianness") == 0) {
> - if (target_words_bigendian()) {
> + if (target_big_endian()) {
> qtest_sendf(chr, "OK big\n");
> } else {
> qtest_sendf(chr, "OK little\n");
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] exec: Rename target_words_bigendian() -> target_big_endian()
2025-04-17 21:00 [PATCH] exec: Rename target_words_bigendian() -> target_big_endian() Philippe Mathieu-Daudé
2025-04-17 21:01 ` Michael S. Tsirkin
2025-04-17 21:37 ` Pierrick Bouvier
@ 2025-04-25 10:44 ` Philippe Mathieu-Daudé
2 siblings, 0 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2025-04-25 10:44 UTC (permalink / raw)
To: qemu-devel
Cc: David Hildenbrand, Zhao Liu, Paolo Bonzini, Peter Xu,
Gerd Hoffmann, Eduardo Habkost, Marcel Apfelbaum,
Michael S. Tsirkin, Yanan Wang, Richard Henderson, Laurent Vivier,
Fabiano Rosas
On 17/4/25 23:00, Philippe Mathieu-Daudé wrote:
> In commit 98ed8ecfc9d ("exec: introduce target_words_bigendian()
> helper") target_words_bigendian() was matching the definition it
> was depending on (TARGET_WORDS_BIGENDIAN). Later in commit
> ee3eb3a7ce7 ("Replace TARGET_WORDS_BIGENDIAN") the definition was
> renamed as TARGET_BIG_ENDIAN but we didn't update the helper.
> Do it now mechanically using:
>
> $ sed -i -e s/target_words_bigendian/target_big_endian/g \
> $(git grep -wl target_words_bigendian)
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> include/exec/tswap.h | 12 ++++++------
> system/memory-internal.h | 2 +-
> cpu-target.c | 4 ++--
> hw/core/cpu-system.c | 2 +-
> hw/display/vga.c | 2 +-
> hw/virtio/virtio.c | 2 +-
> system/memory.c | 4 ++--
> system/qtest.c | 2 +-
> 8 files changed, 15 insertions(+), 15 deletions(-)
Patch queued, thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-04-25 10:44 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-04-17 21:00 [PATCH] exec: Rename target_words_bigendian() -> target_big_endian() Philippe Mathieu-Daudé
2025-04-17 21:01 ` Michael S. Tsirkin
2025-04-17 21:37 ` Pierrick Bouvier
2025-04-25 10:44 ` Philippe Mathieu-Daudé
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).