Re: [PATCH 0/5] crypto: remove blkcipher

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 26 Oct 2019 at 18:19, Eric Biggers <ebiggers@xxxxxxxxxx> wrote:
>
> On Sat, Oct 26, 2019 at 05:32:05PM +0200, Ard Biesheuvel wrote:
> > On Fri, 25 Oct 2019 at 21:45, Eric Biggers <ebiggers@xxxxxxxxxx> wrote:
> > >
> > > Now that all "blkcipher" algorithms have been converted to "skcipher",
> > > this series removes the blkcipher algorithm type.
> > >
> > > The skcipher (symmetric key cipher) algorithm type was introduced a few
> > > years ago to replace both blkcipher and ablkcipher (synchronous and
> > > asynchronous block cipher).  The advantages of skcipher include:
> > >
> > >   - A much less confusing name, since none of these algorithm types have
> > >     ever actually been for raw block ciphers, but rather for all
> > >     length-preserving encryption modes including block cipher modes of
> > >     operation, stream ciphers, and other length-preserving modes.
> > >
> > >   - It unified blkcipher and ablkcipher into a single algorithm type
> > >     which supports both synchronous and asynchronous implementations.
> > >     Note, blkcipher already operated only on scatterlists, so the fact
> > >     that skcipher does too isn't a regression in functionality.
> > >
> > >   - Better type safety by using struct skcipher_alg, struct
> > >     crypto_skcipher, etc. instead of crypto_alg, crypto_tfm, etc.
> > >
> > >   - It sometimes simplifies the implementations of algorithms.
> > >
> > > Also, the blkcipher API was no longer being tested.
> > >
> > > Eric Biggers (5):
> > >   crypto: unify the crypto_has_skcipher*() functions
> > >   crypto: remove crypto_has_ablkcipher()
> > >   crypto: rename crypto_skcipher_type2 to crypto_skcipher_type
> > >   crypto: remove the "blkcipher" algorithm type
> > >   crypto: rename the crypto_blkcipher module and kconfig option
> > >
> >
> >
> > For the series
> >
> > Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx>
> >
> > although obviously, this needs to wait until my albkcipher purge
> > series is applied.
> >
>
> Why does it need to wait?  This just removes blkcipher, not ablkcipher.
>

OK, ignore me - I failed to notice that :-)



[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]

  Powered by Linux