Re: [PATCH] machine_power_off: not only local_irq_disable but also do disable preemption

From: Russell King - ARM Linux
Date: Sat Jul 05 2014 - 15:26:47 EST


On Sun, Jul 06, 2014 at 12:34:14AM +0530, pawandeep oza wrote:
> no this is not a generic code bug, because only in this scenerio, problem
> might happen.
> because other core is plugged out and now there is no chance that it can
> release the spin_lock it would have held previously.
>
> so this core must run with preemption and interrupt disabled.
>
> this is only applicable for shutdown, not for any other scenerio.

No. See my reply I just sent. Preempting when local interrupts
disabled is a bug.

--
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.
--
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/