qemu: Make most qemuDomainMachine*() functions static

Ideally we'd make all of them static, but there are a few
cases where we don't have a virDomainDef instance handy and
so they are the only option.

For the few ones we're forced to keep exporting, document
through comments that the alternative is preferred.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index edc80a8fc1d8988aff1b95919763bbfb26898174..09a55703127964c279633f5a5eb5463fe3c9c269 100644 (file)
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -9875,7 +9875,7 @@ qemuFindAgentConfig(virDomainDefPtr def)
 }
 
 
-bool
+static bool
 qemuDomainMachineIsQ35(const char *machine,
                        const virArch arch)
 {
@@ -9887,7 +9887,7 @@ qemuDomainMachineIsQ35(const char *machine,
 }
 
 
-bool
+static bool
 qemuDomainMachineIsI440FX(const char *machine,
                           const virArch arch)
 {
@@ -9902,7 +9902,7 @@ qemuDomainMachineIsI440FX(const char *machine,
 }
 
 
-bool
+static bool
 qemuDomainMachineIsS390CCW(const char *machine,
                            const virArch arch)
 {
@@ -9913,6 +9913,8 @@ qemuDomainMachineIsS390CCW(const char *machine,
 }
 
 
+/* You should normally avoid this function and use
+ * qemuDomainIsARMVirt() instead. */
 bool
 qemuDomainMachineIsARMVirt(const char *machine,
                            const virArch arch)
@@ -9930,7 +9932,7 @@ qemuDomainMachineIsARMVirt(const char *machine,
 }
 
 
-bool
+static bool
 qemuDomainMachineIsRISCVVirt(const char *machine,
                              const virArch arch)
 {
@@ -9945,6 +9947,8 @@ qemuDomainMachineIsRISCVVirt(const char *machine,
 }
 
 
+/* You should normally avoid this function and use
+ * qemuDomainIsPSeries() instead. */
 bool
 qemuDomainMachineIsPSeries(const char *machine,
                            const virArch arch)
@@ -9960,6 +9964,8 @@ qemuDomainMachineIsPSeries(const char *machine,
 }
 
 
+/* You should normally avoid this function and use
+ * qemuDomainHasBuiltinIDE() instead. */
 bool
 qemuDomainMachineHasBuiltinIDE(const char *machine,
                                const virArch arch)
@@ -9971,7 +9977,7 @@ qemuDomainMachineHasBuiltinIDE(const char *machine,
 }
 
 
-bool
+static bool
 qemuDomainMachineNeedsFDC(const char *machine,
                           const virArch arch)
 {

diff --git a/src/qemu/qemu_domain.h b/src/qemu/qemu_domain.h
index c0b8b07eb1f2f11cb9dc840ca8eb290433763de8..b934fe3088d0022d522cfa4b2e1a88ad6d1656e9 100644 (file)
--- a/src/qemu/qemu_domain.h
+++ b/src/qemu/qemu_domain.h
@@ -828,22 +828,14 @@ void qemuDomainMemoryDeviceAlignSize(virDomainDefPtr def,
 
 virDomainChrDefPtr qemuFindAgentConfig(virDomainDefPtr def);
 
-bool qemuDomainMachineIsQ35(const char *machine,
-                            const virArch arch);
-bool qemuDomainMachineIsI440FX(const char *machine,
-                               const virArch arch);
-bool qemuDomainMachineIsS390CCW(const char *machine,
-                                const virArch arch);
+/* You should normally avoid these functions and use the variant that
+ * doesn't have "Machine" in the name instead. */
 bool qemuDomainMachineIsARMVirt(const char *machine,
                                 const virArch arch);
-bool qemuDomainMachineIsRISCVVirt(const char *machine,
-                                  const virArch arch);
 bool qemuDomainMachineIsPSeries(const char *machine,
                                 const virArch arch);
 bool qemuDomainMachineHasBuiltinIDE(const char *machine,
                                     const virArch arch);
-bool qemuDomainMachineNeedsFDC(const char *machine,
-                               const virArch arch);
 
 bool qemuDomainIsQ35(const virDomainDef *def);
 bool qemuDomainIsI440FX(const virDomainDef *def);