Re: [PATCH] virtinst: Use virtio bus type for disks on pSeries machines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/09/2016 07:10 PM, Cole Robinson wrote:
On 04/09/2016 03:46 AM, Shivaprasad G Bhat wrote:
pSeries doesn't support ide and so the default bus type was changed to use the
scsi before. It is now verified that virtio works better on pSeries with better
performance outcomes. So, change the default to virtio for disks.

Signed-off-by: Shivaprasad G Bhat <sbhat@xxxxxxxxxxxxxxxxxx>
---
  virtinst/guest.py |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/virtinst/guest.py b/virtinst/guest.py
index da07410..ab6dcba 100644
--- a/virtinst/guest.py
+++ b/virtinst/guest.py
@@ -1036,7 +1036,7 @@ class Guest(XMLBuilder):
                      self._os_object.supports_virtiodisk()))):
                  d.bus = "virtio"
              elif self.os.is_pseries():
-                d.bus = "scsi"
+                d.bus = "virtio"
              elif self.os.is_arm():
                  d.bus = "sd"
              elif self.os.is_q35():

This will break the test suite, you'll need to run 'python setup.py tests
--regenerate-output' to update things

Is this fine to do unconditionally? Or is working virtio support dependent on
new enough qemu or libvirt or guest OS?
Hi Cole,

I could verfiy that, virtio support has been there from the early versions of qemu(1.6.2) supporting Power KVM.

Also, all of the supported distros support the virtio from the initial releases itself.

I'll post the V2 as suggested.

Thanks,
Shiva




- Cole


_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list



[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux