[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2] iotests: drop unnecessary accel=kvm in 238
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-block] [PATCH v2] iotests: drop unnecessary accel=kvm in 238 |
Date: |
Mon, 25 Feb 2019 16:19:08 +0000 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Thu, Feb 21, 2019 at 11:24:12AM +0100, Thomas Huth wrote:
> On 21/02/2019 11.11, Stefan Hajnoczi wrote:
> > Test 238 does not require the kvm accelerator. TCG works fine.
> >
> > Use the default accelerator instead of requiring kvm.
> >
> > Suggested-by: Thomas Huth <address@hidden>
> > Signed-off-by: Stefan Hajnoczi <address@hidden>
> > ---
> > Vladimir, I have dropped the 235 change because I don't have time to
> > figure out the kvm/tcg vs qtest differences or add a skip function.
> > Sorry!
> >
> > tests/qemu-iotests/238 | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/tests/qemu-iotests/238 b/tests/qemu-iotests/238
> > index f81ee1112f..e490bb4298 100755
> > --- a/tests/qemu-iotests/238
> > +++ b/tests/qemu-iotests/238
> > @@ -33,7 +33,6 @@ else:
> > virtio_scsi_device = 'virtio-scsi-pci'
> >
> > vm = QEMUMachine(iotests.qemu_prog)
> > -vm.add_args('-machine', 'accel=kvm')
>
> Would it be feasible to change it into "accel=qtest" instead? ... then
> this test would also work when QEMU has been compiled with --disable-tcg
> and when KVM is not available...
Sure. I have tested that qtest still triggered the bug when the fix was
merged.
Will fix in v3.
Stefan
signature.asc
Description: PGP signature