qemu-arm
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [RFC PATCH v2 13/19] tests: do not run test-hmp on all machines for


From: Thomas Huth
Subject: Re: [RFC PATCH v2 13/19] tests: do not run test-hmp on all machines for ARM KVM-only
Date: Wed, 11 Jan 2023 13:22:35 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.13.0

On 11/01/2023 13.08, Claudio Fontana wrote:
On 1/10/23 15:02, Peter Maydell wrote:
On Tue, 10 Jan 2023 at 13:36, Fabiano Rosas <farosas@suse.de> wrote:

Peter Maydell <peter.maydell@linaro.org> writes:

On Tue, 10 Jan 2023 at 13:00, Fabiano Rosas <farosas@suse.de> wrote:

Thomas Huth <thuth@redhat.com> writes:

On 09/01/2023 23.42, Fabiano Rosas wrote:
From: Claudio Fontana <cfontana@suse.de>

on ARM we currently list and build all machines, even when
building KVM-only, without TCG.

Until we fix this (and we only list and build machines that are
compatible with KVM), only test specifically using the "virt"
machine in this case.

Why don't you fix it immediately? ...

My idea was to have in this series the minimum to unbreak the
--disable-tcg build and later bring the rest of the changes
incrementally.

This test is basically checking "all the machines should
work". That's an important invariant to maintain, so
I don't think we should just skip it for Arm targets.

But what does "all machines" mean in a no-TCG build? The original intent
of the patch was that only 'virt' should be present and therefore the
only one tested.

It means "every machine the user can create". If the
machine can't run then either we shouldn't compile it
in, or else we should be compiling in enough extra stuff
to allow it to work.

-- PMM

Hi,

once upon a time there was a series by Philippe to accomplish that (KConfig) 
right?

Maybe that one:

 https://lore.kernel.org/qemu-devel/20190903114729.3400-1-philmd@redhat.com/

?

 Thomas





reply via email to

[Prev in Thread] Current Thread [Next in Thread]