Commit e6037a75 authored by Jason A. Donenfeld's avatar Jason A. Donenfeld Committed by Linus Torvalds

powerpc32: don't adjust unmoved stack pointer in csum_partial_copy_generic() epilogue

A recent change to the checksum code removed usage of some extra
arguments, alongside with storage on the stack for those, and the stack
pointer no longer needed to be adjusted in the function prologue.

But a left over subtraction wasn't removed in the function epilogue,
causing the function to return with the stack pointer moved 16 bytes
away from where it should have.  This corrupted local state and lead to
weird crashes.

This simply removes the leftover instruction from the epilogue.

Fixes: 70d65cd5 ("ppc: propagate the calling conventions change down to csum_partial_copy_generic()")
Cc: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: default avatarJason A. Donenfeld <Jason@zx2c4.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 6448cbf6
...@@ -236,7 +236,6 @@ _GLOBAL(csum_partial_copy_generic) ...@@ -236,7 +236,6 @@ _GLOBAL(csum_partial_copy_generic)
slwi r0,r0,8 slwi r0,r0,8
adde r12,r12,r0 adde r12,r12,r0
66: addze r3,r12 66: addze r3,r12
addi r1,r1,16
beqlr+ cr7 beqlr+ cr7
rlwinm r3,r3,8,0,31 /* odd destination address: rotate one byte */ rlwinm r3,r3,8,0,31 /* odd destination address: rotate one byte */
blr blr
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment