[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] iotests: drop unnecessary accel=kvm in 238
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH v2] iotests: drop unnecessary accel=kvm in 238 |
Date: |
Thu, 21 Feb 2019 11:24:12 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.0 |
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...
Thomas