]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
lib/crypto: arm64/sha3: Migrate optimized code into library
authorEric Biggers <ebiggers@kernel.org>
Sun, 26 Oct 2025 05:50:26 +0000 (22:50 -0700)
committerEric Biggers <ebiggers@kernel.org>
Thu, 6 Nov 2025 04:02:35 +0000 (20:02 -0800)
Instead of exposing the arm64-optimized SHA-3 code via arm64-specific
crypto_shash algorithms, instead just implement the sha3_absorb_blocks()
and sha3_keccakf() library functions.  This is much simpler, it makes
the SHA-3 library functions be arm64-optimized, and it fixes the
longstanding issue where the arm64-optimized SHA-3 code was disabled by
default.  SHA-3 still remains available through crypto_shash, but
individual architectures no longer need to handle it.

Note: to see the diff from arch/arm64/crypto/sha3-ce-glue.c to
lib/crypto/arm64/sha3.h, view this commit with 'git show -M10'.

Reviewed-by: Ard Biesheuvel <ardb@kernel.org>
Link: https://lore.kernel.org/r/20251026055032.1413733-10-ebiggers@kernel.org
Signed-off-by: Eric Biggers <ebiggers@kernel.org>
arch/arm64/configs/defconfig
arch/arm64/crypto/Kconfig
arch/arm64/crypto/Makefile
arch/arm64/crypto/sha3-ce-glue.c [deleted file]
lib/crypto/Kconfig
lib/crypto/Makefile
lib/crypto/arm64/sha3-ce-core.S [moved from arch/arm64/crypto/sha3-ce-core.S with 100% similarity]
lib/crypto/arm64/sha3.h [new file with mode: 0644]

index e3a2d37bd10423b028f59dc40d6e8ee1c610d6b8..20dd3a39faead28e8f632c9d3f3073106e6ed93d 100644 (file)
@@ -1783,10 +1783,10 @@ CONFIG_CRYPTO_CHACHA20=m
 CONFIG_CRYPTO_BENCHMARK=m
 CONFIG_CRYPTO_ECHAINIV=y
 CONFIG_CRYPTO_MICHAEL_MIC=m
+CONFIG_CRYPTO_SHA3=m
 CONFIG_CRYPTO_ANSI_CPRNG=y
 CONFIG_CRYPTO_USER_API_RNG=m
 CONFIG_CRYPTO_GHASH_ARM64_CE=y
-CONFIG_CRYPTO_SHA3_ARM64=m
 CONFIG_CRYPTO_SM3_ARM64_CE=m
 CONFIG_CRYPTO_AES_ARM64_CE_BLK=y
 CONFIG_CRYPTO_AES_ARM64_BS=m
index 91f3093eee6ab51c5259c9ec1fb33c00c0a40799..376d6b50743fff170d78a37ac60b83264eef73ba 100644 (file)
@@ -25,17 +25,6 @@ config CRYPTO_NHPOLY1305_NEON
          Architecture: arm64 using:
          - NEON (Advanced SIMD) extensions
 
-config CRYPTO_SHA3_ARM64
-       tristate "Hash functions: SHA-3 (ARMv8.2 Crypto Extensions)"
-       depends on KERNEL_MODE_NEON
-       select CRYPTO_HASH
-       select CRYPTO_SHA3
-       help
-         SHA-3 secure hash algorithms (FIPS 202)
-
-         Architecture: arm64 using:
-         - ARMv8.2 Crypto Extensions
-
 config CRYPTO_SM3_NEON
        tristate "Hash functions: SM3 (NEON)"
        depends on KERNEL_MODE_NEON
index a8b2cdbe202c16befaf1ebd8a97406fea0cf9a34..fd3d590fa113797dd270a71475d83998276e16cb 100644 (file)
@@ -5,9 +5,6 @@
 # Copyright (C) 2014 Linaro Ltd <ard.biesheuvel@linaro.org>
 #
 
-obj-$(CONFIG_CRYPTO_SHA3_ARM64) += sha3-ce.o
-sha3-ce-y := sha3-ce-glue.o sha3-ce-core.o
-
 obj-$(CONFIG_CRYPTO_SM3_NEON) += sm3-neon.o
 sm3-neon-y := sm3-neon-glue.o sm3-neon-core.o
 
