Message ID | 20210319202335.2397060-4-laurent@vivier.eu |
---|---|
State | New |
Headers | show |
Series | iotests: fix failures with non-PCI machines | expand |
Laurent Vivier <laurent@vivier.eu> writes: > Rather than checking if the machine is an s390x to use virtio-blk-ccw > instead of virtio-blk-pci, use the alias virtio-blk that is set to > the expected target. > > This also enables the use of virtio-blk-device for targets without > PCI or CCW. > > Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> > Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com> > Signed-off-by: Laurent Vivier <laurent@vivier.eu> > --- > blockdev.c | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > > diff --git a/blockdev.c b/blockdev.c > index 5cc7c7effe9f..64da5350e3ad 100644 > --- a/blockdev.c > +++ b/blockdev.c > @@ -969,11 +969,7 @@ DriveInfo *drive_new(QemuOpts *all_opts, BlockInterfaceType block_default_type, > QemuOpts *devopts; > devopts = qemu_opts_create(qemu_find_opts("device"), NULL, 0, > &error_abort); > - if (arch_type == QEMU_ARCH_S390X) { > - qemu_opt_set(devopts, "driver", "virtio-blk-ccw", &error_abort); > - } else { > - qemu_opt_set(devopts, "driver", "virtio-blk-pci", &error_abort); > - } > + qemu_opt_set(devopts, "driver", "virtio-blk", &error_abort); > qemu_opt_set(devopts, "drive", qdict_get_str(bs_opts, "id"), > &error_abort); > } Acked-by: Markus Armbruster <armbru@redhat.com>
diff --git a/blockdev.c b/blockdev.c index 5cc7c7effe9f..64da5350e3ad 100644 --- a/blockdev.c +++ b/blockdev.c @@ -969,11 +969,7 @@ DriveInfo *drive_new(QemuOpts *all_opts, BlockInterfaceType block_default_type, QemuOpts *devopts; devopts = qemu_opts_create(qemu_find_opts("device"), NULL, 0, &error_abort); - if (arch_type == QEMU_ARCH_S390X) { - qemu_opt_set(devopts, "driver", "virtio-blk-ccw", &error_abort); - } else { - qemu_opt_set(devopts, "driver", "virtio-blk-pci", &error_abort); - } + qemu_opt_set(devopts, "driver", "virtio-blk", &error_abort); qemu_opt_set(devopts, "drive", qdict_get_str(bs_opts, "id"), &error_abort); }