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

[libvirt PATCH 03/19] util: virHostMem*Parameters: split out non-Linux stubs



Repeat the whole function header instead of mixing #ifdefs
in the code.

Signed-off-by: Ján Tomko <jtomko redhat com>
---
 src/util/virhostmem.c | 32 ++++++++++++++++++++++++--------
 1 file changed, 24 insertions(+), 8 deletions(-)

diff --git a/src/util/virhostmem.c b/src/util/virhostmem.c
index 7d1871610c..27f192dc24 100644
--- a/src/util/virhostmem.c
+++ b/src/util/virhostmem.c
@@ -354,16 +354,16 @@ virHostMemParametersAreAllSupported(virTypedParameterPtr params,
 }
 #endif
 
+#ifdef __linux__
 int
 virHostMemSetParameters(virTypedParameterPtr params G_GNUC_UNUSED,
                         int nparams G_GNUC_UNUSED,
                         unsigned int flags)
 {
-    virCheckFlags(0, -1);
-
-#ifdef __linux__
     size_t i;
 
+    virCheckFlags(0, -1);
+
     if (virTypedParamsValidate(params, nparams,
                                VIR_NODE_MEMORY_SHARED_PAGES_TO_SCAN,
                                VIR_TYPED_PARAM_UINT,
@@ -383,13 +383,21 @@ virHostMemSetParameters(virTypedParameterPtr params G_GNUC_UNUSED,
     }
 
     return 0;
+}
 #else
+int
+virHostMemSetParameters(virTypedParameterPtr params G_GNUC_UNUSED,
+                        int nparams G_GNUC_UNUSED,
+                        unsigned int flags)
+{
+    virCheckFlags(0, -1);
+
     virReportError(VIR_ERR_NO_SUPPORT, "%s",
                    _("node set memory parameters not implemented"
                      " on this platform"));
     return -1;
-#endif
 }
+#endif
 
 #ifdef __linux__
 static int
@@ -434,14 +442,12 @@ virHostMemGetParameterValue(const char *field,
 #endif
 
 #define NODE_MEMORY_PARAMETERS_NUM 8
+#ifdef __linux__
 int
 virHostMemGetParameters(virTypedParameterPtr params G_GNUC_UNUSED,
                         int *nparams G_GNUC_UNUSED,
                         unsigned int flags)
 {
-    virCheckFlags(VIR_TYPED_PARAM_STRING_OKAY, -1);
-
-#ifdef __linux__
     unsigned int pages_to_scan;
     unsigned int sleep_millisecs;
     unsigned int merge_across_nodes;
@@ -453,6 +459,8 @@ virHostMemGetParameters(virTypedParameterPtr params G_GNUC_UNUSED,
     size_t i;
     int ret;
 
+    virCheckFlags(VIR_TYPED_PARAM_STRING_OKAY, -1);
+
     if ((*nparams) == 0) {
         *nparams = NODE_MEMORY_PARAMETERS_NUM;
         return 0;
@@ -569,13 +577,21 @@ virHostMemGetParameters(virTypedParameterPtr params G_GNUC_UNUSED,
     }
 
     return 0;
+}
 #else
+int
+virHostMemGetParameters(virTypedParameterPtr params G_GNUC_UNUSED,
+                        int *nparams G_GNUC_UNUSED,
+                        unsigned int flags)
+{
+    virCheckFlags(VIR_TYPED_PARAM_STRING_OKAY, -1);
+
     virReportError(VIR_ERR_NO_SUPPORT, "%s",
                    _("node get memory parameters not implemented"
                      " on this platform"));
     return -1;
-#endif
 }
+#endif
 
 
 #ifdef WIN32
-- 
2.26.2


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