[gnome-boxes] vm-configurator: Use SATA when q35 machine type is supported



commit 2880359bbeaee727b14eb0abf46e84cb7104607f
Author: Fabiano FidĂȘncio <fidencio redhat com>
Date:   Fri Oct 12 08:20:07 2018 +0200

    vm-configurator: Use SATA when q35 machine type is supported
    
    IDE is not a valid disk bus when using q35 machine type. For those
    cases, we'd be better using SATA as it's the most widely supported.
    
    Related: https://bugzilla.redhat.com/show_bug.cgi?id=1581422
    
    Signed-off-by: Fabiano FidĂȘncio <fidencio redhat com>

 src/vm-configurator.vala | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
---
diff --git a/src/vm-configurator.vala b/src/vm-configurator.vala
index 148bbd8b..b97ce820 100644
--- a/src/vm-configurator.vala
+++ b/src/vm-configurator.vala
@@ -320,8 +320,13 @@ public static void set_target_media_config (Domain         domain,
             disk.set_target_bus (DomainDiskBus.VIRTIO);
             disk.set_target_dev ("vd" + dev_letter_str);
         } else {
-            debug ("Using IDE controller for the main disk");
-            disk.set_target_bus (DomainDiskBus.IDE);
+            if (install_media.prefers_q35) {
+                debug ("Using SATA controller for the main disk");
+                disk.set_target_bus (DomainDiskBus.SATA);
+            } else {
+                debug ("Using IDE controller for the main disk");
+                disk.set_target_bus (DomainDiskBus.IDE);
+            }
             disk.set_target_dev ("hd" + dev_letter_str);
         }
 


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