On Thu, Dec 12, 2024 at 04:02:41PM +0100, Greg Kroah-Hartman wrote: > 5.4-stable review patch. If anyone has any objections, please let me know. > > ------------------ > > From: Jiri Slaby <jslaby@xxxxxxx> > > [ Upstream commit 74d8b90a889022e306b543ff2147a6941c99b354 ] > > Use the newly added SYM_FUNC_START_LOCAL to annotate beginnings of all > functions which do not have ".globl" annotation, but their endings are > annotated by ENDPROC. This is needed to balance ENDPROC for tools that > generate debuginfo. > > These function names are not prepended with ".L" as they might appear in > call traces and they wouldn't be visible after such change. > > To be symmetric, the functions' ENDPROCs are converted to the new > SYM_FUNC_END. > > Signed-off-by: Jiri Slaby <jslaby@xxxxxxx> > Signed-off-by: Borislav Petkov <bp@xxxxxxx> > Cc: "David S. Miller" <davem@xxxxxxxxxxxxx> > Cc: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> > Cc: "H. Peter Anvin" <hpa@xxxxxxxxx> > Cc: Ingo Molnar <mingo@xxxxxxxxxx> > Cc: linux-arch@xxxxxxxxxxxxxxx > Cc: linux-crypto@xxxxxxxxxxxxxxx > Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > Cc: x86-ml <x86@xxxxxxxxxx> > Link: https://lkml.kernel.org/r/20191011115108.12392-7-jslaby@xxxxxxx > Stable-dep-of: 3b2f2d22fb42 ("crypto: x86/aegis128 - access 32-bit arguments as 32-bit") > Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx> > --- > arch/x86/crypto/aegis128-aesni-asm.S | 8 ++-- > arch/x86/crypto/aesni-intel_asm.S | 49 ++++++++------------ > arch/x86/crypto/camellia-aesni-avx-asm_64.S | 20 ++++---- > arch/x86/crypto/camellia-aesni-avx2-asm_64.S | 20 ++++---- > arch/x86/crypto/cast5-avx-x86_64-asm_64.S | 8 ++-- > arch/x86/crypto/cast6-avx-x86_64-asm_64.S | 8 ++-- > arch/x86/crypto/chacha-ssse3-x86_64.S | 4 +- > arch/x86/crypto/ghash-clmulni-intel_asm.S | 4 +- > arch/x86/crypto/serpent-avx-x86_64-asm_64.S | 8 ++-- > arch/x86/crypto/serpent-avx2-asm_64.S | 8 ++-- > arch/x86/crypto/twofish-avx-x86_64-asm_64.S | 8 ++-- > 11 files changed, 68 insertions(+), 77 deletions(-) Unless the author of this patch acks this I'd rather you skipped this. It's not worth the risk of regressions in the crypto code. - Eric
