PSCI: Set ON_PENDING state early during CPU_ON
authorSoby Mathew <soby.mathew@arm.com>
Mon, 11 May 2015 22:15:06 +0000 (23:15 +0100)
committerSoby Mathew <soby.mathew@arm.com>
Wed, 13 May 2015 09:52:02 +0000 (10:52 +0100)
commit42cae5a166c557d0c5809f39d22a351ca58bf1c3
tree8f90e49e60a7934b58461eb520b8a31f406a4565
parentaaa48a86b9bd74ca9621afffe6cd8e9444b5c235
PSCI: Set ON_PENDING state early during CPU_ON

In the debug build of the function get_power_on_target_afflvl(), there is a
check to ensure that the CPU is emerging from a SUSPEND or ON_PENDING state.
The state is checked without acquiring the lock for the CPU node. The state
could be updated to ON_PENDING in psci_afflvl_on() after the target CPU has
been powered up. This results in a race condition which could cause the
check for the ON_PENDING state in get_power_on_target_afflvl() to fail.
This patch resolves this race condition by setting the state of the target
CPU to ON_PENDING before the platform port attempts to power it on. The
target CPU is thus guaranteed to read the correct the state. In case
the power on operation fails, the state of the CPU is restored to OFF.

Fixes ARM-software/tf-issues#302

Change-Id: I3f2306a78c58d47b1a0fb7e33ab04f917a2d5044
services/std_svc/psci/psci_afflvl_on.c