[PATCH v3 10/18] conf: Rework storage_adapter_conf to use adapter specific typedefs

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

 



Rework the helpers/APIs to use the FCHost and SCSIHost adapter types.
Continue to realign the code for shorter lines.

Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
---
 src/conf/storage_adapter_conf.c | 140 ++++++++++++++++------------------------
 1 file changed, 57 insertions(+), 83 deletions(-)

diff --git a/src/conf/storage_adapter_conf.c b/src/conf/storage_adapter_conf.c
index a361c34..6efe5ae 100644
--- a/src/conf/storage_adapter_conf.c
+++ b/src/conf/storage_adapter_conf.c
@@ -38,14 +38,14 @@ VIR_ENUM_IMPL(virStoragePoolSourceAdapter,
 
 
 static void
-virStorageAdapterFCHostClear(virStoragePoolSourceAdapterPtr adapter)
+virStorageAdapterFCHostClear(virStorageAdapterFCHostPtr fchost)
 {
-    VIR_FREE(adapter->data.fchost.wwnn);
-    VIR_FREE(adapter->data.fchost.wwpn);
-    VIR_FREE(adapter->data.fchost.parent);
-    VIR_FREE(adapter->data.fchost.parent_wwnn);
-    VIR_FREE(adapter->data.fchost.parent_wwpn);
-    VIR_FREE(adapter->data.fchost.parent_fabric_wwn);
+    VIR_FREE(fchost->wwnn);
+    VIR_FREE(fchost->wwpn);
+    VIR_FREE(fchost->parent);
+    VIR_FREE(fchost->parent_wwnn);
+    VIR_FREE(fchost->parent_wwpn);
+    VIR_FREE(fchost->parent_fabric_wwn);
 }
 
 
@@ -53,7 +53,7 @@ void
 virStorageAdapterClear(virStoragePoolSourceAdapterPtr adapter)
 {
     if (adapter->type == VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_FC_HOST)
-        virStorageAdapterFCHostClear(adapter);
+        virStorageAdapterFCHostClear(&adapter->data.fchost);
 
     if (adapter->type == VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_SCSI_HOST)
         VIR_FREE(adapter->data.scsi_host.name);
@@ -62,15 +62,13 @@ virStorageAdapterClear(virStoragePoolSourceAdapterPtr adapter)
 
 static int
 virStorageAdapterFCHostParseXML(xmlNodePtr node,
-                                virStoragePoolSourcePtr source)
+                                virStorageAdapterFCHostPtr fchost)
 {
     char *managed = NULL;
 
-    source->adapter.data.fchost.parent = virXMLPropString(node, "parent");
+    fchost->parent = virXMLPropString(node, "parent");
     if ((managed = virXMLPropString(node, "managed"))) {
-        source->adapter.data.fchost.managed =
-            virTristateBoolTypeFromString(managed);
-        if (source->adapter.data.fchost.managed < 0) {
+        if ((fchost->managed = virTristateBoolTypeFromString(managed)) < 0) {
             virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                            _("unknown fc_host managed setting '%s'"),
                            managed);
@@ -79,15 +77,11 @@ virStorageAdapterFCHostParseXML(xmlNodePtr node,
         }
     }
 
-    source->adapter.data.fchost.parent_wwnn =
-        virXMLPropString(node, "parent_wwnn");
-    source->adapter.data.fchost.parent_wwpn =
-        virXMLPropString(node, "parent_wwpn");
-    source->adapter.data.fchost.parent_fabric_wwn =
-        virXMLPropString(node, "parent_fabric_wwn");
-
-    source->adapter.data.fchost.wwpn = virXMLPropString(node, "wwpn");
-    source->adapter.data.fchost.wwnn = virXMLPropString(node, "wwnn");
+    fchost->parent_wwnn = virXMLPropString(node, "parent_wwnn");
+    fchost->parent_wwpn = virXMLPropString(node, "parent_wwpn");
+    fchost->parent_fabric_wwn = virXMLPropString(node, "parent_fabric_wwn");
+    fchost->wwpn = virXMLPropString(node, "wwpn");
+    fchost->wwnn = virXMLPropString(node, "wwnn");
 
     VIR_FREE(managed);
     return 0;
@@ -97,27 +91,24 @@ virStorageAdapterFCHostParseXML(xmlNodePtr node,
 static int
 virStorageAdapterSCSIHostParseXML(xmlNodePtr node,
                                   xmlXPathContextPtr ctxt,
-                                  virStoragePoolSourcePtr source)
+                                  virStorageAdapterSCSIHostPtr scsi_host)
 {
-    source->adapter.data.scsi_host.name =
-        virXMLPropString(node, "name");
+    scsi_host->name = virXMLPropString(node, "name");
     if (virXPathNode("./parentaddr", ctxt)) {
         xmlNodePtr addrnode = virXPathNode("./parentaddr/address", ctxt);
-        virPCIDeviceAddressPtr addr =
-            &source->adapter.data.scsi_host.parentaddr;
 
         if (!addrnode) {
             virReportError(VIR_ERR_XML_ERROR, "%s",
                            _("Missing scsi_host PCI address element"));
             return -1;
         }
-        source->adapter.data.scsi_host.has_parent = true;
-        if (virPCIDeviceAddressParseXML(addrnode, addr) < 0)
+        scsi_host->has_parent = true;
+        if (virPCIDeviceAddressParseXML(addrnode, &scsi_host->parentaddr) < 0)
             return -1;
         if ((virXPathInt("string(./parentaddr/@unique_id)",
                          ctxt,
-                         &source->adapter.data.scsi_host.unique_id) < 0) ||
-            (source->adapter.data.scsi_host.unique_id < 0)) {
+                         &scsi_host->unique_id) < 0) ||
+            (scsi_host->unique_id < 0)) {
             virReportError(VIR_ERR_XML_ERROR, "%s",
                            _("Missing or invalid scsi adapter "
                              "'unique_id' value"));
@@ -132,7 +123,7 @@ virStorageAdapterSCSIHostParseXML(xmlNodePtr node,
 static int
 virStorageAdapterLegacyParseXML(xmlNodePtr node,
                                 xmlXPathContextPtr ctxt,
-                                virStoragePoolSourcePtr source)
+                                virStoragePoolSourceAdapterPtr adapter)
 {
     char *wwnn = virXMLPropString(node, "wwnn");
     char *wwpn = virXMLPropString(node, "wwpn");
@@ -162,10 +153,8 @@ virStorageAdapterLegacyParseXML(xmlNodePtr node,
     /* To keep back-compat, 'type' is not required to specify
      * for scsi_host adapter.
      */
-    if ((source->adapter.data.scsi_host.name =
-         virXMLPropString(node, "name")))
-        source->adapter.type =
-            VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_SCSI_HOST;
+    if ((adapter->data.scsi_host.name = virXMLPropString(node, "name")))
+        adapter->type = VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_SCSI_HOST;
 
     return 0;
 }
@@ -193,16 +182,16 @@ virStorageAdapterParseXML(virStoragePoolSourcePtr source,
 
         if (source->adapter.type ==
             VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_FC_HOST) {
-            if (virStorageAdapterFCHostParseXML(node, source) < 0)
+            if (virStorageAdapterFCHostParseXML(node, &source->adapter.data.fchost) < 0)
                 goto cleanup;
         } else if (source->adapter.type ==
                    VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_SCSI_HOST) {
-            if (virStorageAdapterSCSIHostParseXML(node, ctxt, source) < 0)
+            if (virStorageAdapterSCSIHostParseXML(node, ctxt, &source->adapter.data.scsi_host) < 0)
                 goto cleanup;
 
         }
     } else {
-        if (virStorageAdapterLegacyParseXML(node, ctxt, source) < 0)
+        if (virStorageAdapterLegacyParseXML(node, ctxt, &source->adapter) < 0)
             goto cleanup;
     }
 
@@ -216,40 +205,33 @@ virStorageAdapterParseXML(virStoragePoolSourcePtr source,
 
 
 static int
-virStorageAdapterFCHostParseValidate(virStoragePoolDefPtr ret)
+virStorageAdapterFCHostParseValidate(virStorageAdapterFCHostPtr fchost)
 {
-    if (!ret->source.adapter.data.fchost.wwnn ||
-        !ret->source.adapter.data.fchost.wwpn) {
+    if (!fchost->wwnn || !fchost->wwpn) {
         virReportError(VIR_ERR_XML_ERROR, "%s",
                        _("'wwnn' and 'wwpn' must be specified for adapter "
                          "type 'fchost'"));
         return -1;
     }
 
-    if (!virValidateWWN(ret->source.adapter.data.fchost.wwnn) ||
-        !virValidateWWN(ret->source.adapter.data.fchost.wwpn))
+    if (!virValidateWWN(fchost->wwnn) || !virValidateWWN(fchost->wwpn))
         return -1;
 
-    if ((ret->source.adapter.data.fchost.parent_wwnn &&
-         !ret->source.adapter.data.fchost.parent_wwpn) ||
-        (!ret->source.adapter.data.fchost.parent_wwnn &&
-         ret->source.adapter.data.fchost.parent_wwpn)) {
+    if ((fchost->parent_wwnn && !fchost->parent_wwpn) ||
+        (!fchost->parent_wwnn && fchost->parent_wwpn)) {
         virReportError(VIR_ERR_XML_ERROR, "%s",
                        _("must supply both parent_wwnn and "
                          "parent_wwpn not just one or the other"));
         return -1;
     }
 
-    if (ret->source.adapter.data.fchost.parent_wwnn &&
-        !virValidateWWN(ret->source.adapter.data.fchost.parent_wwnn))
+    if (fchost->parent_wwnn && !virValidateWWN(fchost->parent_wwnn))
         return -1;
 
-    if (ret->source.adapter.data.fchost.parent_wwpn &&
-        !virValidateWWN(ret->source.adapter.data.fchost.parent_wwpn))
+    if (fchost->parent_wwpn && !virValidateWWN(fchost->parent_wwpn))
         return -1;
 
-    if (ret->source.adapter.data.fchost.parent_fabric_wwn &&
-        !virValidateWWN(ret->source.adapter.data.fchost.parent_fabric_wwn))
+    if (fchost->parent_fabric_wwn && !virValidateWWN(fchost->parent_fabric_wwn))
         return -1;
 
     return 0;
@@ -257,18 +239,16 @@ virStorageAdapterFCHostParseValidate(virStoragePoolDefPtr ret)
 
 
 static int
-virStorageAdapterSCSIHostParseValidate(virStoragePoolDefPtr ret)
+virStorageAdapterSCSIHostParseValidate(virStorageAdapterSCSIHostPtr scsi_host)
 {
-    if (!ret->source.adapter.data.scsi_host.name &&
-        !ret->source.adapter.data.scsi_host.has_parent) {
+    if (!scsi_host->name && !scsi_host->has_parent) {
         virReportError(VIR_ERR_XML_ERROR, "%s",
                        _("Either 'name' or 'parent' must be specified "
                          "for the 'scsi_host' adapter"));
         return -1;
     }
 
-    if (ret->source.adapter.data.scsi_host.name &&
-        ret->source.adapter.data.scsi_host.has_parent) {
+    if (scsi_host->name && scsi_host->has_parent) {
         virReportError(VIR_ERR_XML_ERROR, "%s",
                        _("Both 'name' and 'parent' cannot be specified "
                          "for the 'scsi_host' adapter"));
@@ -290,11 +270,11 @@ virStorageAdapterParseValidate(virStoragePoolDefPtr ret)
 
     if (ret->source.adapter.type ==
         VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_FC_HOST)
-        return virStorageAdapterFCHostParseValidate(ret);
+        return virStorageAdapterFCHostParseValidate(&ret->source.adapter.data.fchost);
 
     if (ret->source.adapter.type ==
         VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_SCSI_HOST)
-        return virStorageAdapterSCSIHostParseValidate(ret);
+        return virStorageAdapterSCSIHostParseValidate(&ret->source.adapter.data.scsi_host);
 
     return 0;
 }
@@ -302,42 +282,36 @@ virStorageAdapterParseValidate(virStoragePoolDefPtr ret)
 
 static void
 virStorageAdapterFCHostFormat(virBufferPtr buf,
-                              virStoragePoolSourcePtr src)
+                              virStorageAdapterFCHostPtr fchost)
 {
-    virBufferEscapeString(buf, " parent='%s'",
-                          src->adapter.data.fchost.parent);
-    if (src->adapter.data.fchost.managed)
+    virBufferEscapeString(buf, " parent='%s'", fchost->parent);
+    if (fchost->managed)
         virBufferAsprintf(buf, " managed='%s'",
-                          virTristateBoolTypeToString(src->adapter.data.fchost.managed));
-    virBufferEscapeString(buf, " parent_wwnn='%s'",
-                          src->adapter.data.fchost.parent_wwnn);
-    virBufferEscapeString(buf, " parent_wwpn='%s'",
-                          src->adapter.data.fchost.parent_wwpn);
+                          virTristateBoolTypeToString(fchost->managed));
+    virBufferEscapeString(buf, " parent_wwnn='%s'", fchost->parent_wwnn);
+    virBufferEscapeString(buf, " parent_wwpn='%s'", fchost->parent_wwpn);
     virBufferEscapeString(buf, " parent_fabric_wwn='%s'",
-                          src->adapter.data.fchost.parent_fabric_wwn);
+                          fchost->parent_fabric_wwn);
 
     virBufferAsprintf(buf, " wwnn='%s' wwpn='%s'/>\n",
-                      src->adapter.data.fchost.wwnn,
-                      src->adapter.data.fchost.wwpn);
+                      fchost->wwnn, fchost->wwpn);
 }
 
 
 static void
 virStorageAdapterSCSIHostFormat(virBufferPtr buf,
-                                virStoragePoolSourcePtr src)
+                                virStorageAdapterSCSIHostPtr scsi_host)
 {
-    if (src->adapter.data.scsi_host.name) {
-        virBufferAsprintf(buf, " name='%s'/>\n",
-                          src->adapter.data.scsi_host.name);
+    if (scsi_host->name) {
+        virBufferAsprintf(buf, " name='%s'/>\n", scsi_host->name);
     } else {
-        virPCIDeviceAddress addr;
         virBufferAddLit(buf, ">\n");
         virBufferAdjustIndent(buf, 2);
         virBufferAsprintf(buf, "<parentaddr unique_id='%d'>\n",
-                          src->adapter.data.scsi_host.unique_id);
+                          scsi_host->unique_id);
         virBufferAdjustIndent(buf, 2);
-        addr = src->adapter.data.scsi_host.parentaddr;
-        ignore_value(virPCIDeviceAddressFormat(buf, addr, false));
+        ignore_value(virPCIDeviceAddressFormat(buf, scsi_host->parentaddr,
+                                               false));
         virBufferAdjustIndent(buf, -2);
         virBufferAddLit(buf, "</parentaddr>\n");
         virBufferAdjustIndent(buf, -2);
@@ -354,8 +328,8 @@ virStorageAdapterFormat(virBufferPtr buf,
                       virStoragePoolSourceAdapterTypeToString(src->adapter.type));
 
     if (src->adapter.type == VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_FC_HOST)
-        virStorageAdapterFCHostFormat(buf, src);
+        virStorageAdapterFCHostFormat(buf, &src->adapter.data.fchost);
 
     if (src->adapter.type == VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_SCSI_HOST)
-        virStorageAdapterSCSIHostFormat(buf, src);
+        virStorageAdapterSCSIHostFormat(buf, &src->adapter.data.scsi_host);
 }
-- 
2.9.3

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[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