On 30/10/2019 15:32, Jan Beulich wrote: > On 30.10.2019 15:54, Sergey Dyasli wrote: >> @@ -317,11 +316,6 @@ void __init early_cpu_init(void) >> c->x86_capability[cpufeat_word(X86_FEATURE_FPU)] = edx; >> c->x86_capability[cpufeat_word(X86_FEATURE_SSE3)] = ecx; >> >> - printk(XENLOG_INFO >> - "CPU Vendor: %s, Family %u (%#x), Model %u (%#x), Stepping %u >> (raw %08x)\n", >> - x86_cpuid_vendor_to_str(c->x86_vendor), c->x86, c->x86, >> - c->x86_model, c->x86_model, c->x86_mask, eax); > > I'm slightly concerned of the code immediately ahead of this > printk() now running _much_ earlier. Did you inspect that there's > no change of the relevant cleared_caps[] entries between the new > and the old call position in setup.c?
You are right, this idea requires a more sophisticated approach. Please disregard this patch. For now I'll add something like early_cpu_has_hypervisor(). Will send a v3 soon. -- Thanks, Sergey _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
