Thread (2 messages) 2 messages, 2 authors, 2025-07-14

Re: [PATCH v2] lib/crypto: arm/poly1305: Remove unneeded empty weak function

From: Ard Biesheuvel <ardb@kernel.org>
Date: 2025-07-14 05:37:09
Also in: linux-crypto, lkml

On Sat, 12 Jul 2025 at 07:28, Eric Biggers [off-list ref] wrote:
Fix poly1305-armv4.pl to not do '.globl poly1305_blocks_neon' when
poly1305_blocks_neon() is not defined.  Then, remove the empty __weak
definition of poly1305_blocks_neon(), which was still needed only
because of that unnecessary globl statement.  (It also used to be needed
because the compiler could generate calls to it when
CONFIG_KERNEL_MODE_NEON=n, but that has been fixed.)

Thanks to Arnd Bergmann for reporting that the globl statement in the
asm file was still depending on the weak symbol.

Signed-off-by: Eric Biggers <ebiggers@kernel.org>
---
 lib/crypto/arm/poly1305-armv4.pl | 2 +-
 lib/crypto/arm/poly1305-glue.c   | 5 -----
 2 files changed, 1 insertion(+), 6 deletions(-)
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help