From 7ddd4475a8f7cabfa86c7f61452cc49facbe172a Mon Sep 17 00:00:00 2001 From: Wolfgang Denk Date: Fri, 23 Dec 2011 01:29:12 +0000 Subject: [PATCH] post/lib_powerpc/multi.c: fix stack overflow error The code and comment disagreed: the comment claimed that r6...r31 were copied, and consequently the arrays for "src" and "dst" were declared with 26 entries, but the actual code ("lmw r5,0(r3)" and "stmw r5,0(r4)") copied _27_ words (r5 through r31), which resulted in false "POST cpu Error at multi test" messages. Fix the comment and the array sizes. Signed-off-by: Wolfgang Denk Cc: Anatolij Gustschin Cc: Stefan Roese Cc: Kumar Gala Cc: Kim Phillips Cc: Andy Fleming Acked-by: Anatolij Gustschin Tested-by: Anatolij Gustschin --- post/lib_powerpc/multi.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/post/lib_powerpc/multi.c b/post/lib_powerpc/multi.c index f9fe719da8..e9814a6b07 100644 --- a/post/lib_powerpc/multi.c +++ b/post/lib_powerpc/multi.c @@ -27,9 +27,9 @@ * CPU test * Load/store multiple word instructions: lmw, stmw * - * 26 consecutive words are loaded from a source memory buffer - * into GPRs r6 through r31. After that, 26 consecutive words are stored - * from the GPRs r6 through r31 into a target memory buffer. The contents + * 27 consecutive words are loaded from a source memory buffer + * into GPRs r5 through r31. After that, 27 consecutive words are stored + * from the GPRs r5 through r31 into a target memory buffer. The contents * of the source and target buffers are then compared. */ @@ -44,7 +44,7 @@ int cpu_post_test_multi(void) { int ret = 0; unsigned int i; - ulong src[26], dst[26]; + ulong src[27], dst[27]; int flag = disable_interrupts(); ulong code[] = { -- 2.39.5