virsysinfo: Drop global @sysinfoDmidecode

Since nobody sets custom dmidecode path anymore, we can drop all
code that exists only because of that.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
Michal Privoznik 2020-06-08 12:53:04 +02:00
parent d954fbc74e
commit 994e56ba42
3 changed files with 4 additions and 11 deletions

View File

@ -45,23 +45,18 @@ VIR_ENUM_IMPL(virSysinfo,
"smbios",
);
static const char *sysinfoDmidecode = DMIDECODE;
static const char *sysinfoSysinfo = "/proc/sysinfo";
static const char *sysinfoCpuinfo = "/proc/cpuinfo";
#define SYSINFO_SMBIOS_DECODER sysinfoDmidecode
#define SYSINFO sysinfoSysinfo
#define CPUINFO sysinfoCpuinfo
#define CPUINFO_FILE_LEN (1024*1024) /* 1MB limit for /proc/cpuinfo file */
void
virSysinfoSetup(const char *dmidecode,
const char *sysinfo,
virSysinfoSetup(const char *sysinfo,
const char *cpuinfo)
{
if (dmidecode)
sysinfoDmidecode = dmidecode;
sysinfoSysinfo = sysinfo;
sysinfoCpuinfo = cpuinfo;
}
@ -1124,8 +1119,7 @@ virSysinfoReadDMI(void)
g_autofree char *outbuf = NULL;
g_autoptr(virCommand) cmd = NULL;
cmd = virCommandNewArgList(SYSINFO_SMBIOS_DECODER,
"-q", "-t", "0,1,2,3,4,17", NULL);
cmd = virCommandNewArgList(DMIDECODE, "-q", "-t", "0,1,2,3,4,17", NULL);
virCommandSetOutputBuffer(cmd, &outbuf);
if (virCommandRun(cmd, NULL) < 0)
return NULL;

View File

@ -24,8 +24,7 @@
#pragma once
void
virSysinfoSetup(const char *dmidecode,
const char *sysinfo,
virSysinfoSetup(const char *sysinfo,
const char *cpuinfo);
virSysinfoDefPtr

View File

@ -84,7 +84,7 @@ testSysinfo(const void *data)
virCommandSetDryRun(NULL, testDMIDecodeDryRun, sysinfo);
virSysinfoSetup(NULL, sysinfo, cpuinfo);
virSysinfoSetup(sysinfo, cpuinfo);
ret = testdata->func();
virCommandSetDryRun(NULL, NULL, NULL);