* [PATCH] crypto: inside-secure - select CONFIG_CRYPTO_SM3
@ 2019-10-22 14:28 Arnd Bergmann
2019-10-22 15:42 ` Pascal Van Leeuwen
0 siblings, 1 reply; 4+ messages in thread
From: Arnd Bergmann @ 2019-10-22 14:28 UTC (permalink / raw)
To: Herbert Xu, David S. Miller
Cc: Arnd Bergmann, Antoine Tenart, Pascal van Leeuwen,
Ard Biesheuvel, Pascal van Leeuwen, linux-crypto, linux-kernel
Without this symbol, the safexcel driver causes a link error:
drivers/crypto/inside-secure/safexcel_hash.o: In function `safexcel_ahash_final':
safexcel_hash.c:(.text+0x3c4): undefined reference to `sm3_zero_message_hash'
Fixes: 0f2bc13181ce ("crypto: inside-secure - Added support for basic SM3 ahash")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/crypto/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/crypto/Kconfig b/drivers/crypto/Kconfig
index 357e230769c8..1ca8d9a15f2a 100644
--- a/drivers/crypto/Kconfig
+++ b/drivers/crypto/Kconfig
@@ -753,6 +753,7 @@ config CRYPTO_DEV_SAFEXCEL
select CRYPTO_SHA512
select CRYPTO_CHACHA20POLY1305
select CRYPTO_SHA3
+ select CRYPTO_SM3
help
This driver interfaces with the SafeXcel EIP-97 and EIP-197 cryptographic
engines designed by Inside Secure. It currently accelerates DES, 3DES and
--
2.20.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* RE: [PATCH] crypto: inside-secure - select CONFIG_CRYPTO_SM3
2019-10-22 14:28 [PATCH] crypto: inside-secure - select CONFIG_CRYPTO_SM3 Arnd Bergmann
@ 2019-10-22 15:42 ` Pascal Van Leeuwen
2019-10-22 16:17 ` Arnd Bergmann
0 siblings, 1 reply; 4+ messages in thread
From: Pascal Van Leeuwen @ 2019-10-22 15:42 UTC (permalink / raw)
To: Arnd Bergmann, Herbert Xu, David S. Miller
Cc: Antoine Tenart, Ard Biesheuvel, Pascal van Leeuwen, linux-crypto,
linux-kernel
> -----Original Message-----
> From: Arnd Bergmann <arnd@arndb.de>
> Sent: Tuesday, October 22, 2019 4:29 PM
> To: Herbert Xu <herbert@gondor.apana.org.au>; David S. Miller <davem@davemloft.net>
> Cc: Arnd Bergmann <arnd@arndb.de>; Antoine Tenart <antoine.tenart@bootlin.com>; Pascal Van
> Leeuwen <pvanleeuwen@verimatrix.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>; Pascal van
> Leeuwen <pascalvanl@gmail.com>; linux-crypto@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: [PATCH] crypto: inside-secure - select CONFIG_CRYPTO_SM3
>
> Without this symbol, the safexcel driver causes a link error:
>
> drivers/crypto/inside-secure/safexcel_hash.o: In function `safexcel_ahash_final':
> safexcel_hash.c:(.text+0x3c4): undefined reference to `sm3_zero_message_hash'
>
> Fixes: 0f2bc13181ce ("crypto: inside-secure - Added support for basic SM3 ahash")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> drivers/crypto/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/crypto/Kconfig b/drivers/crypto/Kconfig
> index 357e230769c8..1ca8d9a15f2a 100644
> --- a/drivers/crypto/Kconfig
> +++ b/drivers/crypto/Kconfig
> @@ -753,6 +753,7 @@ config CRYPTO_DEV_SAFEXCEL
> select CRYPTO_SHA512
> select CRYPTO_CHACHA20POLY1305
> select CRYPTO_SHA3
> + select CRYPTO_SM3
>
Was this problem observed with the latest state of the Cryptodev GIT?
Because I already attempted to fix this issue with commit 99a59da3723b9725
Can you please double check if you still get the compile error with that
commit included?
(I can't tell from this mail which version of the sources you are using)
> help
> This driver interfaces with the SafeXcel EIP-97 and EIP-197 cryptographic
> engines designed by Inside Secure. It currently accelerates DES, 3DES and
> --
> 2.20.0
Regards,
Pascal van Leeuwen
Silicon IP Architect, Multi-Protocol Engines @ Verimatrix
www.insidesecure.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] crypto: inside-secure - select CONFIG_CRYPTO_SM3
2019-10-22 15:42 ` Pascal Van Leeuwen
@ 2019-10-22 16:17 ` Arnd Bergmann
2019-10-22 16:18 ` Pascal Van Leeuwen
0 siblings, 1 reply; 4+ messages in thread
From: Arnd Bergmann @ 2019-10-22 16:17 UTC (permalink / raw)
To: Pascal Van Leeuwen
Cc: Herbert Xu, David S. Miller, Antoine Tenart, Ard Biesheuvel,
Pascal van Leeuwen, linux-crypto, linux-kernel
On Tue, Oct 22, 2019 at 5:42 PM Pascal Van Leeuwen
<pvanleeuwen@verimatrix.com> wrote:
> > diff --git a/drivers/crypto/Kconfig b/drivers/crypto/Kconfig
> > index 357e230769c8..1ca8d9a15f2a 100644
> > --- a/drivers/crypto/Kconfig
> > +++ b/drivers/crypto/Kconfig
> > @@ -753,6 +753,7 @@ config CRYPTO_DEV_SAFEXCEL
> > select CRYPTO_SHA512
> > select CRYPTO_CHACHA20POLY1305
> > select CRYPTO_SHA3
> > + select CRYPTO_SM3
> >
> Was this problem observed with the latest state of the Cryptodev GIT?
> Because I already attempted to fix this issue with commit 99a59da3723b9725
> Can you please double check if you still get the compile error with that
> commit included?
> (I can't tell from this mail which version of the sources you are using)
I was testing on linux-5.4-rc3 plus some of my own patches, so your fix
was not included.
With your patch applied, mine is no longer needed.
Arnd
^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH] crypto: inside-secure - select CONFIG_CRYPTO_SM3
2019-10-22 16:17 ` Arnd Bergmann
@ 2019-10-22 16:18 ` Pascal Van Leeuwen
0 siblings, 0 replies; 4+ messages in thread
From: Pascal Van Leeuwen @ 2019-10-22 16:18 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Herbert Xu, David S. Miller, Antoine Tenart, Ard Biesheuvel,
Pascal van Leeuwen, linux-crypto, linux-kernel
> -----Original Message-----
> From: Arnd Bergmann <arnd@arndb.de>
> Sent: Tuesday, October 22, 2019 6:17 PM
> To: Pascal Van Leeuwen <pvanleeuwen@verimatrix.com>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>; David S. Miller <davem@davemloft.net>; Antoine
> Tenart <antoine.tenart@bootlin.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>; Pascal van
> Leeuwen <pascalvanl@gmail.com>; linux-crypto@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH] crypto: inside-secure - select CONFIG_CRYPTO_SM3
>
> On Tue, Oct 22, 2019 at 5:42 PM Pascal Van Leeuwen
> <pvanleeuwen@verimatrix.com> wrote:
>
> > > diff --git a/drivers/crypto/Kconfig b/drivers/crypto/Kconfig
> > > index 357e230769c8..1ca8d9a15f2a 100644
> > > --- a/drivers/crypto/Kconfig
> > > +++ b/drivers/crypto/Kconfig
> > > @@ -753,6 +753,7 @@ config CRYPTO_DEV_SAFEXCEL
> > > select CRYPTO_SHA512
> > > select CRYPTO_CHACHA20POLY1305
> > > select CRYPTO_SHA3
> > > + select CRYPTO_SM3
> > >
> > Was this problem observed with the latest state of the Cryptodev GIT?
> > Because I already attempted to fix this issue with commit 99a59da3723b9725
> > Can you please double check if you still get the compile error with that
> > commit included?
> > (I can't tell from this mail which version of the sources you are using)
>
> I was testing on linux-5.4-rc3 plus some of my own patches, so your fix
> was not included.
>
> With your patch applied, mine is no longer needed.
>
> Arnd
Thanks for confirming that :-)
Regards,
Pascal van Leeuwen
Silicon IP Architect, Multi-Protocol Engines @ Verimatrix
www.insidesecure.com
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-10-22 16:19 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-22 14:28 [PATCH] crypto: inside-secure - select CONFIG_CRYPTO_SM3 Arnd Bergmann
2019-10-22 15:42 ` Pascal Van Leeuwen
2019-10-22 16:17 ` Arnd Bergmann
2019-10-22 16:18 ` Pascal Van Leeuwen
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.