Re: [RESEND PATCH 2/4] ARM: msm: Re-organize platsmp to make itextensible

From: David Rientjes
Date: Tue Aug 20 2013 - 02:59:54 EST


On Mon, 12 Aug 2013, Mark Rutland wrote:

> > static void __init msm_smp_prepare_cpus(unsigned int max_cpus)
> > {
> > + int cpu, map;
> > + unsigned int flags = 0;
> > +
> > + for_each_present_cpu(cpu) {
> > + map = cpu_logical_map(cpu);
> > + if (map > ARRAY_SIZE(cold_boot_flags)) {
> > + set_cpu_present(cpu, false);
> > + __WARN();
> > + continue;
> > + }
> > + flags |= cold_boot_flags[map];

__WARN() can't be used in generic code because it's possible to have
CONFIG_BUG=n, you probably want something like WARN_ON(1) instead.
--
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/