[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 5/5] configure: Add -Wno-psabi
From: |
Richard Henderson |
Subject: |
Re: [PATCH v2 5/5] configure: Add -Wno-psabi |
Date: |
Thu, 11 Jun 2020 09:57:55 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.8.0 |
On 6/11/20 9:44 AM, Alex Bennée wrote:
>
> Richard Henderson <richard.henderson@linaro.org> writes:
>
>> 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 reptitions. 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.
>>
>> Cc: Alex Bennée <alex.bennee@linaro.org>
>> Cc: Peter Maydell <peter.maydell@linaro.org>
>> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
>> ---
>> TODO: The only portion of QEMU which does have a public api,
>> and so must have a stable abi, is "qemu/plugin.h". We could
>> test this by forcing -Wpsabi or -Werror=psabi in tests/plugin.
>> I can't seem to make that work -- Alex?
>
> modified plugins/Makefile.objs
> @@ -5,6 +5,7 @@
> obj-y += loader.o
> obj-y += core.o
> obj-y += api.o
> +api.o-cflags := -Wpsabi
>
> # Abuse -libs suffix to only link with --dynamic-list/-exported_symbols_list
> # when the final binary includes the plugin object.
>
> Seems to work for me.
Wrong directory -- that's the part that goes into qemu, which also uses other
qemu internal headers. As opposed to the tests/, which only use the one
"qemu/plugins.h" header (plus libc).
r~
[PATCH v2 4/5] configure: Disable -Wtautological-type-limit-compare, Richard Henderson, 2020/06/10