diff --git a/arch/arm64/crypto/sha3-ce-glue.c b/arch/arm64/crypto/sha3-ce-glue.c
deleted file mode 100644 (file)
index 250f4fb..0000000
+++ /dev/null
@@ -1,150 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0
-/*
- * sha3-ce-glue.c - core SHA-3 transform using v8.2 Crypto Extensions
- *
- * Copyright (C) 2018 Linaro Ltd <ard.biesheuvel@linaro.org>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- */
-
-#include <asm/hwcap.h>
-#include <asm/neon.h>
-#include <asm/simd.h>
-#include <crypto/internal/hash.h>
-#include <crypto/sha3.h>
-#include <linux/cpufeature.h>
-#include <linux/kernel.h>
-#include <linux/module.h>
-#include <linux/string.h>
-#include <linux/unaligned.h>
-
-MODULE_DESCRIPTION("SHA3 secure hash using ARMv8 Crypto Extensions");
-MODULE_AUTHOR("Ard Biesheuvel <ard.biesheuvel@linaro.org>");
-MODULE_LICENSE("GPL v2");
-MODULE_ALIAS_CRYPTO("sha3-224");
-MODULE_ALIAS_CRYPTO("sha3-256");
-MODULE_ALIAS_CRYPTO("sha3-384");
-MODULE_ALIAS_CRYPTO("sha3-512");
-
-asmlinkage size_t sha3_ce_transform(struct sha3_state *state, const u8 *data,
-                                   size_t nblocks, size_t block_size);
-
-static int arm64_sha3_update(struct shash_desc *desc, const u8 *data,
-                            unsigned int len)
-{
-       struct sha3_state *sctx = shash_desc_ctx(desc);
-       struct crypto_shash *tfm = desc->tfm;
-       unsigned int bs;
-       int blocks;
-
-       bs = crypto_shash_blocksize(tfm);
-       blocks = len / bs;
-       len -= blocks * bs;
-       do {
-               int rem;
-
-               kernel_neon_begin();
-               rem = sha3_ce_transform(sctx, data, blocks, bs);
-               kernel_neon_end();
-               data += (blocks - rem) * bs;
-               blocks = rem;
-       } while (blocks);
-       return len;
-}
-
-static int sha3_finup(struct shash_desc *desc, const u8 *src, unsigned int len,
-                     u8 *out)
-{
-       struct sha3_state *sctx = shash_desc_ctx(desc);
-       struct crypto_shash *tfm = desc->tfm;
-       __le64 *digest = (__le64 *)out;
-       u8 block[SHA3_224_BLOCK_SIZE];
-       unsigned int bs, ds;
-       int i;
-
-       ds = crypto_shash_digestsize(tfm);
-       bs = crypto_shash_blocksize(tfm);
-       memcpy(block, src, len);
-
-       block[len++] = 0x06;
-       memset(block + len, 0, bs - len);
-       block[bs - 1] |= 0x80;
-
-       kernel_neon_begin();
-       sha3_ce_transform(sctx, block, 1, bs);
-       kernel_neon_end();
-       memzero_explicit(block , sizeof(block));
-
-       for (i = 0; i < ds / 8; i++)
-               put_unaligned_le64(sctx->st[i], digest++);
-
-       if (ds & 4)
-               put_unaligned_le32(sctx->st[i], (__le32 *)digest);
-
-       return 0;
-}
-
-static struct shash_alg algs[] = { {
-       .digestsize             = SHA3_224_DIGEST_SIZE,
-       .init                   = crypto_sha3_init,
-       .update                 = arm64_sha3_update,
-       .finup                  = sha3_finup,
-       .descsize               = SHA3_STATE_SIZE,
-       .base.cra_name          = "sha3-224",
-       .base.cra_driver_name   = "sha3-224-ce",
-       .base.cra_flags         = CRYPTO_AHASH_ALG_BLOCK_ONLY,
-       .base.cra_blocksize     = SHA3_224_BLOCK_SIZE,
-       .base.cra_module        = THIS_MODULE,
-       .base.cra_priority      = 200,
-}, {
-       .digestsize             = SHA3_256_DIGEST_SIZE,
-       .init                   = crypto_sha3_init,
-       .update                 = arm64_sha3_update,
-       .finup                  = sha3_finup,
-       .descsize               = SHA3_STATE_SIZE,
-       .base.cra_name          = "sha3-256",
-       .base.cra_driver_name   = "sha3-256-ce",
-       .base.cra_flags         = CRYPTO_AHASH_ALG_BLOCK_ONLY,
-       .base.cra_blocksize     = SHA3_256_BLOCK_SIZE,
-       .base.cra_module        = THIS_MODULE,
-       .base.cra_priority      = 200,
-}, {
-       .digestsize             = SHA3_384_DIGEST_SIZE,
-       .init                   = crypto_sha3_init,
-       .update                 = arm64_sha3_update,
-       .finup                  = sha3_finup,
-       .descsize               = SHA3_STATE_SIZE,
-       .base.cra_name          = "sha3-384",
-       .base.cra_driver_name   = "sha3-384-ce",
-       .base.cra_flags         = CRYPTO_AHASH_ALG_BLOCK_ONLY,
-       .base.cra_blocksize     = SHA3_384_BLOCK_SIZE,
-       .base.cra_module        = THIS_MODULE,
-       .base.cra_priority      = 200,
-}, {
-       .digestsize             = SHA3_512_DIGEST_SIZE,
-       .init                   = crypto_sha3_init,
-       .update                 = arm64_sha3_update,
-       .finup                  = sha3_finup,
-       .descsize               = SHA3_STATE_SIZE,
-       .base.cra_name          = "sha3-512",
-       .base.cra_driver_name   = "sha3-512-ce",
-       .base.cra_flags         = CRYPTO_AHASH_ALG_BLOCK_ONLY,
-       .base.cra_blocksize     = SHA3_512_BLOCK_SIZE,
-       .base.cra_module        = THIS_MODULE,
-       .base.cra_priority      = 200,
-} };
-
-static int __init sha3_neon_mod_init(void)
-{
-       return crypto_register_shashes(algs, ARRAY_SIZE(algs));
-}
-
-static void __exit sha3_neon_mod_fini(void)
-{
-       crypto_unregister_shashes(algs, ARRAY_SIZE(algs));
-}
-
-module_cpu_feature_match(SHA3, sha3_neon_mod_init);
-module_exit(sha3_neon_mod_fini);
index a05f5a349cd8cebf38b2675a62e045077e8279e5..587490ca65654940c85370f57dae1a7577a5c9ae 100644 (file)
@@ -202,6 +202,11 @@ config CRYPTO_LIB_SHA3
          The SHA3 library functions.  Select this if your module uses any of
          the functions from <crypto/sha3.h>.
 
