crypto: sparc/sha256 - implement library instead of shash
authorEric Biggers <ebiggers@google.com>
Mon, 28 Apr 2025 17:00:35 +0000 (10:00 -0700)
committerHerbert Xu <herbert@gondor.apana.org.au>
Mon, 5 May 2025 10:20:44 +0000 (18:20 +0800)
Instead of providing crypto_shash algorithms for the arch-optimized
SHA-256 code, instead implement the SHA-256 library.  This is much
simpler, it makes the SHA-256 library functions be arch-optimized, and
it fixes the longstanding issue where the arch-optimized SHA-256 was
disabled by default.  SHA-256 still remains available through
crypto_shash, but individual architectures no longer need to handle it.

Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
arch/sparc/crypto/Kconfig
arch/sparc/crypto/Makefile
arch/sparc/crypto/sha256_asm.S [deleted file]
arch/sparc/crypto/sha256_glue.c [deleted file]
arch/sparc/lib/Makefile
arch/sparc/lib/crypto/Kconfig [new file with mode: 0644]
arch/sparc/lib/crypto/Makefile [new file with mode: 0644]
arch/sparc/lib/crypto/sha256.c [new file with mode: 0644]
arch/sparc/lib/crypto/sha256_asm.S [new file with mode: 0644]
lib/crypto/Kconfig

index e858597de89dbbef411f20555bd9a7bfffc7a117..a6ba319c42dce430a7ce052bc8b22dc18f463f1e 100644 (file)
@@ -36,16 +36,6 @@ config CRYPTO_SHA1_SPARC64
 
          Architecture: sparc64
 
-config CRYPTO_SHA256_SPARC64
-       tristate "Hash functions: SHA-224 and SHA-256"
-       depends on SPARC64
-       select CRYPTO_SHA256
-       select CRYPTO_HASH
-       help
-         SHA-224 and SHA-256 secure hash algorithms (FIPS 180)
-
-         Architecture: sparc64 using crypto instructions, when available
-
 config CRYPTO_SHA512_SPARC64
        tristate "Hash functions: SHA-384 and SHA-512"
        depends on SPARC64
index a2d7fca40cb4b5250ddc76505a07b28c44d1f445..701c39edb0d733b472054b49a74d4b3fbed5672b 100644 (file)
@@ -4,7 +4,6 @@
 #
 
 obj-$(CONFIG_CRYPTO_SHA1_SPARC64) += sha1-sparc64.o
-obj-$(CONFIG_CRYPTO_SHA256_SPARC64) += sha256-sparc64.o
 obj-$(CONFIG_CRYPTO_SHA512_SPARC64) += sha512-sparc64.o
 obj-$(CONFIG_CRYPTO_MD5_SPARC64) += md5-sparc64.o
 
@@ -13,7 +12,6 @@ obj-$(CONFIG_CRYPTO_DES_SPARC64) += des-sparc64.o
 obj-$(CONFIG_CRYPTO_CAMELLIA_SPARC64) += camellia-sparc64.o
 
 sha1-sparc64-y := sha1_asm.o sha1_glue.o
-sha256-sparc64-y := sha256_asm.o sha256_glue.o
 sha512-sparc64-y := sha512_asm.o sha512_glue.o
 md5-sparc64-y := md5_asm.o md5_glue.o
 
