From: Andrew Cooper Date: Mon, 6 Jan 2020 13:22:11 +0000 (+0000) Subject: x86/smpboot: Use printk_once() rather than opencoding it X-Git-Tag: 4.14.0-rc1~841 X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=cb396d8256ddb416a9a5583fa19560a3e05d5cc6;p=xen.git x86/smpboot: Use printk_once() rather than opencoding it Shrink the text to be less verbose. Signed-off-by: Andrew Cooper Reviewed-by: Jan Beulich --- diff --git a/xen/arch/x86/smpboot.c b/xen/arch/x86/smpboot.c index a7e7013097..7e29704080 100644 --- a/xen/arch/x86/smpboot.c +++ b/xen/arch/x86/smpboot.c @@ -1291,14 +1291,11 @@ int cpu_add(uint32_t apic_id, uint32_t acpi_id, uint32_t pxm) /* Physically added CPUs do not have synchronised TSC. */ if ( boot_cpu_has(X86_FEATURE_TSC_RELIABLE) ) { - static bool once_only; - - if ( !test_and_set_bool(once_only) ) - printk(XENLOG_WARNING - " ** New physical CPU %u may have skewed TSC and hence " - "break assumed cross-CPU TSC coherency.\n" - " ** Consider using boot parameter \"tsc=skewed\" " - "which forces TSC emulation where appropriate.\n", cpu); + printk_once( + XENLOG_WARNING + "New CPU %u may have skewed TSC and break cross-CPU TSC coherency\n" + "Consider using \"tsc=skewed\" to force emulation where appropriate\n", + cpu); cpumask_set_cpu(cpu, &tsc_sync_cpu_mask); }