Michael S. Tsirkin
2022-Mar-08 21:12 UTC
[PATCH -next] crypto: virtio - Select new dependencies
On Tue, Mar 08, 2022 at 01:53:09PM -0700, Nathan Chancellor wrote:> With ARCH=riscv defconfig, there are errors at link time: > > virtio_crypto_akcipher_algs.c:(.text+0x3ea): undefined reference to `mpi_free' > virtio_crypto_akcipher_algs.c:(.text+0x48a): undefined reference to `rsa_parse_priv_key' > virtio_crypto_akcipher_algs.c:(.text+0x4bc): undefined reference to `rsa_parse_pub_key' > virtio_crypto_akcipher_algs.c:(.text+0x4d0): undefined reference to `mpi_read_raw_data' > virtio_crypto_akcipher_algs.c:(.text+0x960): undefined reference to `crypto_register_akcipher' > virtio_crypto_akcipher_algs.c:(.text+0xa3a): undefined reference to `crypto_unregister_akcipher' > > The virtio crypto driver started making use of certain libraries and > algorithms without selecting them. Do so to fix these errors. > > Fixes: 8a75f36b5d7a ("virtio-crypto: implement RSA algorithm") > Signed-off-by: Nathan Chancellor <nathan at kernel.org>Thanks! I'll squash this into the original commit so we don't have a broken commit during bisect. zhenwei pi, ack pls?> --- > drivers/crypto/virtio/Kconfig | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/crypto/virtio/Kconfig b/drivers/crypto/virtio/Kconfig > index b894e3a8be4f..5f8915f4a9ff 100644 > --- a/drivers/crypto/virtio/Kconfig > +++ b/drivers/crypto/virtio/Kconfig > @@ -3,8 +3,11 @@ config CRYPTO_DEV_VIRTIO > tristate "VirtIO crypto driver" > depends on VIRTIO > select CRYPTO_AEAD > + select CRYPTO_AKCIPHER2 > select CRYPTO_SKCIPHER > select CRYPTO_ENGINE > + select CRYPTO_RSA > + select MPILIB > help > This driver provides support for virtio crypto device. If you > choose 'M' here, this module will be called virtio_crypto. > > base-commit: c5f633abfd09491ae7ecbc7fcfca08332ad00a8b > -- > 2.35.1
On 3/9/22 05:12, Michael S. Tsirkin wrote:> On Tue, Mar 08, 2022 at 01:53:09PM -0700, Nathan Chancellor wrote: >> With ARCH=riscv defconfig, there are errors at link time: >> >> virtio_crypto_akcipher_algs.c:(.text+0x3ea): undefined reference to `mpi_free' >> virtio_crypto_akcipher_algs.c:(.text+0x48a): undefined reference to `rsa_parse_priv_key' >> virtio_crypto_akcipher_algs.c:(.text+0x4bc): undefined reference to `rsa_parse_pub_key' >> virtio_crypto_akcipher_algs.c:(.text+0x4d0): undefined reference to `mpi_read_raw_data' >> virtio_crypto_akcipher_algs.c:(.text+0x960): undefined reference to `crypto_register_akcipher' >> virtio_crypto_akcipher_algs.c:(.text+0xa3a): undefined reference to `crypto_unregister_akcipher' >> >> The virtio crypto driver started making use of certain libraries and >> algorithms without selecting them. Do so to fix these errors. >> >> Fixes: 8a75f36b5d7a ("virtio-crypto: implement RSA algorithm") >> Signed-off-by: Nathan Chancellor <nathan at kernel.org> > > Thanks! I'll squash this into the original commit so we don't > have a broken commit during bisect. > zhenwei pi, ack pls? >It looks good to me, thanks a lot.>> --- >> drivers/crypto/virtio/Kconfig | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/drivers/crypto/virtio/Kconfig b/drivers/crypto/virtio/Kconfig >> index b894e3a8be4f..5f8915f4a9ff 100644 >> --- a/drivers/crypto/virtio/Kconfig >> +++ b/drivers/crypto/virtio/Kconfig >> @@ -3,8 +3,11 @@ config CRYPTO_DEV_VIRTIO >> tristate "VirtIO crypto driver" >> depends on VIRTIO >> select CRYPTO_AEAD >> + select CRYPTO_AKCIPHER2 >> select CRYPTO_SKCIPHER >> select CRYPTO_ENGINE >> + select CRYPTO_RSA >> + select MPILIB >> help >> This driver provides support for virtio crypto device. If you >> choose 'M' here, this module will be called virtio_crypto. >> >> base-commit: c5f633abfd09491ae7ecbc7fcfca08332ad00a8b >> -- >> 2.35.1-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.linuxfoundation.org/pipermail/virtualization/attachments/20220309/fef97997/attachment.html>