On 05/17/2016 12:21 AM, Chen-Yu Tsai wrote:
Hi,
On Tue, May 17, 2016 at 10:38 AM, tpearson@raptorengineering.com wrote:
arch/arm/cpu/armv7/psci.S | 1 + arch/arm/cpu/armv7/sunxi/psci_sun6i.S | 4 ++-- arch/arm/cpu/armv7/sunxi/psci_sun7i.S | 4 ++-- 3 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/arch/arm/cpu/armv7/psci.S b/arch/arm/cpu/armv7/psci.S index 87c0c0b..18a0cc2 100644 --- a/arch/arm/cpu/armv7/psci.S +++ b/arch/arm/cpu/armv7/psci.S @@ -203,6 +203,7 @@ ENTRY(psci_get_cpu_stack_top) add r5, r5, #0x2000 @ Skip two pages lsr r5, r5, #12 @ Align to start of page
Why not just change this line instead?
ChenYu
That would break page alignment, no? The idea is to align the lower stack range address to a page (possibly wasting up to slightly less than a page in the process); changing this line would result in two pages wasted and the same amount of space available for the stack.