[PATCH 10/12] domain_validate.c: make virDomainDeviceDefValidateInternal() helpers static

Daniel Henrique Barboza danielhb413 at gmail.com
Fri Dec 11 18:05:14 UTC 2020


After the move from the previous patch, these functions are now all
used in domain_validate.c and doesn't need to be public.

Signed-off-by: Daniel Henrique Barboza <danielhb413 at gmail.com>
---
 src/conf/domain_validate.c | 27 ++++++++++++++-------------
 src/conf/domain_validate.h | 20 --------------------
 2 files changed, 14 insertions(+), 33 deletions(-)

diff --git a/src/conf/domain_validate.c b/src/conf/domain_validate.c
index 8acb7c1044..2a881f1f06 100644
--- a/src/conf/domain_validate.c
+++ b/src/conf/domain_validate.c
@@ -76,7 +76,7 @@ virDomainDefVideoValidate(const virDomainDef *def)
 }
 
 
-int
+static int
 virDomainVideoDefValidate(const virDomainVideoDef *video,
                           const virDomainDef *def)
 {
@@ -229,7 +229,7 @@ virSecurityDeviceLabelDefValidate(virSecurityDeviceLabelDefPtr *seclabels,
 #define VENDOR_LEN  8
 #define PRODUCT_LEN 16
 
-int
+static int
 virDomainDiskDefValidate(const virDomainDef *def,
                          const virDomainDiskDef *disk)
 {
@@ -446,7 +446,7 @@ virDomainChrSourceDefValidate(const virDomainChrSourceDef *src_def,
 }
 
 
-int
+static int
 virDomainRedirdevDefValidate(const virDomainDef *def,
                              const virDomainRedirdevDef *redirdev)
 {
@@ -462,7 +462,7 @@ virDomainRedirdevDefValidate(const virDomainDef *def,
 }
 
 
-int
+static int
 virDomainChrDefValidate(const virDomainChrDef *chr,
                         const virDomainDef *def)
 {
@@ -470,7 +470,7 @@ virDomainChrDefValidate(const virDomainChrDef *chr,
 }
 
 
-int
+static int
 virDomainRNGDefValidate(const virDomainRNGDef *rng,
                         const virDomainDef *def)
 {
@@ -481,7 +481,7 @@ virDomainRNGDefValidate(const virDomainRNGDef *rng,
 }
 
 
-int
+static int
 virDomainSmartcardDefValidate(const virDomainSmartcardDef *smartcard,
                               const virDomainDef *def)
 {
@@ -520,7 +520,7 @@ virDomainDefTunablesValidate(const virDomainDef *def)
 }
 
 
-int
+static int
 virDomainControllerDefValidate(const virDomainControllerDef *controller)
 {
     if (controller->type == VIR_DOMAIN_CONTROLLER_TYPE_PCI) {
@@ -1273,7 +1273,7 @@ virDomainActualNetDefValidate(const virDomainNetDef *net)
 }
 
 
-int
+static int
 virDomainNetDefValidate(const virDomainNetDef *net)
 {
     char macstr[VIR_MAC_STRING_BUFLEN];
@@ -1323,7 +1323,7 @@ virDomainNetDefValidate(const virDomainNetDef *net)
 }
 
 
-int
+static int
 virDomainHostdevDefValidate(const virDomainHostdevDef *hostdev)
 {
     if (hostdev->mode == VIR_DOMAIN_HOSTDEV_MODE_SUBSYS) {
@@ -1374,7 +1374,7 @@ virDomainHostdevDefValidate(const virDomainHostdevDef *hostdev)
 }
 
 
-int
+static int
 virDomainMemoryDefValidate(const virDomainMemoryDef *mem,
                            const virDomainDef *def)
 {
@@ -1402,7 +1402,7 @@ virDomainMemoryDefValidate(const virDomainMemoryDef *mem,
 }
 
 
-int
+static int
 virDomainVsockDefValidate(const virDomainVsockDef *vsock)
 {
     if (vsock->guest_cid > 0 && vsock->guest_cid <= 2) {
@@ -1414,7 +1414,8 @@ virDomainVsockDefValidate(const virDomainVsockDef *vsock)
     return 0;
 }
 
-int
+
+static int
 virDomainInputDefValidate(const virDomainInputDef *input)
 {
     switch ((virDomainInputType) input->type) {
@@ -1448,7 +1449,7 @@ virDomainInputDefValidate(const virDomainInputDef *input)
 }
 
 
-int
+static int
 virDomainShmemDefValidate(const virDomainShmemDef *shmem)
 {
     if (strchr(shmem->name, '/')) {
diff --git a/src/conf/domain_validate.h b/src/conf/domain_validate.h
index 710515a6ff..addffc3819 100644
--- a/src/conf/domain_validate.h
+++ b/src/conf/domain_validate.h
@@ -25,23 +25,10 @@
 #include "domain_conf.h"
 #include "virconftypes.h"
 
-int virDomainVideoDefValidate(const virDomainVideoDef *video,
-                              const virDomainDef *def);
 int virSecurityDeviceLabelDefValidate(virSecurityDeviceLabelDefPtr *seclabels,
                                       size_t nseclabels,
                                       virSecurityLabelDefPtr *vmSeclabels,
                                       size_t nvmSeclabels);
-int virDomainDiskDefValidate(const virDomainDef *def,
-                             const virDomainDiskDef *disk);
-int virDomainRedirdevDefValidate(const virDomainDef *def,
-                                 const virDomainRedirdevDef *redirdev);
-int virDomainChrDefValidate(const virDomainChrDef *chr,
-                            const virDomainDef *def);
-int virDomainRNGDefValidate(const virDomainRNGDef *rng,
-                            const virDomainDef *def);
-int virDomainSmartcardDefValidate(const virDomainSmartcardDef *smartcard,
-                                  const virDomainDef *def);
-int virDomainControllerDefValidate(const virDomainControllerDef *controller);
 int virDomainDeviceValidateAliasForHotplug(virDomainObjPtr vm,
                                            virDomainDeviceDefPtr dev,
                                            unsigned int flags);
@@ -50,13 +37,6 @@ int virDomainDefValidate(virDomainDefPtr def,
                          virDomainXMLOptionPtr xmlopt,
                          void *parseOpaque);
 int virDomainActualNetDefValidate(const virDomainNetDef *net);
-int virDomainNetDefValidate(const virDomainNetDef *net);
-int virDomainHostdevDefValidate(const virDomainHostdevDef *hostdev);
-int virDomainMemoryDefValidate(const virDomainMemoryDef *mem,
-                               const virDomainDef *def);
-int virDomainVsockDefValidate(const virDomainVsockDef *vsock);
-int virDomainInputDefValidate(const virDomainInputDef *input);
-int virDomainShmemDefValidate(const virDomainShmemDef *shmem);
 int virDomainDeviceDefValidate(const virDomainDeviceDef *dev,
                                const virDomainDef *def,
                                unsigned int parseFlags,
-- 
2.26.2




More information about the libvir-list mailing list