[libvirt] [PATCH v2 1/2] virsysinfo: Rename virSysinfoReadX86 to virSysinfoReadDMI

Michal Privoznik mprivozn at redhat.com
Fri May 10 12:20:24 UTC 2019


There's nothing x86 specific about this function. Rename the
function so that it has DMI suffix which enables it to be reused
on different arches (as using X86 from say ARM would look
suspicious).

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 src/libvirt_private.syms  | 2 +-
 src/util/virsysinfo.c     | 4 ++--
 src/util/virsysinfopriv.h | 2 +-
 tests/sysinfotest.c       | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 909975750c..f2b8dc445d 100644
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -3070,9 +3070,9 @@ virSysinfoSystemDefFree;
 
 # util/virsysinfopriv.h
 virSysinfoReadARM;
+virSysinfoReadDMI;
 virSysinfoReadPPC;
 virSysinfoReadS390;
-virSysinfoReadX86;
 
 
 # util/virsystemd.h
diff --git a/src/util/virsysinfo.c b/src/util/virsysinfo.c
index 6c3adc23ab..b371e8dd26 100644
--- a/src/util/virsysinfo.c
+++ b/src/util/virsysinfo.c
@@ -1139,7 +1139,7 @@ virSysinfoParseX86Memory(const char *base, virSysinfoDefPtr ret)
 }
 
 virSysinfoDefPtr
-virSysinfoReadX86(void)
+virSysinfoReadDMI(void)
 {
     char *path;
     virSysinfoDefPtr ret = NULL;
@@ -1220,7 +1220,7 @@ virSysinfoRead(void)
     (defined(__x86_64__) || \
      defined(__i386__) || \
      defined(__amd64__))
-    return virSysinfoReadX86();
+    return virSysinfoReadDMI();
 #else /* WIN32 || not supported arch */
     /*
      * this can probably be extracted from Windows using API or registry
diff --git a/src/util/virsysinfopriv.h b/src/util/virsysinfopriv.h
index 3ef675441e..b7beb44807 100644
--- a/src/util/virsysinfopriv.h
+++ b/src/util/virsysinfopriv.h
@@ -39,6 +39,6 @@ virSysinfoDefPtr
 virSysinfoReadS390(void);
 
 virSysinfoDefPtr
-virSysinfoReadX86(void);
+virSysinfoReadDMI(void);
 
 #endif /* LIBVIRT_VIRSYSINFOPRIV_H */
diff --git a/tests/sysinfotest.c b/tests/sysinfotest.c
index 7fa9a2dfd6..4d03fd3809 100644
--- a/tests/sysinfotest.c
+++ b/tests/sysinfotest.c
@@ -133,7 +133,7 @@ mymain(void)
     TEST("s390", virSysinfoReadS390);
     TEST("s390-freq", virSysinfoReadS390);
     TEST("ppc", virSysinfoReadPPC);
-    TEST_FULL("x86", virSysinfoReadX86, "/sysinfodata/dmidecode.sh");
+    TEST_FULL("x86", virSysinfoReadDMI, "/sysinfodata/dmidecode.sh");
     TEST("arm", virSysinfoReadARM);
     TEST("arm-rpi2", virSysinfoReadARM);
     TEST("aarch64", virSysinfoReadARM);
-- 
2.21.0




More information about the libvir-list mailing list