feat(psci): make pabandon support generic
Support for aborted powerdowns does not require much dedicated code.
Rather, it is largely a matter of orchestrating things to happen in the
right order.
The only exception to this are older secure world dispatchers, which
assume that a CPU_SUSPEND call will be terminal and therefore can
clobber context. This was patched over in common code and hidden behind
a flag. This patch moves this to the dispatchers themselves.
Dispatchers that don't register svc_suspend{_finish} are unaffected.
Those that do must save the NS context before clobbering it and
restoring in only in case of a pabandon. Due to this operation being
non-trivial, this patch makes the assumption that these dispatchers will
only be present on hardware that does not support pabandon and therefore
does not add any contexting for them. In case this assumption ever
changes, asserts are added that should alert us of this change.
Change-Id: I94a907515b782b4d2136c0d274246cfe1d567c0e
Signed-off-by: Boyan Karatotev <boyan.karatotev@arm.com>
diff --git a/lib/psci/psci_common.c b/lib/psci/psci_common.c
index 15a32f1..04917ea 100644
--- a/lib/psci/psci_common.c
+++ b/lib/psci/psci_common.c
@@ -1056,7 +1056,7 @@
unsigned int max_off_lvl = psci_find_max_off_lvl(&state_info);
assert(max_off_lvl != PSCI_INVALID_PWR_LVL);
- psci_cpu_suspend_to_powerdown_finish(cpu_idx, max_off_lvl, &state_info);
+ psci_cpu_suspend_to_powerdown_finish(cpu_idx, max_off_lvl, &state_info, false);
}
/*
@@ -1311,15 +1311,13 @@
/*
* Waking up does not require hardware-assisted coherency, but that is
- * the case for every core that can wake up. Untangling the cache
- * coherency code from powerdown is a non-trivial effort which isn't
- * needed for our purposes.
+ * the case for every core that can wake up. Can either happen because
+ * of errata or pabandon.
*/
-#if !FEAT_PABANDON || !defined(__aarch64__)
- ERROR("Systems without FEAT_PABANDON shouldn't wake up.\n");
+#if !defined(__aarch64__) || !HW_ASSISTED_COHERENCY
+ ERROR("AArch32 systems shouldn't wake up.\n");
panic();
-#else /* FEAT_PABANDON */
-
+#endif
/*
* Begin unwinding. Everything can be shared with CPU_ON and co later,
* except the CPU specific bit. Cores that have hardware-assisted
@@ -1327,7 +1325,6 @@
* the simplest way to achieve this
*/
prepare_cpu_pwr_dwn(power_level);
-#endif /* FEAT_PABANDON */
}
/*******************************************************************************