diff --git a/arch/sparc/crypto/sha256_asm.S b/arch/sparc/crypto/sha256_asm.S
deleted file mode 100644 (file)
index 8ce8861..0000000
+++ /dev/null
@@ -1,78 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0 */
-#include <linux/linkage.h>
-#include <asm/opcodes.h>
-#include <asm/visasm.h>
-
-ENTRY(sha256_sparc64_transform)
-       /* %o0 = digest, %o1 = data, %o2 = rounds */
-       VISEntryHalf
-       ld      [%o0 + 0x00], %f0
-       ld      [%o0 + 0x04], %f1
-       ld      [%o0 + 0x08], %f2
-       ld      [%o0 + 0x0c], %f3
-       ld      [%o0 + 0x10], %f4
-       ld      [%o0 + 0x14], %f5
-       andcc   %o1, 0x7, %g0
-       ld      [%o0 + 0x18], %f6
-       bne,pn  %xcc, 10f
-        ld     [%o0 + 0x1c], %f7
-
-1:
-       ldd     [%o1 + 0x00], %f8
-       ldd     [%o1 + 0x08], %f10
-       ldd     [%o1 + 0x10], %f12
-       ldd     [%o1 + 0x18], %f14
-       ldd     [%o1 + 0x20], %f16
-       ldd     [%o1 + 0x28], %f18
-       ldd     [%o1 + 0x30], %f20
-       ldd     [%o1 + 0x38], %f22
-
-       SHA256
-
-       subcc   %o2, 1, %o2
-       bne,pt  %xcc, 1b
-        add    %o1, 0x40, %o1
-
-5:
-       st      %f0, [%o0 + 0x00]
-       st      %f1, [%o0 + 0x04]
-       st      %f2, [%o0 + 0x08]
-       st      %f3, [%o0 + 0x0c]
-       st      %f4, [%o0 + 0x10]
-       st      %f5, [%o0 + 0x14]
-       st      %f6, [%o0 + 0x18]
-       st      %f7, [%o0 + 0x1c]
-       retl
-        VISExitHalf
-10:
-       alignaddr %o1, %g0, %o1
-
-       ldd     [%o1 + 0x00], %f10
-1:
-       ldd     [%o1 + 0x08], %f12
-       ldd     [%o1 + 0x10], %f14
-       ldd     [%o1 + 0x18], %f16
-       ldd     [%o1 + 0x20], %f18
-       ldd     [%o1 + 0x28], %f20
-       ldd     [%o1 + 0x30], %f22
-       ldd     [%o1 + 0x38], %f24
-       ldd     [%o1 + 0x40], %f26
-
-       faligndata %f10, %f12, %f8
-       faligndata %f12, %f14, %f10
-       faligndata %f14, %f16, %f12
-       faligndata %f16, %f18, %f14
-       faligndata %f18, %f20, %f16
-       faligndata %f20, %f22, %f18
-       faligndata %f22, %f24, %f20
-       faligndata %f24, %f26, %f22
-
-       SHA256
-
-       subcc   %o2, 1, %o2
-       fsrc2   %f26, %f10
-       bne,pt  %xcc, 1b
-        add    %o1, 0x40, %o1
-
-       ba,a,pt %xcc, 5b
-ENDPROC(sha256_sparc64_transform)
diff --git a/arch/sparc/crypto/sha256_glue.c b/arch/sparc/crypto/sha256_glue.c
deleted file mode 100644 (file)
index 2500860..0000000
+++ /dev/null
@@ -1,128 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0-only
-/* Glue code for SHA256 hashing optimized for sparc64 crypto opcodes.
- *
- * This is based largely upon crypto/sha256_generic.c
- *
- * Copyright (c) Jean-Luc Cooke <jlcooke@certainkey.com>
- * Copyright (c) Andrew McDonald <andrew@mcdonald.org.uk>
- * Copyright (c) 2002 James Morris <jmorris@intercode.com.au>
- * SHA224 Support Copyright 2007 Intel Corporation <jonathan.lynch@intel.com>
- */
-
-#define pr_fmt(fmt)    KBUILD_MODNAME ": " fmt
-
-#include <asm/elf.h>
-#include <asm/opcodes.h>
-#include <asm/pstate.h>
-#include <crypto/internal/hash.h>
-#include <crypto/sha2.h>
-#include <crypto/sha256_base.h>
-#include <linux/kernel.h>
-#include <linux/module.h>
-
-asmlinkage void sha256_sparc64_transform(u32 *digest, const char *data,
-                                        unsigned int rounds);
-
-static void sha256_block(struct crypto_sha256_state *sctx, const u8 *src,
-                        int blocks)
-{
-       sha256_sparc64_transform(sctx->state, src, blocks);
-}
-
-static int sha256_sparc64_update(struct shash_desc *desc, const u8 *data,
-                                unsigned int len)
-{
-       return sha256_base_do_update_blocks(desc, data, len, sha256_block);
-}
-
-static int sha256_sparc64_finup(struct shash_desc *desc, const u8 *src,
-                               unsigned int len, u8 *out)
-{
-       sha256_base_do_finup(desc, src, len, sha256_block);
-       return sha256_base_finish(desc, out);
-}
-
-static struct shash_alg sha256_alg = {
-       .digestsize     =       SHA256_DIGEST_SIZE,
-       .init           =       sha256_base_init,
-       .update         =       sha256_sparc64_update,
-       .finup          =       sha256_sparc64_finup,
-       .descsize       =       sizeof(struct crypto_sha256_state),
-       .base           =       {
-               .cra_name       =       "sha256",
-               .cra_driver_name=       "sha256-sparc64",
-               .cra_priority   =       SPARC_CR_OPCODE_PRIORITY,
-               .cra_flags      =       CRYPTO_AHASH_ALG_BLOCK_ONLY |
-                                       CRYPTO_AHASH_ALG_FINUP_MAX,
-               .cra_blocksize  =       SHA256_BLOCK_SIZE,
-               .cra_module     =       THIS_MODULE,
-       }
-};
-
-static struct shash_alg sha224_alg = {
-       .digestsize     =       SHA224_DIGEST_SIZE,
-       .init           =       sha224_base_init,
-       .update         =       sha256_sparc64_update,
-       .finup          =       sha256_sparc64_finup,
-       .descsize       =       sizeof(struct crypto_sha256_state),
-       .base           =       {
-               .cra_name       =       "sha224",
-               .cra_driver_name=       "sha224-sparc64",
-               .cra_priority   =       SPARC_CR_OPCODE_PRIORITY,
-               .cra_flags      =       CRYPTO_AHASH_ALG_BLOCK_ONLY |
-                                       CRYPTO_AHASH_ALG_FINUP_MAX,
-               .cra_blocksize  =       SHA224_BLOCK_SIZE,
-               .cra_module     =       THIS_MODULE,
-       }
-};
-
-static bool __init sparc64_has_sha256_opcode(void)
-{
-       unsigned long cfr;
-
-       if (!(sparc64_elf_hwcap & HWCAP_SPARC_CRYPTO))
-               return false;
-
-       __asm__ __volatile__("rd %%asr26, %0" : "=r" (cfr));
-       if (!(cfr & CFR_SHA256))
-               return false;
-
-       return true;
-}
-
-static int __init sha256_sparc64_mod_init(void)
-{
-       if (sparc64_has_sha256_opcode()) {
-               int ret = crypto_register_shash(&sha224_alg);
-               if (ret < 0)
-                       return ret;
-
-               ret = crypto_register_shash(&sha256_alg);
-               if (ret < 0) {
-                       crypto_unregister_shash(&sha224_alg);
-                       return ret;
-               }
-
-               pr_info("Using sparc64 sha256 opcode optimized SHA-256/SHA-224 implementation\n");
-               return 0;
-       }
-       pr_info("sparc64 sha256 opcode not available.\n");
-       return -ENODEV;
-}
-
-static void __exit sha256_sparc64_mod_fini(void)
-{
-       crypto_unregister_shash(&sha224_alg);
-       crypto_unregister_shash(&sha256_alg);
-}
-
-module_init(sha256_sparc64_mod_init);
-module_exit(sha256_sparc64_mod_fini);
-
-MODULE_LICENSE("GPL");
-MODULE_DESCRIPTION("SHA-224 and SHA-256 Secure Hash Algorithm, sparc64 sha256 opcode accelerated");
-
-MODULE_ALIAS_CRYPTO("sha224");
-MODULE_ALIAS_CRYPTO("sha256");
-
-#include "crop_devid.c"
index 5724d0f356eb5efa0a8869c97aec0d294d7ff703..98887dc295a1eb4728d2a396bfaa79e05c669254 100644 (file)
@@ -4,6 +4,7 @@
 
 asflags-y := -ansi -DST_DIV0=0x02
 
