Re: [PATCH] lib/crc-t10dif: remove digest and block size constants

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

 



On Thu, Feb 06, 2025 at 12:09:22PM +0100, Ard Biesheuvel wrote:
> On Tue, 4 Feb 2025 at 21:11, Eric Biggers <ebiggers@xxxxxxxxxx> wrote:
> >
> > From: Eric Biggers <ebiggers@xxxxxxxxxx>
> >
> > These constants are only used in crypto/crct10dif_generic.c, and they
> > are better off just hardcoded there.
> >
> > Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx>
> > ---
> >
> > I'm planning to take this via the crc tree.
> >
> 
> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx>
> 
> Could you remind me why we are keeping this driver?
> 

I don't know.  I had to keep "crc32" and "crc32c" around for now because they
are still used, and I just defaulted to doing the same for "crct10dif".  It
*looks like* "crct10dif" is different and we could indeed safely remove it like
I ended up doing for "crc64-rocksoft", though.  "crct10dif" has no remaining
references in the kernel; it has no references in cryptsetup, iwd, or libell;
and Debian Code Search doesn't find anything else.  It's over a decade old, so I
was worried there was time for users to have crept in, but that doesn't seem to
have happened.  So yeah, good idea.  I'll send a patch that removes it, with the
device-mapper folks Cc'd in case they know of anything else.

- Eric




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