From 24fa7004e47ce86b92bc23c1f2ef9c3d6152c3a8 Mon Sep 17 00:00:00 2001
From: Michal Privoznik <mprivozn@redhat.com>
Date: Thu, 7 Jul 2022 16:29:18 +0200
Subject: [PATCH] domain_conf: Format <defaultiothread/> more often
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The <defaultiothread/> element is formatted inside
virDomainDefaultIOThreadDefFormat() which is called only from
virDomainDefIOThreadsFormat() (so that IOThread related stuff is
formatted calling one function). However, when there are no
<iothreadids/> defined (or only autoallocated ones are present),
then the outer formatting function exits early never calling the
<defaultiothread/> formatter.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
---
 src/conf/domain_conf.c | 60 ++++++++++++++++++++----------------------
 1 file changed, 29 insertions(+), 31 deletions(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index b639022396..24f17a8b91 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -26349,39 +26349,37 @@ static void
 virDomainDefIOThreadsFormat(virBuffer *buf,
                             const virDomainDef *def)
 {
-    g_auto(virBuffer) childrenBuf = VIR_BUFFER_INIT_CHILD(buf);
-    size_t i;
-
-    if (def->niothreadids == 0)
-        return;
-
-    virBufferAsprintf(buf, "<iothreads>%zu</iothreads>\n",
-                      def->niothreadids);
-
-    if (!virDomainDefIothreadShouldFormat(def))
-        return;
-
-    for (i = 0; i < def->niothreadids; i++) {
-        virDomainIOThreadIDDef *iothread = def->iothreadids[i];
-        g_auto(virBuffer) attrBuf = VIR_BUFFER_INITIALIZER;
-
-        virBufferAsprintf(&attrBuf, " id='%u'",
-                          iothread->iothread_id);
-
-        if (iothread->thread_pool_min >= 0) {
-            virBufferAsprintf(&attrBuf, " thread_pool_min='%d'",
-                              iothread->thread_pool_min);
-        }
-
-        if (iothread->thread_pool_max >= 0) {
-            virBufferAsprintf(&attrBuf, " thread_pool_max='%d'",
-                              iothread->thread_pool_max);
-        }
-
-        virXMLFormatElement(&childrenBuf, "iothread", &attrBuf, NULL);
+    if (def->niothreadids > 0) {
+        virBufferAsprintf(buf, "<iothreads>%zu</iothreads>\n",
+                          def->niothreadids);
     }
 
-    virXMLFormatElement(buf, "iothreadids", NULL, &childrenBuf);
+    if (virDomainDefIothreadShouldFormat(def)) {
+        g_auto(virBuffer) childrenBuf = VIR_BUFFER_INIT_CHILD(buf);
+        size_t i;
+
+        for (i = 0; i < def->niothreadids; i++) {
+            virDomainIOThreadIDDef *iothread = def->iothreadids[i];
+            g_auto(virBuffer) attrBuf = VIR_BUFFER_INITIALIZER;
+
+            virBufferAsprintf(&attrBuf, " id='%u'",
+                              iothread->iothread_id);
+
+            if (iothread->thread_pool_min >= 0) {
+                virBufferAsprintf(&attrBuf, " thread_pool_min='%d'",
+                                  iothread->thread_pool_min);
+            }
+
+            if (iothread->thread_pool_max >= 0) {
+                virBufferAsprintf(&attrBuf, " thread_pool_max='%d'",
+                                  iothread->thread_pool_max);
+            }
+
+            virXMLFormatElement(&childrenBuf, "iothread", &attrBuf, NULL);
+        }
+
+        virXMLFormatElement(buf, "iothreadids", NULL, &childrenBuf);
+    }
 
     virDomainDefaultIOThreadDefFormat(buf, def);
 }