|
 |
101dff |
From 780525f32851c0daaf4250dc734ba73d1571c1c1 Mon Sep 17 00:00:00 2001
|
|
|
101dff |
From: Felipe Borges <felipeborges@gnome.org>
|
|
|
101dff |
Date: Mon, 2 Nov 2020 15:35:36 +0100
|
|
|
101dff |
Subject: [PATCH] vm-configurator: Pass discard "unmap" to virtio-blk disks
|
|
|
101dff |
|
|
|
101dff |
Ever since QEMU 4.0, virtio-blk disks have discard support[0]. For
|
|
|
101dff |
this reason, we don't need to migrate to virtio-scsi[1] to benefit from
|
|
|
101dff |
discard support.
|
|
|
101dff |
|
|
|
101dff |
See also https://chrisirwin.ca/posts/discard-with-kvm-2020/
|
|
|
101dff |
|
|
|
101dff |
[0] https://bugzilla.redhat.com/1672680
|
|
|
101dff |
[1] https://bugzilla.redhat.com/1152037
|
|
|
101dff |
|
|
|
101dff |
Fixes #35
|
|
|
101dff |
---
|
|
|
101dff |
src/vm-configurator.vala | 11 ++++++++---
|
|
|
101dff |
1 file changed, 8 insertions(+), 3 deletions(-)
|
|
|
101dff |
|
|
|
101dff |
diff --git a/src/vm-configurator.vala b/src/vm-configurator.vala
|
|
|
101dff |
index ee8cf133..f2a50bb8 100644
|
|
|
101dff |
--- a/src/vm-configurator.vala
|
|
|
101dff |
+++ b/src/vm-configurator.vala
|
|
|
101dff |
@@ -343,16 +343,21 @@ public static void set_target_media_config (Domain domain,
|
|
|
101dff |
var disk = new DomainDisk ();
|
|
|
101dff |
disk.set_type (DomainDiskType.FILE);
|
|
|
101dff |
disk.set_guest_device_type (DomainDiskGuestDeviceType.DISK);
|
|
|
101dff |
- disk.set_driver_name ("qemu");
|
|
|
101dff |
- disk.set_driver_format (DomainDiskFormat.QCOW2);
|
|
|
101dff |
disk.set_source (target_path);
|
|
|
101dff |
- disk.set_driver_cache (DomainDiskCacheType.WRITEBACK);
|
|
|
101dff |
+
|
|
|
101dff |
+ var driver = new DomainDiskDriver ();
|
|
|
101dff |
+ driver.set_name ("qemu");
|
|
|
101dff |
+ driver.set_format (DomainDiskFormat.QCOW2);
|
|
|
101dff |
+ driver.set_cache (DomainDiskCacheType.WRITEBACK);
|
|
|
101dff |
+ disk.set_driver (driver);
|
|
|
101dff |
|
|
|
101dff |
var dev_letter_str = ((char) (dev_index + 97)).to_string ();
|
|
|
101dff |
if (install_media.supports_virtio_disk || install_media.supports_virtio1_disk) {
|
|
|
101dff |
debug ("Using virtio controller for the main disk");
|
|
|
101dff |
disk.set_target_bus (DomainDiskBus.VIRTIO);
|
|
|
101dff |
disk.set_target_dev ("vd" + dev_letter_str);
|
|
|
101dff |
+
|
|
|
101dff |
+ driver.set_discard (DomainDiskDriverDiscard.UNMAP);
|
|
|
101dff |
} else {
|
|
|
101dff |
if (install_media.prefers_q35) {
|
|
|
101dff |
debug ("Using SATA controller for the main disk");
|
|
|
101dff |
--
|
|
|
101dff |
2.26.2
|
|
|
101dff |
|