Re: [PATCH v3 4/5] msm: hotplug: support cpu hotplug on msm

[Catalin Marinas <catalin.marinas@xxxxxxx>]

On 13 December 2010 01:20,  <ykaoua@xxxxxxxxxxxx> wrote:
> --- /dev/null
> +++ b/arch/arm/mach-msm/hotplug.c
[...]
> +static inline void platform_do_lowpower(unsigned int cpu)
> +{
> + Â Â Â /* Just enter wfe for now. */
> + Â Â Â for (;;) {
> + Â Â Â Â Â Â Â asm("wfe");
> + Â Â Â Â Â Â Â if (pen_release == cpu) {
> + Â Â Â Â Â Â Â Â Â Â Â /*
> + Â Â Â Â Â Â Â Â Â Â Â Â* OK, proper wakeup, we're done
> + Â Â Â Â Â Â Â Â Â Â Â Â*/
> + Â Â Â Â Â Â Â Â Â Â Â break;
> + Â Â Â Â Â Â Â }
> + Â Â Â Â Â Â Â smp_rmb();

Do you need some barrier after wfe?

-- 
Catalin
ÿô.nlj·Ÿ®‰­†+%ŠË±é¥Šwÿº{.nlj·¥Š{±ýþ)íèjg¬±¨¶‰šŽŠÝjÿ¾«þG«é¸¢·¦j:+v‰¨Šwèm¶Ÿÿþø®w¥þŠà£¢·hšâÿ†Ù