[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 3/3] iotests: use virtio aliases for 067
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH v2 3/3] iotests: use virtio aliases for 067 |
Date: |
Wed, 13 Sep 2017 11:17:22 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 13.09.2017 11:10, Cornelia Huck wrote:
> The default cpu model on s390x does not provide zPCI, which is
> not yet wired up on tcg. Moreover, virtio-ccw is the standard
> on s390x.
>
> Using virtio-scsi will implicitly pick the right device, so just
> switch to that for simplicity.
>
> Signed-off-by: Cornelia Huck <address@hidden>
> ---
> tests/qemu-iotests/067 | 3 ++-
> tests/qemu-iotests/067.out | 2 +-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/tests/qemu-iotests/067 b/tests/qemu-iotests/067
> index 5d4ca4bc61..cbb3da286a 100755
> --- a/tests/qemu-iotests/067
> +++ b/tests/qemu-iotests/067
> @@ -141,7 +141,7 @@ echo
> echo === Empty drive with -device and device_del ===
> echo
>
> -run_qemu -device virtio-scsi-pci -device scsi-cd,id=cd0 <<EOF
> +run_qemu -device virtio-scsi -device scsi-cd,id=cd0 <<EOF
> { "execute": "qmp_capabilities" }
> { "execute": "query-block" }
> { "execute": "device_del", "arguments": { "id": "cd0" } }
> @@ -150,6 +150,7 @@ run_qemu -device virtio-scsi-pci -device scsi-cd,id=cd0
> <<EOF
> { "execute": "quit" }
> EOF
>
> +
Superfluous white space change?
With that nit removed:
Reviewed-by: Thomas Huth <address@hidden>
- [Qemu-devel] [PATCH v2 0/3] iotests: cure s390x failures by switching to ccw/aliases, Cornelia Huck, 2017/09/13
- [Qemu-devel] [PATCH v2 1/3] iotests: use -ccw on s390x for 040, 139, and 182, Cornelia Huck, 2017/09/13
- [Qemu-devel] [PATCH v2 2/3] iotests: use -ccw on s390x for 051, Cornelia Huck, 2017/09/13
- [Qemu-devel] [PATCH v2 3/3] iotests: use virtio aliases for 067, Cornelia Huck, 2017/09/13
- Re: [Qemu-devel] [PATCH v2 0/3] iotests: cure s390x failures by switching to ccw/aliases, QingFeng Hao, 2017/09/15
- Re: [Qemu-devel] [PATCH v2 0/3] iotests: cure s390x failures by switching to ccw/aliases, Kevin Wolf, 2017/09/15