rhtyd commented on a change in pull request #5012:
URL: https://github.com/apache/cloudstack/pull/5012#discussion_r660350465
##########
File path:
plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
##########
@@ -2719,6 +2731,18 @@ private KVMPhysicalDisk
getPhysicalDiskPrimaryStore(PrimaryDataStoreTO primaryDa
return storagePool.getPhysicalDisk(data.getPath());
}
+ /**
+ * Set Disk IO Driver, if supported by the Libvirt/Qemu version.
+ * IO Driver works for:
+ * (i) Qemu >= 5.0;
+ * (ii) Libvirt >= 6.3.0
+ */
+ protected void setDiskIoDriver(DiskDef disk) {
+ if (getHypervisorLibvirtVersion() >=
HYPERVISOR_LIBVIRT_VERSION_SUPPORTS_IO_URING && getHypervisorQemuVersion() >=
HYPERVISOR_QEMU_VERSION_SUPPORTS_IO_URING) {
Review comment:
LGTM and tests have passed, but this enables io_uring as default for the
matching version of qemu and libvirt, could that cause an issue
@GabrielBrascher for different type of guest OS?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
