[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tests: switch tests to accel=kvm:tcg
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH] tests: switch tests to accel=kvm:tcg |
Date: |
Tue, 15 Aug 2017 22:28:11 +0300 |
On Tue, Aug 15, 2017 at 08:47:35AM -0700, Richard Henderson wrote:
> On 08/14/2017 08:33 AM, Cornelia Huck wrote:
> > On Mon, 14 Aug 2017 17:34:15 +0300
> > "Michael S. Tsirkin" <address@hidden> wrote:
> >
> >> Speed up tests on host systems with kvm support.
> >> In particular, this fixes tests with --disable-tcg.
> >>
> >> Cc: Paolo Bonzini <address@hidden>
> >> Cc: Thomas Huth <address@hidden>
> >> Cc: Laurent Vivier <address@hidden>
> >> Suggested-by: Cornelia Huck <address@hidden>
> >> Signed-off-by: Michael S. Tsirkin <address@hidden>
> >> ---
> >>
> >> Tested on x86 only.
> >>
> >> tests/boot-serial-test.c | 2 +-
> >> tests/pnv-xscom-test.c | 4 ++--
> >> tests/prom-env-test.c | 2 +-
> >> 3 files changed, 4 insertions(+), 4 deletions(-)
> >>
> >> diff --git a/tests/boot-serial-test.c b/tests/boot-serial-test.c
> >> index 11f48b0..c3b2e4e 100644
> >> --- a/tests/boot-serial-test.c
> >> +++ b/tests/boot-serial-test.c
> >> @@ -78,7 +78,7 @@ static void test_machine(const void *data)
> >> fd = mkstemp(tmpname);
> >> g_assert(fd != -1);
> >>
> >> - args = g_strdup_printf("-M %s,accel=tcg -chardev
> >> file,id=serial0,path=%s"
> >> + args = g_strdup_printf("-M %s,accel=kvm:tcg -chardev
> >> file,id=serial0,path=%s"
> >> " -no-shutdown -serial chardev:serial0 %s",
> >> test->machine, tmpname, test->extra);
> >
> > This has already been changed upstream.
>
> Ouch. This is the only real smoke test we have for the tcg backend for the
> host. While it is still going to test tcg for whatever machines do not run
> natively on the host, I can't help think we've lost testing.
>
> Can we use accel=tcg:kvm instead?
>
>
> r~
We can add an environment to disable kvm on qemu side,
for e.g. CI systems who have spare time.
--
MST