F #6435: Automatic setting of NIC IO queues

This commit adds support for "auto" setting for virtio queues of NICs. It follows the approach implemented by VIRTIO_BLK_QUEUES for DISKS. The commit also includes default setting and RSunstone implementation Signed-off-by: Kristian Feldsam <feldsam@gmail.com>
OpenNebula · Jan 30, 2025 · 983066a · 983066a
1 parent 950f33c
commit 983066a
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 3 deletions.
diff --git a/...one/public/app/tabs/templates-tab/form-panels/create/wizard-tabs/network/nic-tab/html.hbs b/...one/public/app/tabs/templates-tab/form-panels/create/wizard-tabs/network/nic-tab/html.hbs
@@ -313,7 +313,10 @@
           {{tr "Transmission queue"}}
           {{{tip (tr "Only supported for virtio driver.")}}}
         </label>
-        <input type="number" min="0" wizard_field="VIRTIO_QUEUES" id="VIRTIO_QUEUES" name="VIRTIO_QUEUES" />
+        <select wizard_field="VIRTIO_QUEUES" id="VIRTIO_QUEUES" name="VIRTIO_QUEUES">
+          <option value=""> - </option>
+          <option value="auto">Auto</option>
+        </select>
       </div>
     </div>
     <div class="row pci-row">

diff --git a/src/vmm/LibVirtDriverKVM.cc b/src/vmm/LibVirtDriverKVM.cc
@@ -615,7 +615,8 @@ int LibVirtDriver::deployment_description_kvm(
     string o_peak_kb;
 
     string default_filter;
-    string default_model ;
+    string default_model;
+    string default_virtio_queues;
 
     const VectorAttribute * graphics;
 
@@ -1697,6 +1698,8 @@ int LibVirtDriver::deployment_description_kvm(
 
     get_attribute(nullptr, host, cluster, "NIC", "MODEL", default_model);
 
+    get_attribute(nullptr, host, cluster, "NIC", "VIRTIO_QUEUES", default_virtio_queues);
+
     num = vm->get_template_attribute("NIC", nic);
 
     for (int i=0; i<num; i++)
@@ -1795,6 +1798,17 @@ int LibVirtDriver::deployment_description_kvm(
             file << "\t\t\t<model type="
                  << one_util::escape_xml_attr(*the_model) << "/>\n";
 
+            if (!virtio_queues.empty())
+            {
+                set_queues(virtio_queues, vcpu)
+            }
+            else if (!default_virtio_queues.empty())
+            {
+                set_queues(default_virtio_queues, vcpu)
+
+                virtio_queues = default_virtio_queues;
+            }
+
             if (!virtio_queues.empty() && *the_model == "virtio")
             {
                 file << "\t\t\t<driver name='vhost' queues="

diff --git a/src/vmm_mad/exec/vmm_exec_kvm.conf b/src/vmm_mad/exec/vmm_exec_kvm.conf
@@ -70,6 +70,7 @@
 #  * NIC:
 #      - FILTER
 #      - MODEL
+#      - VIRTIO_QUEUES
 #  * RAW: raw attribute value is appended to that on the VM template
 #  * HYPERV_OPTIONS: options used for FEATURES = [ HYPERV = yes ]
 #  * HYPERV_TIMERS: timers added when HYPERV is set to yes in FEATURES
@@ -114,7 +115,8 @@ DISK = [
 ]
 
 NIC = [
-    MODEL = "virtio"
+    MODEL = "virtio",
+    VIRTIO_QUEUES = "auto"
     # FILTER = "clean-traffic"
 ]