+config CRYPTO_LIB_SHA3_ARCH
+       bool
+       depends on CRYPTO_LIB_SHA3 && !UML
+       default y if ARM64 && KERNEL_MODE_NEON
+
 config CRYPTO_LIB_SM3
        tristate
 
index 0cfdb511f32b6d85f5327525fab05bb09d7cd907..5515e73bfd5e33ba90a44c32648fe130c6785c19 100644 (file)
@@ -281,6 +281,11 @@ endif # CONFIG_CRYPTO_LIB_SHA512_ARCH
 obj-$(CONFIG_CRYPTO_LIB_SHA3) += libsha3.o
 libsha3-y := sha3.o
 
+ifeq ($(CONFIG_CRYPTO_LIB_SHA3_ARCH),y)
+CFLAGS_sha3.o += -I$(src)/$(SRCARCH)
+libsha3-$(CONFIG_ARM64) += arm64/sha3-ce-core.o
+endif # CONFIG_CRYPTO_LIB_SHA3_ARCH
+
 ################################################################################
 
 obj-$(CONFIG_MPILIB) += mpi/
diff --git a/lib/crypto/arm64/sha3.h b/lib/crypto/arm64/sha3.h
new file mode 100644 (file)
index 0000000..6dd5183
--- /dev/null
@@ -0,0 +1,62 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Copyright (C) 2018 Linaro Ltd <ard.biesheuvel@linaro.org>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+#include <asm/neon.h>
+#include <asm/simd.h>
+#include <linux/cpufeature.h>
+
+static __ro_after_init DEFINE_STATIC_KEY_FALSE(have_sha3);
+
+asmlinkage size_t sha3_ce_transform(struct sha3_state *state, const u8 *data,
+                                   size_t nblocks, size_t block_size);
+
+static void sha3_absorb_blocks(struct sha3_state *state, const u8 *data,
+                              size_t nblocks, size_t block_size)
+{
+       if (static_branch_likely(&have_sha3) && likely(may_use_simd())) {
+               do {
+                       size_t rem;
+
+                       kernel_neon_begin();
+                       rem = sha3_ce_transform(state, data, nblocks,
+                                               block_size);
+                       kernel_neon_end();
+                       data += (nblocks - rem) * block_size;
+                       nblocks = rem;
+               } while (nblocks);
+       } else {
+               sha3_absorb_blocks_generic(state, data, nblocks, block_size);
+       }
+}
+
+static void sha3_keccakf(struct sha3_state *state)
+{
+       if (static_branch_likely(&have_sha3) && likely(may_use_simd())) {
+               /*
+                * Passing zeroes into sha3_ce_transform() gives the plain
+                * Keccak-f permutation, which is what we want here.  Any
+                * supported block size may be used.  Use SHA3_512_BLOCK_SIZE
+                * since it's the shortest.
+                */
+               static const u8 zeroes[SHA3_512_BLOCK_SIZE];
+
+               kernel_neon_begin();
+               sha3_ce_transform(state, zeroes, 1, sizeof(zeroes));
+               kernel_neon_end();
+       } else {
+               sha3_keccakf_generic(state);
+       }
+}
+
+#define sha3_mod_init_arch sha3_mod_init_arch
+static void sha3_mod_init_arch(void)
+{
+       if (cpu_have_named_feature(SHA3))
+               static_branch_enable(&have_sha3);
+}