[virt-tools-list] [PATCH v1] virtinst: Use virtio bus type for cdrom disk on s390x
Leno Hou
lenohou at gmail.com
Thu May 5 17:59:16 UTC 2016
s390x doesn't support scsi and so was changed to use the scsi. virtio works
better on s390x with better performance outcomes. Change the default to
virtio for cdrom disks
Signed-off-by: Leno Hou <lenohou at gmail.com>
---
tests/cli-test-xml/compare/virt-install-s390x-cdrom.xml | 2 +-
virtinst/devicedisk.py | 3 ---
2 files changed, 1 insertion(+), 4 deletions(-)
diff --git a/tests/cli-test-xml/compare/virt-install-s390x-cdrom.xml b/tests/cli-test-xml/compare/virt-install-s390x-cdrom.xml
index 0083dc0..a4d6fcc 100644
--- a/tests/cli-test-xml/compare/virt-install-s390x-cdrom.xml
+++ b/tests/cli-test-xml/compare/virt-install-s390x-cdrom.xml
@@ -23,7 +23,7 @@
<disk type="file" device="cdrom">
<driver name="qemu"/>
<source file="/tmp/__virtinst_cli_exist1.img"/>
- <target dev="sda" bus="scsi"/>
+ <target dev="vdb" bus="virtio"/>
<readonly/>
</disk>
<interface type="bridge">
diff --git a/virtinst/devicedisk.py b/virtinst/devicedisk.py
index c7cd04d..7367074 100644
--- a/virtinst/devicedisk.py
+++ b/virtinst/devicedisk.py
@@ -870,9 +870,6 @@ class VirtualDisk(VirtualDevice):
if self.is_cdrom():
self.read_only = True
- if self.is_cdrom() and guest.os.is_s390x():
- self.bus = "scsi"
-
if not self.conn.is_qemu():
return
if not self.is_disk():
--
1.9.1
More information about the virt-tools-list
mailing list