Re: [PATCH] x86, boot: Further compress CPUs bootup message

From: Ingo Molnar
Date: Sat Sep 28 2013 - 13:54:57 EST



* Borislav Petkov <bp@xxxxxxxxx> wrote:

> +void smp_announce(void)
> +{
> + int num_nodes = num_online_nodes();
> +
> + printk(KERN_INFO "x86: Booted up %d node%s, %ld CPUs\n",
> + num_nodes, (num_nodes > 1 ? "s" : ""), (long)num_online_cpus());
> +}

That (long) cast looks weird - num_online_cpus() is 'unsigned int' so
changing the format to %d should do the trick, right?

> +void __weak smp_announce(void)
> +{
> + printk(KERN_INFO "Brought up %ld CPUs\n", (long)num_online_cpus());
> +}

Ditto.

Thanks,

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/