[PATCH 1/8] nodedev: refactor CCW device address format

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

 



Refactor for reuse in the following patch.

Reviewed-by: Marc Hartmayer <mhartmay@xxxxxxxxxxxxx>
Signed-off-by: Boris Fiuczynski <fiuczy@xxxxxxxxxxxxx>
---
 src/conf/node_device_conf.c | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/src/conf/node_device_conf.c b/src/conf/node_device_conf.c
index 08a89942ba..9711594994 100644
--- a/src/conf/node_device_conf.c
+++ b/src/conf/node_device_conf.c
@@ -623,6 +623,19 @@ virNodeDeviceCapVDPADefFormat(virBuffer *buf,
 }
 
 
+static void
+virCCWDeviceAddressFormat(virBuffer *buf,
+                          const virCCWDeviceAddress *ccw_address)
+{
+    virBufferAsprintf(buf, "<cssid>0x%x</cssid>\n",
+                      ccw_address->cssid);
+    virBufferAsprintf(buf, "<ssid>0x%x</ssid>\n",
+                      ccw_address->ssid);
+    virBufferAsprintf(buf, "<devno>0x%04x</devno>\n",
+                      ccw_address->devno);
+}
+
+
 static void
 virNodeDeviceCapCCWDefFormat(virBuffer *buf,
                              const virNodeDevCapData *data)
@@ -645,12 +658,9 @@ virNodeDeviceCapCSSDefFormat(virBuffer *buf,
     virNodeDeviceCapCCWDefFormat(buf, data);
 
     if (ccw_dev.channel_dev_addr) {
-        virCCWDeviceAddress *ccw = ccw_dev.channel_dev_addr;
         virBufferAddLit(buf, "<channel_dev_addr>\n");
         virBufferAdjustIndent(buf, 2);
-        virBufferAsprintf(buf, "<cssid>0x%x</cssid>\n", ccw->cssid);
-        virBufferAsprintf(buf, "<ssid>0x%x</ssid>\n", ccw->ssid);
-        virBufferAsprintf(buf, "<devno>0x%04x</devno>\n", ccw->devno);
+        virCCWDeviceAddressFormat(buf, ccw_dev.channel_dev_addr);
         virBufferAdjustIndent(buf, -2);
         virBufferAddLit(buf, "</channel_dev_addr>\n");
     }
-- 
2.47.0



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux