[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