]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
util: virHostMem*Parameters: split out non-Linux stubs
authorJán Tomko <jtomko@redhat.com>
Mon, 3 Aug 2020 15:58:03 +0000 (17:58 +0200)
committerJán Tomko <jtomko@redhat.com>
Tue, 25 Aug 2020 17:03:11 +0000 (19:03 +0200)
Repeat the whole function header instead of mixing #ifdefs
in the code.

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
src/util/virhostmem.c

index 7d1871610cc269140ab63c3464f149a2a17b7e60..27f192dc24fa8bdf0edd20d1d48551578ab8f0e9 100644 (file)
@@ -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