[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] qemu-iotests: 140: use virtio-scsi instead
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] qemu-iotests: 140: use virtio-scsi instead of IDE |
Date: |
Wed, 10 Feb 2016 17:15:45 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 |
On 09.02.2016 14:15, Sascha Silbe wrote:
> IDE is only implemented by very few architectures (mostly PC). Use
> virtio-scsi instead so the test works on all architectures that
> support virtio. In particular, this fixes qemu-iotests on s390x.
>
> Fixes: 16dee418 ("iotests: Add test for eject under NBD server")
> Signed-off-by: Sascha Silbe <address@hidden>
> ---
> tests/qemu-iotests/140 | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/qemu-iotests/140 b/tests/qemu-iotests/140
> index f78c317..0c448e6 100755
> --- a/tests/qemu-iotests/140
> +++ b/tests/qemu-iotests/140
> @@ -49,8 +49,8 @@ _make_test_img 64k
> $QEMU_IO -c 'write -P 42 0 64k' "$TEST_IMG" | _filter_qemu_io
>
> keep_stderr=y \
> -_launch_qemu -drive
> if=ide,media=cdrom,id=drv,file="$TEST_IMG",format=$IMGFMT \
> - 2> >(_filter_nbd)
> +_launch_qemu -drive
> if=none,media=cdrom,id=drv,file="$TEST_IMG",format=$IMGFMT \
> + -device virtio-scsi -device scsi-cd,drive=drv 2> >(_filter_nbd)
Why not just omit the device (and the media=cdrom along with it, keeping
if=none)? This will change the reference output because there is no
longer any tray to be moved, but this will still test what it's supposed to.
(This may sound hypocritical coming from me, because I wrote this test
so I could have just done so in the first place; I guess I just didn't
realize that 'eject' works on device-less drives, too.)
Max
> _send_qemu_cmd $QEMU_HANDLE \
> "{ 'execute': 'qmp_capabilities' }" \
>
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 1/3] qdev-monitor: sort alias table by typename, (continued)
- [Qemu-devel] [PATCH 1/3] qdev-monitor: sort alias table by typename, Sascha Silbe, 2016/02/09
- [Qemu-devel] [PATCH 2/3] qdev-monitor: add missing aliases for virtio-{9p, balloon, rng, scsi}, Sascha Silbe, 2016/02/09
- Re: [Qemu-devel] [PATCH 2/3] qdev-monitor: add missing aliases for virtio-{9p, balloon, rng, scsi}, Markus Armbruster, 2016/02/11
- Re: [Qemu-devel] [PATCH 2/3] qdev-monitor: add missing aliases for virtio-{9p, balloon, rng, scsi}, Cornelia Huck, 2016/02/11
- Re: [Qemu-devel] [PATCH 2/3] qdev-monitor: add missing aliases for virtio-{9p, balloon, rng, scsi}, Sascha Silbe, 2016/02/11
- Re: [Qemu-devel] [PATCH 2/3] qdev-monitor: add missing aliases for virtio-{9p, balloon, rng, scsi}, Cornelia Huck, 2016/02/15
[Qemu-devel] [PATCH 3/3] qemu-iotests: 140: use virtio-scsi instead of IDE, Sascha Silbe, 2016/02/09
- Re: [Qemu-devel] [PATCH 3/3] qemu-iotests: 140: use virtio-scsi instead of IDE,
Max Reitz <=