[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: |
Alex Bennée |
Subject: |
Re: [PATCH v4 12/12] tests/meson: Only build softfloat objects if TCG is selected |
Date: |
Wed, 28 Apr 2021 20:04:54 +0100 |
User-agent: |
mu4e 1.5.12; emacs 28.0.50 |
Philippe Mathieu-Daudé <philmd@redhat.com> writes:
> On 4/28/21 7:06 PM, Alex Bennée wrote:
>> Philippe Mathieu-Daudé <philmd@redhat.com> writes:
>>
>>> Alex, Richard, do you mind reviewing this one please?
>>
>> Isn't it already merged (with my r-b tag no less ;-)
>>
>> f77147cd4de8c726f89b2702f7a9d0c9711d8875
>
> See ...
>
>> Author: Philippe Mathieu-Daudé <philmd@redhat.com>
>> AuthorDate: Fri Jan 22 21:44:31 2021 +0100
>> Commit: Paolo Bonzini <pbonzini@redhat.com>
>> CommitDate: Mon Feb 8 14:43:55 2021 +0100
>>
>>>
>>> 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
>
> ... ^ this comment :(
Ahh - my tooling was confused having searched by the subject title ;-)
>
>>>> 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()
>>>>
>>
>>
--
Alex Bennée
- Re: [PATCH v4 10/12] qtest/qmp-cmd-test: Make test build-independent from accelerator, (continued)
- 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 11/12] tests/qtest: Do not restrict bios-tables-test to Aarch64 hosts anymore, Philippe Mathieu-Daudé, 2021/04/15
[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, Alex Bennée, 2021/04/30