Skip to content

Commit bac8d22

Browse files
rth7680pm215
authored andcommitted
configure: Add -Wno-psabi
On aarch64, gcc 9.3 is generating qemu/exec.c: In function ‘address_space_translate_iommu’: qemu/exec.c:431:28: note: parameter passing for argument of type \ ‘MemTxAttrs’ {aka ‘struct MemTxAttrs’} changed in GCC 9.1 and many other repetitions. This structure, and the functions amongst which it is passed, are not part of a QEMU public API. Therefore we do not care how the compiler passes the argument, so long as the compiler is self-consistent. The only portion of QEMU which does have a public api, and so must have a stable abi, is "qemu/plugin.h". We test this by forcing -Wpsabi in tests/plugin/Makefile. Buglink: https://bugs.launchpad.net/qemu/+bug/1881552 Reviewed-by: Alex Bennée <[email protected]> Reviewed-by: Philippe Mathieu-Daudé <[email protected]> Signed-off-by: Richard Henderson <[email protected]> Message-id: [email protected] Signed-off-by: Peter Maydell <[email protected]>
1 parent aabab96 commit bac8d22

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

configure

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2055,6 +2055,7 @@ add_to nowarn_flags -Wno-shift-negative-value
20552055
add_to nowarn_flags -Wno-string-plus-int
20562056
add_to nowarn_flags -Wno-typedef-redefinition
20572057
add_to nowarn_flags -Wno-tautological-type-limit-compare
2058+
add_to nowarn_flags -Wno-psabi
20582059

20592060
gcc_flags="$warn_flags $nowarn_flags"
20602061

tests/plugin/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ NAMES += lockstep
1717

1818
SONAMES := $(addsuffix .so,$(addprefix lib,$(NAMES)))
1919

20-
QEMU_CFLAGS += -fPIC
20+
QEMU_CFLAGS += -fPIC -Wpsabi
2121
QEMU_CFLAGS += -I$(SRC_PATH)/include/qemu
2222

2323
all: $(SONAMES)

0 commit comments

Comments
 (0)