lib/crypto: arm/poly1305: Remove unneeded empty weak function
Subsystems (2)
Eric Biggers
maintainerPATCH v2 1/1 lib/crypto: arm/poly1305: Remove unneeded empty weak function
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.
--- lib/crypto/arm/poly1305-armv4.pl | 2 +- lib/crypto/arm/poly1305-glue.c | 5 ----- 2 files changed, 1 insertion(+), 6 deletions(-)
lib/crypto/arm/poly1305-armv4.pl +1 -1
@@ -44,11 +44,10 @@ $code.=<<___; # define __ARM_ARCH__ __LINUX_ARM_ARCH__ # define __ARM_MAX_ARCH__ __LINUX_ARM_ARCH__ # define poly1305_init poly1305_block_init_arch # define poly1305_blocks poly1305_blocks_arm # define poly1305_emit poly1305_emit_arch -.globl poly1305_blocks_neon #endif #if defined(__thumb2__) .syntax unified .thumb
@@ -720,10 +719,11 @@ poly1305_init_neon: .Lno_init_neon: ret @ bx lr .size poly1305_init_neon,.-poly1305_init_neon +.globl poly1305_blocks_neon .type poly1305_blocks_neon,%function .align 5 poly1305_blocks_neon: .Lpoly1305_blocks_neon: ldr ip,[$ctx,#36] @ is_base2_26
lib/crypto/arm/poly1305-glue.c +0 -5
@@ -25,15 +25,10 @@ asmlinkage void poly1305_blocks_neon(struct poly1305_block_state *state, asmlinkage void poly1305_emit_arch(const struct poly1305_state *state,u8 digest[POLY1305_DIGEST_SIZE ],const u32 nonce[4 ]);EXPORT_SYMBOL_GPL (poly1305_emit_arch); -void __weak poly1305_blocks_neon (struct poly1305_block_state *state, -const u8 *src,u32 len,u32 hibit) -{ -} -static __ro_after_init DEFINE_STATIC_KEY_FALSE (have_neon);void poly1305_blocks_arch (struct poly1305_block_state *state,const u8 *src,unsigned int len,u32 padbit) {
base-commit: 57b15e9260a31438e91cf83dbfcb63333b24c684
Signature · 1 line
2.50.1
Quoted 8 lines · Eric Biggers
Fix poly1305-armv4.pl to not do '.globl
poly1305_blocks_neon' whenpoly1305_blocks_neon()is not defined. Then, remove the empty__weakdefinition ofpoly1305_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 whenCONFIG_KERNEL_MODE_NEON=n, but that has been fixed.)
Quoted 9 lines
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 <[email protected]>--- lib/crypto/arm/poly1305-armv4.pl | 2 +- lib/crypto/arm/poly1305-glue.c | 5 ----- 2 files changed, 1 insertion(+), 6 deletions(-)