[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected |
Date: |
Wed, 28 Apr 2021 18:38:04 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 |
Alex, Richard, do you mind reviewing this one please?
On 4/15/21 6:33 PM, Philippe Mathieu-Daudé wrote:
> From: Philippe Mathieu-Daudé <f4bug@amsat.org>
>
> The previous attempt (commit f77147cd4de) doesn't work as
> expected, as we still have CONFIG_TCG=1 when using:
>
> configure --disable-system --disable-user
>
> Now than we have removed the use of CONFIG_TCG from target-dependent
> files in tests/qtest/, we can remove the unconditional definition of
> CONFIG_TCG in config_host.
>
> This avoid to build a bunch of unrequired objects when building with
> --disable-tcg (in particular the softfloat tests):
>
> Before:
>
> $ make
> [1/812] Generating trace-qom.h with a custom command
> ...
>
> After:
>
> $ make
> [1/349] Generating trace-qom.h with a custom command
> ...
>
> A difference of 463 objects...
>
> Reported-by: Claudio Fontana <cfontana@suse.de>
> Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> v3: Include Paolo's feedback:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg793872.html
> therefore o not include Alex's R-b tag.
>
> Cc: Richard Henderson <richard.henderson@linaro.org>
> Cc: Alex Bennée <alex.bennee@linaro.org>
> Cc: Emilio G. Cota <cota@braap.org>
> ---
> meson.build | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/meson.build b/meson.build
> index c6f4b0cf5e8..623cbe50685 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -262,7 +262,6 @@
> language: ['c', 'cpp', 'objc'])
>
> accelerators += 'CONFIG_TCG'
> - config_host += { 'CONFIG_TCG': 'y' }
> endif
>
> if 'CONFIG_KVM' not in accelerators and get_option('kvm').enabled()
>
- [PATCH v4 11/12] tests/qtest: Do not restrict bios-tables-test to Aarch64 hosts anymore, (continued)
- [PATCH v4 11/12] tests/qtest: Do not restrict bios-tables-test to Aarch64 hosts anymore, Philippe Mathieu-Daudé, 2021/04/15
- [PATCH v4 10/12] qtest/qmp-cmd-test: Make test build-independent from accelerator, Philippe Mathieu-Daudé, 2021/04/15
- Re: [PATCH v4 10/12] qtest/qmp-cmd-test: Make test build-independent from accelerator, Philippe Mathieu-Daudé, 2021/04/28
- Re: [PATCH v4 10/12] qtest/qmp-cmd-test: Make test build-independent from accelerator, Markus Armbruster, 2021/04/29
- Re: [PATCH v4 10/12] qtest/qmp-cmd-test: Make test build-independent from accelerator, Philippe Mathieu-Daudé, 2021/04/29
- Re: [PATCH v4 10/12] qtest/qmp-cmd-test: Make test build-independent from accelerator, Markus Armbruster, 2021/04/29
- Re: [PATCH v4 10/12] qtest/qmp-cmd-test: Make test build-independent from accelerator, Philippe Mathieu-Daudé, 2021/04/29
- Re: [PATCH v4 10/12] qtest/qmp-cmd-test: Make test build-independent from accelerator, Markus Armbruster, 2021/04/30
- Re: [PATCH v4 10/12] qtest/qmp-cmd-test: Make test build-independent from accelerator, Philippe Mathieu-Daudé, 2021/04/30
[PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected, Philippe Mathieu-Daudé, 2021/04/15
- Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected,
Philippe Mathieu-Daudé <=
- Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected, Alex Bennée, 2021/04/28
- Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected, Philippe Mathieu-Daudé, 2021/04/28
- Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected, Alex Bennée, 2021/04/28
- Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected, Philippe Mathieu-Daudé, 2021/04/28
- Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected, Philippe Mathieu-Daudé, 2021/04/29
Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected, Alex Bennée, 2021/04/30