[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 2/3] iotests: use -ccw on s390x for 051
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH 2/3] iotests: use -ccw on s390x for 051 |
Date: |
Fri, 8 Sep 2017 13:04:25 +0200 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
Am 05.09.2017 um 17:16 hat Cornelia Huck geschrieben:
> 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, so use the -ccw instead of the -pci versions of virtio
> devices on s390x.
>
> Provide an output file for s390x.
>
> Signed-off-by: Cornelia Huck <address@hidden>
> ---
> tests/qemu-iotests/051 | 9 +-
> tests/qemu-iotests/051.s390-ccw-virtio.out | 434
> +++++++++++++++++++++++++++++
> 2 files changed, 442 insertions(+), 1 deletion(-)
> create mode 100644 tests/qemu-iotests/051.s390-ccw-virtio.out
It's already a pain to have two separate output files for 051, let's try
to avoid adding a third one. Even more so since I think that the split
between 051.out and 051.pc.out was already made for s390, so I'm not
sure if anyone would actually still make use of the plain 051.out
output if s390 got it's own one.
> diff --git a/tests/qemu-iotests/051 b/tests/qemu-iotests/051
> index c8cfc764bc..f6ad0f4f0b 100755
> --- a/tests/qemu-iotests/051
> +++ b/tests/qemu-iotests/051
> @@ -103,7 +103,14 @@ echo
> echo === Device without drive ===
> echo
>
> -run_qemu -device virtio-scsi-pci -device scsi-hd
> +case "$QEMU_DEFAULT_MACHINE" in
> + s390-ccw-virtio)
> + run_qemu -device virtio-scsi-ccw -device scsi-hd
> + ;;
> + *)
> + run_qemu -device virtio-scsi-pci -device scsi-hd
> + ;;
> +esac
The only real difference between 051.out and 051.s390-ccw-virtio.out is
in this one command line. So if we don't want to just skip this part of
the test for non-pc like we already skip ther parts, we generally solve
this kind of thing by just filtering out strings that differ between
setups.
For example:
case "$QEMU_DEFAULT_MACHINE" in
s390-ccw-virtio)
virtio_scsi=virtio-scsi-ccw
;;
*)
virtio_scsi=virtio-scsi-pci
;;
esac
run_qemu -device $virtio_scsi -device scsi-hd |
sed -e "s/$virtio_scsi/VIRTIO-SCSI/"
Kevin
- [Qemu-block] [PATCH 0/3] iotests: cure s390x failures by switching to ccw, Cornelia Huck, 2017/09/05
- [Qemu-block] [PATCH 3/3] iotests: use -ccw on s390x for 067, Cornelia Huck, 2017/09/05
- [Qemu-block] [PATCH 2/3] iotests: use -ccw on s390x for 051, Cornelia Huck, 2017/09/05
- Re: [Qemu-block] [PATCH 2/3] iotests: use -ccw on s390x for 051,
Kevin Wolf <=
- Re: [Qemu-block] [PATCH 2/3] iotests: use -ccw on s390x for 051, Cornelia Huck, 2017/09/08
- Re: [Qemu-block] [PATCH 2/3] iotests: use -ccw on s390x for 051, Kevin Wolf, 2017/09/08
- Re: [Qemu-block] [Qemu-devel] [PATCH 2/3] iotests: use -ccw on s390x for 051, Thomas Huth, 2017/09/08
- Re: [Qemu-block] [Qemu-devel] [PATCH 2/3] iotests: use -ccw on s390x for 051, Lukáš Doktor, 2017/09/11
- Re: [Qemu-block] [Qemu-devel] [PATCH 2/3] iotests: use -ccw on s390x for 051, Cornelia Huck, 2017/09/12
- Re: [Qemu-block] [Qemu-devel] [PATCH 2/3] iotests: use -ccw on s390x for 051, Kevin Wolf, 2017/09/12
[Qemu-block] [PATCH 1/3] iotests: use -ccw on s390x for 040, 139, and 182, Cornelia Huck, 2017/09/05
Re: [Qemu-block] [Qemu-devel] [PATCH 0/3] iotests: cure s390x failures by switching to ccw, QingFeng Hao, 2017/09/06