diff mbox series

[v2] Enable config for sm3, sm4, polyval, xctr cipher test as built in module.

Message ID 20240205112933.3084766-1-kshitiz.varshney@nxp.com
State New
Headers show
Series [v2] Enable config for sm3, sm4, polyval, xctr cipher test as built in module. | expand

Commit Message

Kshitiz Varshney Feb. 5, 2024, 11:29 a.m. UTC
Issue:-
Multiple crypto tests like sm3, sm4 xctr and polyval failing.

skcipher: failed to allocate transform for xctr(aes): -2
alg: self-tests for xctr(aes) using xctr(aes) failed (rc=-2)
alg: self-tests for sm3 using sm3 failed (rc=-2)
tcrypt: failed to load transform for sm3: -2
alg: hash: failed to allocate transform for polyval: -2
alg: self-tests for polyval using polyval failed (rc=-2)

Resolution:-
Enabling CONFIG_CRYPTO_SM3_GENERIC, CONFIG_CRYPTO_SM4_GENERIC,
CONFIG_CRYPTO_HCTR2, CONFIG_CRYPTO_ARIA as module enables support of
sm3, sm4, aria, polyval & xctr.

Signed-off-by: Kshitiz Varshney <kshitiz.varshney@nxp.com>
---
 arch/arm/configs/imx_v6_v7_defconfig | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Eric Biggers Feb. 5, 2024, 8:19 p.m. UTC | #1
On Mon, Feb 05, 2024 at 12:29:33PM +0100, Kshitiz Varshney wrote:
> Issue:-
> Multiple crypto tests like sm3, sm4 xctr and polyval failing.
> 
> skcipher: failed to allocate transform for xctr(aes): -2
> alg: self-tests for xctr(aes) using xctr(aes) failed (rc=-2)
> alg: self-tests for sm3 using sm3 failed (rc=-2)
> tcrypt: failed to load transform for sm3: -2
> alg: hash: failed to allocate transform for polyval: -2
> alg: self-tests for polyval using polyval failed (rc=-2)
> 
> Resolution:-
> Enabling CONFIG_CRYPTO_SM3_GENERIC, CONFIG_CRYPTO_SM4_GENERIC,
> CONFIG_CRYPTO_HCTR2, CONFIG_CRYPTO_ARIA as module enables support of
> sm3, sm4, aria, polyval & xctr.
> 
> Signed-off-by: Kshitiz Varshney <kshitiz.varshney@nxp.com>

This looks like an issue in tcrypt.  When algorithms are unavailable it should
consider the corresponding test cases to be skipped, not failed.

>  arch/arm/configs/imx_v6_v7_defconfig | 4 ++++

There should be no need to enable every crypto algorithm in every defconfig
file.  Can you consider fixing the underlying issue instead of doing that?

- Eric
diff mbox series

Patch

diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
index 0a90583f9f01..bc2bfc7c61b0 100644
--- a/arch/arm/configs/imx_v6_v7_defconfig
+++ b/arch/arm/configs/imx_v6_v7_defconfig
@@ -466,6 +466,10 @@  CONFIG_NLS_ISO8859_1=y
 CONFIG_NLS_ISO8859_15=m
 CONFIG_NLS_UTF8=y
 CONFIG_SECURITYFS=y
+CONFIG_CRYPTO_ARIA=m
+CONFIG_CRYPTO_SM4_GENERIC=m
+CONFIG_CRYPTO_HCTR2=m
+CONFIG_CRYPTO_SM3_GENERIC=m
 CONFIG_CRYPTO_DEV_FSL_CAAM=y
 CONFIG_CRYPTO_DEV_SAHARA=y
 CONFIG_CRYPTO_DEV_MXS_DCP=y