> -----Original Message----- > From: linux-crypto-owner@xxxxxxxxxxxxxxx <linux-crypto-owner@xxxxxxxxxxxxxxx> On Behalf Of Ard Biesheuvel > Sent: Friday, September 11, 2020 4:11 PM > To: linux-crypto@xxxxxxxxxxxxxxx > Cc: herbert@xxxxxxxxxxxxxxxxxxx; ebiggers@xxxxxxxxxx; Ard Biesheuvel <ardb@xxxxxxxxxx> > Subject: [PATCH] crypto: mark unused ciphers as obsolete > > <<< External Email >>> > We have a few interesting pieces in our cipher museum, which are never > used internally, and were only ever provided as generic C implementations. > > Unfortunately, we cannot simply remove this code, as we cannot be sure > that it is not being used via the AF_ALG socket API, however unlikely. > So let's mark the Anubis, Khazad, SEED and TEA algorithms as obsolete, > Wouldn't the IKE deamon be able to utilize these algorithms through the XFRM API? I'm by no means an expert on the subject, but it looks like the cipher template is provided there directly via XFRM, so it does not need to live in the kernel source. And I know for a fact that SEED is being used for IPsec (and TLS) in Korea. The point being, there are more users to consider beyond "internal" (meaning hard coded in the kernel source in this context?) and AF_ALG. I'm not aware of any real use cases for Anubis, Khazad and TEA though. > which means they can only be enabled in the build if the socket API is > enabled in the first place. > > Signed-off-by: Ard Biesheuvel <ardb@xxxxxxxxxx> > --- > Hopefully, I will be able to convince the distro kernel maintainers to > disable CRYPTO_USER_API_ENABLE_OBSOLETE in their v5.10+ builds once the > iwd changes for arc4 make it downstream (Debian already has an updated > version in its unstable distro). With the joint coverage of their QA, > we should be able to confirm that these algos are never used, and > actually remove them altogether. > > crypto/Kconfig | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/crypto/Kconfig b/crypto/Kconfig > index e85d8a059489..fac10143d23f 100644 > --- a/crypto/Kconfig > +++ b/crypto/Kconfig > @@ -1185,6 +1185,7 @@ config CRYPTO_AES_PPC_SPE > > config CRYPTO_ANUBIS > tristate "Anubis cipher algorithm" > +depends on CRYPTO_USER_API_ENABLE_OBSOLETE > select CRYPTO_ALGAPI > help > Anubis cipher algorithm. > @@ -1424,6 +1425,7 @@ config CRYPTO_FCRYPT > > config CRYPTO_KHAZAD > tristate "Khazad cipher algorithm" > +depends on CRYPTO_USER_API_ENABLE_OBSOLETE > select CRYPTO_ALGAPI > help > Khazad cipher algorithm. > @@ -1487,6 +1489,7 @@ config CRYPTO_CHACHA_MIPS > > config CRYPTO_SEED > tristate "SEED cipher algorithm" > +depends on CRYPTO_USER_API_ENABLE_OBSOLETE > select CRYPTO_ALGAPI > help > SEED cipher algorithm (RFC4269). > @@ -1613,6 +1616,7 @@ config CRYPTO_SM4 > > config CRYPTO_TEA > tristate "TEA, XTEA and XETA cipher algorithms" > +depends on CRYPTO_USER_API_ENABLE_OBSOLETE > select CRYPTO_ALGAPI > help > TEA cipher algorithm. > -- > 2.17.1 Regards, Pascal van Leeuwen Silicon IP Architect Multi-Protocol Engines, Rambus Security Rambus ROTW Holding BV +31-73 6581953 Note: The Inside Secure/Verimatrix Silicon IP team was recently acquired by Rambus. Please be so kind to update your e-mail address book with my new e-mail address. ** This message and any attachments are for the sole use of the intended recipient(s). It may contain information that is confidential and privileged. If you are not the intended recipient of this message, you are prohibited from printing, copying, forwarding or saving it. Please delete the message and attachments and notify the sender immediately. ** Rambus Inc.<http://www.rambus.com>
