|
 |
304038 |
From 1b6deafbe0a671d4fd7b8b6e9cc23c8dfcd8683c Mon Sep 17 00:00:00 2001
|
|
|
304038 |
From: Karel Zak <kzak@redhat.com>
|
|
|
304038 |
Date: Tue, 13 Jun 2017 12:15:11 +0200
|
|
|
304038 |
Subject: [PATCH 130/135] lscpu: cleanup DMI detection return codes
|
|
|
304038 |
|
|
|
304038 |
Michal wrote:
|
|
|
304038 |
There is weird mix of logic in lscpu-dmi.c which sometimes returns 0 and
|
|
|
304038 |
sometimes -1 on error. Since most checks are if (rc) goto done; this
|
|
|
304038 |
bails out early on error skipping some detection methods. Further, in
|
|
|
304038 |
lscpu.c all following detections are guarder by if(hyper) so returning
|
|
|
304038 |
-1 causes all following methods to be skipped.
|
|
|
304038 |
|
|
|
304038 |
Upstream: http://github.com/karelzak/util-linux/commit/c972852b29391c35b1d5c7d3e1e6413e0cc86908
|
|
|
304038 |
Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1468646
|
|
|
304038 |
Reported-by: Michal Suchanek <msuchanek@suse.de>
|
|
|
304038 |
Signed-off-by: Karel Zak <kzak@redhat.com>
|
|
|
304038 |
---
|
|
|
304038 |
sys-utils/lscpu-dmi.c | 21 +++++++++++++--------
|
|
|
304038 |
1 file changed, 13 insertions(+), 8 deletions(-)
|
|
|
304038 |
|
|
|
304038 |
diff --git a/sys-utils/lscpu-dmi.c b/sys-utils/lscpu-dmi.c
|
|
|
304038 |
index a8298ff74..e4afd0b92 100644
|
|
|
304038 |
--- a/sys-utils/lscpu-dmi.c
|
|
|
304038 |
+++ b/sys-utils/lscpu-dmi.c
|
|
|
304038 |
@@ -172,7 +172,7 @@ done:
|
|
|
304038 |
static int hypervisor_decode_legacy(uint8_t *buf, const char *devmem)
|
|
|
304038 |
{
|
|
|
304038 |
if (!checksum(buf, 0x0F))
|
|
|
304038 |
- return HYPER_NONE;
|
|
|
304038 |
+ return -1;
|
|
|
304038 |
|
|
|
304038 |
return hypervisor_from_dmi_table(DWORD(buf + 0x08), WORD(buf + 0x06),
|
|
|
304038 |
WORD(buf + 0x0C),
|
|
|
304038 |
@@ -252,11 +252,15 @@ int read_hypervisor_dmi(void)
|
|
|
304038 |
|| sizeof(uint16_t) != 2
|
|
|
304038 |
|| sizeof(uint32_t) != 4
|
|
|
304038 |
|| '\0' != 0)
|
|
|
304038 |
- return rc;
|
|
|
304038 |
+ goto done;
|
|
|
304038 |
|
|
|
304038 |
+ /* -1 : no DMI in /sys,
|
|
|
304038 |
+ * 0 : DMI exist, nothing detected (HYPER_NONE)
|
|
|
304038 |
+ * >0 : hypervisor detected
|
|
|
304038 |
+ */
|
|
|
304038 |
rc = hypervisor_decode_sysfw();
|
|
|
304038 |
- if (rc >= 0)
|
|
|
304038 |
- return rc;
|
|
|
304038 |
+ if (rc >= HYPER_NONE)
|
|
|
304038 |
+ goto done;
|
|
|
304038 |
|
|
|
304038 |
/* First try EFI (ia64, Intel-based Mac) */
|
|
|
304038 |
switch (address_from_efi(&fp)) {
|
|
|
304038 |
@@ -271,8 +275,9 @@ int read_hypervisor_dmi(void)
|
|
|
304038 |
goto done;
|
|
|
304038 |
|
|
|
304038 |
rc = hypervisor_decode_smbios(buf, _PATH_DEV_MEM);
|
|
|
304038 |
- if (rc)
|
|
|
304038 |
+ if (rc >= HYPER_NONE)
|
|
|
304038 |
goto done;
|
|
|
304038 |
+
|
|
|
304038 |
free(buf);
|
|
|
304038 |
buf = NULL;
|
|
|
304038 |
memory_scan:
|
|
|
304038 |
@@ -285,17 +290,17 @@ memory_scan:
|
|
|
304038 |
for (fp = 0; fp <= 0xFFF0; fp += 16) {
|
|
|
304038 |
if (memcmp(buf + fp, "_SM_", 4) == 0 && fp <= 0xFFE0) {
|
|
|
304038 |
rc = hypervisor_decode_smbios(buf + fp, _PATH_DEV_MEM);
|
|
|
304038 |
- if (rc == -1)
|
|
|
304038 |
+ if (rc < 0)
|
|
|
304038 |
fp += 16;
|
|
|
304038 |
|
|
|
304038 |
} else if (memcmp(buf + fp, "_DMI_", 5) == 0)
|
|
|
304038 |
rc = hypervisor_decode_legacy(buf + fp, _PATH_DEV_MEM);
|
|
|
304038 |
|
|
|
304038 |
- if (rc >= 0)
|
|
|
304038 |
+ if (rc >= HYPER_NONE)
|
|
|
304038 |
break;
|
|
|
304038 |
}
|
|
|
304038 |
#endif
|
|
|
304038 |
done:
|
|
|
304038 |
free(buf);
|
|
|
304038 |
- return rc;
|
|
|
304038 |
+ return rc < 0 ? HYPER_NONE : rc;
|
|
|
304038 |
}
|
|
|
304038 |
--
|
|
|
304038 |
2.13.6
|
|
|
304038 |
|