x86/mpparse: Don't print "limit reached" for every subsequent processor

When you boot Xen with the default 256 NR_CPUS, on a box with rather more
processors, the resulting spew is unnecesserily verbose.  Instead, print the
message once, e.g:

 (XEN) ACPI: X2APIC (apic_id[0x115] uid[0x115] enabled)
 (XEN) WARNING: NR_CPUS limit of 256 reached - ignoring further processors
 (XEN) ACPI: X2APIC (apic_id[0x119] uid[0x119] enabled)
 (XEN) ACPI: X2APIC (apic_id[0x11d] uid[0x11d] enabled)
 (XEN) ACPI: X2APIC (apic_id[0x121] uid[0x121] enabled)

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>

diff --git a/xen/arch/x86/mpparse.c b/xen/arch/x86/mpparse.c
index 16c93a935d882f812ba86276fd9b25a83f0307a9..f057d9162fefaf276dd820289ad12aaabc9cce74 100644 (file)
--- a/xen/arch/x86/mpparse.c
+++ b/xen/arch/x86/mpparse.c
@@ -157,15 +157,16 @@ static int MP_processor_info_x(struct mpc_config_processor *m,
        set_apicid(apicid, &phys_cpu_present_map);
 
        if (num_processors >= nr_cpu_ids) {
-               printk(KERN_WARNING "WARNING: NR_CPUS limit of %u reached."
-                       "  Processor ignored.\n", nr_cpu_ids);
+               printk_once(XENLOG_WARNING
+                           "WARNING: NR_CPUS limit of %u reached - ignoring further processors\n",
+                           nr_cpu_ids);
                return -ENOSPC;
        }
 
        if (num_processors >= 8 && hotplug
            && genapic.name == apic_default.name) {
-               printk(KERN_WARNING "WARNING: CPUs limit of 8 reached."
-                       " Processor ignored.\n");
+               printk_once(XENLOG_WARNING
+                           "WARNING: CPUs limit of 8 reached - ignoring futher processors\n");
                return -ENOSPC;
        }