util: refactor virDomainDeviceCCWAddressIsValid into virccw

Refactor virDomainDeviceCCWAddressIsValid into virccw and rename method
as virCCWDeviceAddressIsValid.

Signed-off-by: Boris Fiuczynski <fiuczy@linux.ibm.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

diff --git a/src/conf/device_conf.c b/src/conf/device_conf.c
index bdc9219f84380d106b4f45de4dcffba64c6a0681..2385e81994f2d2a7e7012cc2b0bf8affc3c36e4a 100644 (file)
--- a/src/conf/device_conf.c
+++ b/src/conf/device_conf.c
@@ -258,14 +258,6 @@ virPCIDeviceAddressFormat(virBuffer *buf,
                       addr.function);
 }
 
-bool
-virDomainDeviceCCWAddressIsValid(virCCWDeviceAddress *addr)
-{
-    return addr->cssid <= VIR_CCW_DEVICE_MAX_CSSID &&
-           addr->ssid <= VIR_CCW_DEVICE_MAX_SSID &&
-           addr->devno <= VIR_CCW_DEVICE_MAX_DEVNO;
-}
-
 int
 virDomainDeviceCCWAddressParseXML(xmlNodePtr node,
                                   virCCWDeviceAddress *addr)
@@ -288,7 +280,7 @@ virDomainDeviceCCWAddressParseXML(xmlNodePtr node,
                                 &addr->devno)) < 0)
         return -1;
 
-    if (!virDomainDeviceCCWAddressIsValid(addr)) {
+    if (!virCCWDeviceAddressIsValid(addr)) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
                        _("Invalid specification for virtio ccw address: cssid='0x%x' ssid='0x%x' devno='0x%04x'"),
                        addr->cssid, addr->ssid, addr->devno);
@@ -453,7 +445,7 @@ virDomainDeviceAddressIsValid(virDomainDeviceInfo *info,
         return true;
 
     case VIR_DOMAIN_DEVICE_ADDRESS_TYPE_CCW:
-        return virDomainDeviceCCWAddressIsValid(&info->addr.ccw);
+        return virCCWDeviceAddressIsValid(&info->addr.ccw);
 
     case VIR_DOMAIN_DEVICE_ADDRESS_TYPE_USB:
         return true;

diff --git a/src/conf/device_conf.h b/src/conf/device_conf.h
index 5c4b7b2f8e13f6289ac8262b69a2698bbb640a12..0ed6991c23566d67ece4674e46d1d13cac06c737 100644 (file)
--- a/src/conf/device_conf.h
+++ b/src/conf/device_conf.h
@@ -193,7 +193,6 @@ void virPCIDeviceAddressFormat(virBuffer *buf,
                                virPCIDeviceAddress addr,
                                bool includeTypeInAddr);
 
-bool virDomainDeviceCCWAddressIsValid(virCCWDeviceAddress *addr);
 int virDomainDeviceCCWAddressParseXML(xmlNodePtr node,
                                       virCCWDeviceAddress *addr);
 bool virDomainDeviceCCWAddressEqual(virCCWDeviceAddress *addr1,

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 070d914910217e34b810fff4ce32724e8c90db50..415a012e179fd79199e96ef90525a17795713a22 100644 (file)
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -132,7 +132,6 @@ virDomainDeviceAddressIsValid;
 virDomainDeviceAddressTypeToString;
 virDomainDeviceCcidAddressParseXML;
 virDomainDeviceCCWAddressEqual;
-virDomainDeviceCCWAddressIsValid;
 virDomainDeviceCCWAddressParseXML;
 virDomainDeviceDriveAddressParseXML;
 virDomainDeviceInfoAddressIsEqual;
@@ -1928,6 +1927,7 @@ virBufferVasprintf;
 # util/virccw.h
 virCCWDeviceAddressAsString;
 virCCWDeviceAddressIncrement;
+virCCWDeviceAddressIsValid;
 
 
 # util/vircgroup.h

diff --git a/src/util/virccw.c b/src/util/virccw.c
index d14d432414c43f7f7be1f5bc86e930c5b14909a3..c3dfda2613d8224131e508ea65f23f9224adc3c2 100644 (file)
--- a/src/util/virccw.c
+++ b/src/util/virccw.c
@@ -22,6 +22,14 @@
 #include "virccw.h"
 
 
+bool
+virCCWDeviceAddressIsValid(virCCWDeviceAddress *addr)
+{
+    return addr->cssid <= VIR_CCW_DEVICE_MAX_CSSID &&
+           addr->ssid <= VIR_CCW_DEVICE_MAX_SSID &&
+           addr->devno <= VIR_CCW_DEVICE_MAX_DEVNO;
+}
+
 char*
 virCCWDeviceAddressAsString(virCCWDeviceAddress *addr)
 {

diff --git a/src/util/virccw.h b/src/util/virccw.h
index 4c48c9605e8b3757758c9fbf5b2d005709d71e1b..c4daaff7b3bf8e4bb05ba766e880400c6bcce847 100644 (file)
--- a/src/util/virccw.h
+++ b/src/util/virccw.h
@@ -35,6 +35,8 @@ struct _virCCWDeviceAddress {
     bool         assigned;
 };
 
+bool virCCWDeviceAddressIsValid(virCCWDeviceAddress *addr);
+
 char* virCCWDeviceAddressAsString(virCCWDeviceAddress *addr)
     ATTRIBUTE_NONNULL(1);
 int virCCWDeviceAddressIncrement(virCCWDeviceAddress *addr);