@@ -793,7 +793,6 @@ static void init_amd(struct cpuinfo_x86 *c)
}
const struct cpu_dev amd_cpu_dev = {
- .c_vendor = "AMD",
.c_early_init = early_init_amd,
.c_init = init_amd,
};
@@ -55,6 +55,5 @@ static void init_centaur(struct cpuinfo_x86 *c)
}
const struct cpu_dev centaur_cpu_dev = {
- .c_vendor = "Centaur",
.c_init = init_centaur,
};
@@ -102,7 +102,6 @@ static void default_init(struct cpuinfo_x86 * c)
static const struct cpu_dev default_cpu = {
.c_init = default_init,
- .c_vendor = "Unknown",
};
static const struct cpu_dev *this_cpu = &default_cpu;
@@ -306,7 +305,7 @@ static void __init early_cpu_detect(void)
printk(XENLOG_INFO
"CPU Vendor: %s, Family %u (%#x), Model %u (%#x), Stepping %u (raw %08x)\n",
- this_cpu->c_vendor, c->x86, c->x86,
+ x86_cpuid_vendor_to_str(c->x86_vendor), c->x86, c->x86,
c->x86_model, c->x86_model, c->x86_mask, eax);
eax = cpuid_eax(0x80000000);
@@ -661,12 +660,8 @@ void print_cpu_info(unsigned int cpu)
printk("CPU%u: ", cpu);
- if (c->x86_vendor < X86_VENDOR_NUM)
- vendor = this_cpu->c_vendor;
- else
- vendor = c->x86_vendor_id;
-
- if (vendor && strncmp(c->x86_model_id, vendor, strlen(vendor)))
+ vendor = x86_cpuid_vendor_to_str(c->x86_vendor);
+ if (strncmp(c->x86_model_id, vendor, strlen(vendor)))
printk("%s ", vendor);
if (!c->x86_model_id[0])
@@ -1,7 +1,5 @@
/* attempt to consolidate cpu attributes */
struct cpu_dev {
- char c_vendor[8];
-
void (*c_early_init)(struct cpuinfo_x86 *c);
void (*c_init)(struct cpuinfo_x86 * c);
};
@@ -349,7 +349,6 @@ static void init_intel(struct cpuinfo_x86 *c)
}
const struct cpu_dev intel_cpu_dev = {
- .c_vendor = "Intel",
.c_early_init = early_init_intel,
.c_init = init_intel,
};
@@ -16,6 +16,5 @@ static void init_shanghai(struct cpuinfo_x86 *c)
}
const struct cpu_dev shanghai_cpu_dev = {
- .c_vendor = " Shang",
.c_init = init_shanghai,
};
@@ -30,6 +30,4 @@
#define X86_VENDOR_SHANGHAI_ECX 0x20206961U
#define X86_VENDOR_SHANGHAI_EDX 0x68676e61U
-#define X86_VENDOR_NUM 5
-
#endif /* __XEN_X86_VENDORS_H__ */
@@ -71,6 +71,12 @@ static inline void cpuid_count_leaf(
*/
unsigned int x86_cpuid_lookup_vendor(uint32_t ebx, uint32_t ecx, uint32_t edx);
+/**
+ * Given Xen's internal vendor ID, return a string suitable for printing.
+ * Returns "Unknown" for any unrecognised ID.
+ */
+const char *x86_cpuid_vendor_to_str(unsigned int vendor);
+
#define CPUID_GUEST_NR_BASIC (0xdu + 1)
#define CPUID_GUEST_NR_CACHE (5u + 1)
#define CPUID_GUEST_NR_FEAT (0u + 1)
@@ -34,6 +34,18 @@ unsigned int x86_cpuid_lookup_vendor(uint32_t ebx, uint32_t ecx, uint32_t edx)
return X86_VENDOR_UNKNOWN;
}
+const char *x86_cpuid_vendor_to_str(unsigned int vendor)
+{
+ switch ( vendor )
+ {
+ case X86_VENDOR_INTEL: return "Intel";
+ case X86_VENDOR_AMD: return "AMD";
+ case X86_VENDOR_CENTAUR: return "Centaur";
+ case X86_VENDOR_SHANGHAI: return "Shanghai";
+ default: return "Unknown";
+ }
+}
+
/* Recalculate the content in a CPUID policy which is derived from raw data. */
static void recalculate_synth(struct cpuid_policy *p)
{
cpu_dev.c_vendor[] is a char[8] array which is printed using %s in two locations. This leads to subtle lack-of-NUL bugs when using an 8 character vendor name. Introduce x86_cpuid_vendor_to_str() to turn an x86_vendor into a printable string, use it in the two locations that c_vendor is used, and drop c_vendor. This drops the final user of X86_VENDOR_NUM, so drop that as well. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> --- CC: Jan Beulich <JBeulich@suse.com> CC: Wei Liu <wei.liu2@citrix.com> CC: Roger Pau Monné <roger.pau@citrix.com> CC: David Wang <davidwang@zhaoxin.com> CC: Pu Wen <puwen@hygon.cn> David: Same question as Wei asked. Presuambly, " Shang" is a mistake, and "Shanghai" is the correct term to use? --- xen/arch/x86/cpu/amd.c | 1 - xen/arch/x86/cpu/centaur.c | 1 - xen/arch/x86/cpu/common.c | 11 +++-------- xen/arch/x86/cpu/cpu.h | 2 -- xen/arch/x86/cpu/intel.c | 1 - xen/arch/x86/cpu/shanghai.c | 1 - xen/include/asm-x86/x86-vendors.h | 2 -- xen/include/xen/lib/x86/cpuid.h | 6 ++++++ xen/lib/x86/cpuid.c | 12 ++++++++++++ 9 files changed, 21 insertions(+), 16 deletions(-)