| Message ID | 20241004163042.85922-1-philmd@linaro.org |
|---|---|
| Headers | show |
| Series | misc: Use explicit endian LD/ST API | expand |
On 4/10/24 18:30, Philippe Mathieu-Daudé wrote: > For targets (or HW) which are only built for a particular > endianness, the generic LD/ST helpers are defined as the > target endianness variant. For example, on big-endian > targets, stl_p() is equivalent of stl_be_p(). > > This series replaces in bulk these LD/ST calls. This is the first part where we only convert the targets built for a single endianness. The rest (MIPS, ARM, PPC, MicroBlaze and Xtensa) will be handled in different series. I'm keeping hw/virtio/virtio-config-io.c last. Possibly current API will then be restricted to user-emu & system/ to avoid further uses. $ git grep -wlE '(ld|st)t?u?[wlq]_p' hw/mips/bootloader.c hw/mips/fuloong2e.c hw/mips/malta.c hw/ppc/spapr.c hw/ppc/spapr_vhyp_mmu.c target/arm/cpu.c target/arm/gdbstub.c target/arm/gdbstub64.c target/microblaze/gdbstub.c target/mips/gdbstub.c target/ppc/gdbstub.c target/ppc/mmu-hash64.h target/xtensa/gdbstub.c accel/tcg/user-exec.c hw/virtio/virtio-config-io.c include/exec/cpu-all.h include/gdbstub/helpers.h monitor/hmp-cmds-target.c system/ioport.c system/memory_ldst.c.inc > This is helpful for the single binary project where we > want to build a single binary for multiple targets of > different endianness. > > Philippe Mathieu-Daudé (25): > gdbstub/helpers: Have ldtul_p() definition use ldn_p() > target/hexagon: Replace ldtul_p() -> ldl_p() > target/alpha: Replace ldtul_p() -> ldq_p() > target/s390x: Replace ldtul_p() -> ldq_p() > gdbstub/helpers: Introduce ldtul_$endian_p() helpers > target/alpha: Use explicit little-endian LD/ST API > target/hexagon: Use explicit little-endian LD/ST API > hw/i386: Use explicit little-endian LD/ST API > target/i386: Use explicit little-endian LD/ST API > target/avr: Use explicit little-endian LD/ST API > linux-user/i386: Use explicit little-endian LD/ST API > target/loongarch: Use explicit little-endian LD/ST API > target/sh4: Use explicit little-endian LD/ST API > target/tricore: Use explicit little-endian LD/ST API > target/rx: Use explicit little-endian LD/ST API > target/riscv: Use explicit little-endian LD/ST API > hw/m68k: Use explicit big-endian LD/ST API > target/m68k: Use explicit big-endian LD/ST API > hw/sparc: Use explicit big-endian LD/ST API > target/sparc: Use explicit big-endian LD/ST API > target/hppa: Use explicit big-endian LD/ST API > hw/s390x: Use explicit big-endian LD/ST API > target/s390x: Use explicit big-endian LD/ST API > target/openrisc: Use explicit big-endian LD/ST API > hw/ppc/e500: Use explicit big-endian LD/ST API > > hw/m68k/bootinfo.h | 28 ++--- > include/gdbstub/helpers.h | 6 +- > hw/i386/multiboot.c | 36 +++--- > hw/i386/x86-common.c | 26 ++--- > hw/m68k/mcf5208.c | 2 +- > hw/m68k/next-cube.c | 2 +- > hw/m68k/q800.c | 4 +- > hw/ppc/ppce500_spin.c | 24 ++-- > hw/s390x/ipl.c | 4 +- > hw/s390x/s390-pci-inst.c | 166 +++++++++++++-------------- > hw/sparc/leon3.c | 42 +++---- > hw/sparc/sun4m.c | 6 +- > hw/sparc64/sun4u.c | 6 +- > linux-user/i386/signal.c | 4 +- > target/alpha/gdbstub.c | 2 +- > target/avr/gdbstub.c | 4 +- > target/hexagon/gdbstub.c | 10 +- > target/hppa/gdbstub.c | 2 +- > target/i386/gdbstub.c | 30 ++--- > target/i386/tcg/sysemu/excp_helper.c | 4 +- > target/i386/xsave_helper.c | 32 +++--- > target/loongarch/gdbstub.c | 8 +- > target/m68k/gdbstub.c | 2 +- > target/m68k/helper.c | 10 +- > target/openrisc/gdbstub.c | 2 +- > target/riscv/gdbstub.c | 14 +-- > target/rx/cpu.c | 2 +- > target/rx/gdbstub.c | 24 ++-- > target/s390x/gdbstub.c | 34 +++--- > target/s390x/ioinst.c | 2 +- > target/sh4/gdbstub.c | 36 +++--- > target/sparc/gdbstub.c | 6 +- > target/tricore/gdbstub.c | 2 +- > 33 files changed, 292 insertions(+), 290 deletions(-) >