+obj-y                 += crypto/
 lib-$(CONFIG_SPARC32) += ashrdi3.o
 lib-$(CONFIG_SPARC32) += memcpy.o memset.o
 lib-y                 += strlen.o
diff --git a/arch/sparc/lib/crypto/Kconfig b/arch/sparc/lib/crypto/Kconfig
new file mode 100644 (file)
index 0000000..e5c3e4d
--- /dev/null
@@ -0,0 +1,8 @@
+# SPDX-License-Identifier: GPL-2.0-only
+
+config CRYPTO_SHA256_SPARC64
+       tristate
+       depends on SPARC64
+       default CRYPTO_LIB_SHA256
+       select CRYPTO_ARCH_HAVE_LIB_SHA256
+       select CRYPTO_LIB_SHA256_GENERIC
diff --git a/arch/sparc/lib/crypto/Makefile b/arch/sparc/lib/crypto/Makefile
new file mode 100644 (file)
index 0000000..75ee244
--- /dev/null
@@ -0,0 +1,4 @@
+# SPDX-License-Identifier: GPL-2.0-only
+
+obj-$(CONFIG_CRYPTO_SHA256_SPARC64) += sha256-sparc64.o
+sha256-sparc64-y := sha256.o sha256_asm.o
diff --git a/arch/sparc/lib/crypto/sha256.c b/arch/sparc/lib/crypto/sha256.c
new file mode 100644 (file)
index 0000000..6f118a2
--- /dev/null
@@ -0,0 +1,64 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * SHA-256 accelerated using the sparc64 sha256 opcodes
+ *
+ * Copyright (c) Jean-Luc Cooke <jlcooke@certainkey.com>
+ * Copyright (c) Andrew McDonald <andrew@mcdonald.org.uk>
+ * Copyright (c) 2002 James Morris <jmorris@intercode.com.au>
+ * SHA224 Support Copyright 2007 Intel Corporation <jonathan.lynch@intel.com>
+ */
+
+#define pr_fmt(fmt)    KBUILD_MODNAME ": " fmt
+
+#include <asm/elf.h>
+#include <asm/opcodes.h>
+#include <asm/pstate.h>
+#include <crypto/internal/sha2.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+
+static __ro_after_init DEFINE_STATIC_KEY_FALSE(have_sha256_opcodes);
+
+asmlinkage void sha256_sparc64_transform(u32 state[SHA256_STATE_WORDS],
+                                        const u8 *data, size_t nblocks);
+
+void sha256_blocks_arch(u32 state[SHA256_STATE_WORDS],
+                       const u8 *data, size_t nblocks)
+{
+       if (static_branch_likely(&have_sha256_opcodes))
+               sha256_sparc64_transform(state, data, nblocks);
+       else
+               sha256_blocks_generic(state, data, nblocks);
+}
+EXPORT_SYMBOL(sha256_blocks_arch);
+
+bool sha256_is_arch_optimized(void)
+{
+       return static_key_enabled(&have_sha256_opcodes);
+}
+EXPORT_SYMBOL(sha256_is_arch_optimized);
+
+static int __init sha256_sparc64_mod_init(void)
+{
+       unsigned long cfr;
+
+       if (!(sparc64_elf_hwcap & HWCAP_SPARC_CRYPTO))
+               return 0;
+
+       __asm__ __volatile__("rd %%asr26, %0" : "=r" (cfr));
+       if (!(cfr & CFR_SHA256))
+               return 0;
+
+       static_branch_enable(&have_sha256_opcodes);
+       pr_info("Using sparc64 sha256 opcode optimized SHA-256/SHA-224 implementation\n");
+       return 0;
+}
+arch_initcall(sha256_sparc64_mod_init);
+
+static void __exit sha256_sparc64_mod_exit(void)
+{
+}
+module_exit(sha256_sparc64_mod_exit);
+
+MODULE_LICENSE("GPL");
+MODULE_DESCRIPTION("SHA-256 accelerated using the sparc64 sha256 opcodes");
diff --git a/arch/sparc/lib/crypto/sha256_asm.S b/arch/sparc/lib/crypto/sha256_asm.S
new file mode 100644 (file)
index 0000000..ddcdd3d
--- /dev/null
@@ -0,0 +1,78 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+#include <linux/linkage.h>
+#include <asm/opcodes.h>
+#include <asm/visasm.h>
+
+ENTRY(sha256_sparc64_transform)
+       /* %o0 = state, %o1 = data, %o2 = nblocks */
+       VISEntryHalf
+       ld      [%o0 + 0x00], %f0
+       ld      [%o0 + 0x04], %f1
+       ld      [%o0 + 0x08], %f2
+       ld      [%o0 + 0x0c], %f3
+       ld      [%o0 + 0x10], %f4
+       ld      [%o0 + 0x14], %f5
+       andcc   %o1, 0x7, %g0
+       ld      [%o0 + 0x18], %f6
+       bne,pn  %xcc, 10f
+        ld     [%o0 + 0x1c], %f7
+
+1:
+       ldd     [%o1 + 0x00], %f8
+       ldd     [%o1 + 0x08], %f10
+       ldd     [%o1 + 0x10], %f12
+       ldd     [%o1 + 0x18], %f14
+       ldd     [%o1 + 0x20], %f16
+       ldd     [%o1 + 0x28], %f18
+       ldd     [%o1 + 0x30], %f20
+       ldd     [%o1 + 0x38], %f22
+
+       SHA256
+
+       subcc   %o2, 1, %o2
+       bne,pt  %xcc, 1b
+        add    %o1, 0x40, %o1
+
+5:
+       st      %f0, [%o0 + 0x00]
+       st      %f1, [%o0 + 0x04]
+       st      %f2, [%o0 + 0x08]
+       st      %f3, [%o0 + 0x0c]
+       st      %f4, [%o0 + 0x10]
+       st      %f5, [%o0 + 0x14]
+       st      %f6, [%o0 + 0x18]
+       st      %f7, [%o0 + 0x1c]
+       retl
+        VISExitHalf
+10:
+       alignaddr %o1, %g0, %o1
+
+       ldd     [%o1 + 0x00], %f10
+1:
+       ldd     [%o1 + 0x08], %f12
+       ldd     [%o1 + 0x10], %f14
+       ldd     [%o1 + 0x18], %f16
+       ldd     [%o1 + 0x20], %f18
+       ldd     [%o1 + 0x28], %f20
+       ldd     [%o1 + 0x30], %f22
+       ldd     [%o1 + 0x38], %f24
+       ldd     [%o1 + 0x40], %f26
+
+       faligndata %f10, %f12, %f8
+       faligndata %f12, %f14, %f10
+       faligndata %f14, %f16, %f12
+       faligndata %f16, %f18, %f14
+       faligndata %f18, %f20, %f16
+       faligndata %f20, %f22, %f18
+       faligndata %f22, %f24, %f20
+       faligndata %f24, %f26, %f22
+
+       SHA256
+
+       subcc   %o2, 1, %o2
+       fsrc2   %f26, %f10
+       bne,pt  %xcc, 1b
+        add    %o1, 0x40, %o1
+
+       ba,a,pt %xcc, 5b
+ENDPROC(sha256_sparc64_transform)
index 7fe678047939b03adc8f6f73dcacd2dc68b13be7..6319358b38c204d30e9724313406d3d97069493f 100644 (file)
@@ -181,6 +181,9 @@ endif
 if S390
 source "arch/s390/lib/crypto/Kconfig"
 endif
+if SPARC
+source "arch/sparc/lib/crypto/Kconfig"
+endif
 if X86
 source "arch/x86/lib/crypto/Kconfig"
 endif