On Sat, 8 Feb 2025 at 18:57, Eric Biggers <ebiggers@xxxxxxxxxx> wrote: > > From: Eric Biggers <ebiggers@xxxxxxxxxx> > > With the "crct10dif" algorithm having been removed from the crypto API, > crc_t10dif_is_optimized() is no longer used. > > Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx> > --- > > This applies to > https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git/log/?h=crc-next > > arch/arm/lib/crc-t10dif-glue.c | 6 ------ > arch/arm64/lib/crc-t10dif-glue.c | 6 ------ > arch/powerpc/lib/crc-t10dif-glue.c | 6 ------ > arch/x86/lib/crc-t10dif-glue.c | 6 ------ > include/linux/crc-t10dif.h | 9 --------- > 5 files changed, 33 deletions(-) > Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> > diff --git a/arch/arm/lib/crc-t10dif-glue.c b/arch/arm/lib/crc-t10dif-glue.c > index d24dee62670e..f3584ba70e57 100644 > --- a/arch/arm/lib/crc-t10dif-glue.c > +++ b/arch/arm/lib/crc-t10dif-glue.c > @@ -67,14 +67,8 @@ arch_initcall(crc_t10dif_arm_init); > static void __exit crc_t10dif_arm_exit(void) > { > } > module_exit(crc_t10dif_arm_exit); > > -bool crc_t10dif_is_optimized(void) > -{ > - return static_key_enabled(&have_neon); > -} > -EXPORT_SYMBOL(crc_t10dif_is_optimized); > - > MODULE_AUTHOR("Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>"); > MODULE_DESCRIPTION("Accelerated CRC-T10DIF using ARM NEON and Crypto Extensions"); > MODULE_LICENSE("GPL v2"); > diff --git a/arch/arm64/lib/crc-t10dif-glue.c b/arch/arm64/lib/crc-t10dif-glue.c > index dab7e3796232..a007d0c5f3fe 100644 > --- a/arch/arm64/lib/crc-t10dif-glue.c > +++ b/arch/arm64/lib/crc-t10dif-glue.c > @@ -68,14 +68,8 @@ arch_initcall(crc_t10dif_arm64_init); > static void __exit crc_t10dif_arm64_exit(void) > { > } > module_exit(crc_t10dif_arm64_exit); > > -bool crc_t10dif_is_optimized(void) > -{ > - return static_key_enabled(&have_asimd); > -} > -EXPORT_SYMBOL(crc_t10dif_is_optimized); > - > MODULE_AUTHOR("Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>"); > MODULE_DESCRIPTION("CRC-T10DIF using arm64 NEON and Crypto Extensions"); > MODULE_LICENSE("GPL v2"); > diff --git a/arch/powerpc/lib/crc-t10dif-glue.c b/arch/powerpc/lib/crc-t10dif-glue.c > index 730850dbc51d..f411b0120cc5 100644 > --- a/arch/powerpc/lib/crc-t10dif-glue.c > +++ b/arch/powerpc/lib/crc-t10dif-glue.c > @@ -76,14 +76,8 @@ arch_initcall(crc_t10dif_powerpc_init); > static void __exit crc_t10dif_powerpc_exit(void) > { > } > module_exit(crc_t10dif_powerpc_exit); > > -bool crc_t10dif_is_optimized(void) > -{ > - return static_key_enabled(&have_vec_crypto); > -} > -EXPORT_SYMBOL(crc_t10dif_is_optimized); > - > MODULE_AUTHOR("Daniel Axtens <dja@xxxxxxxxxx>"); > MODULE_DESCRIPTION("CRCT10DIF using vector polynomial multiply-sum instructions"); > MODULE_LICENSE("GPL"); > diff --git a/arch/x86/lib/crc-t10dif-glue.c b/arch/x86/lib/crc-t10dif-glue.c > index 13f07ddc9122..7734bdbc2e39 100644 > --- a/arch/x86/lib/crc-t10dif-glue.c > +++ b/arch/x86/lib/crc-t10dif-glue.c > @@ -39,13 +39,7 @@ arch_initcall(crc_t10dif_x86_init); > static void __exit crc_t10dif_x86_exit(void) > { > } > module_exit(crc_t10dif_x86_exit); > > -bool crc_t10dif_is_optimized(void) > -{ > - return static_key_enabled(&have_pclmulqdq); > -} > -EXPORT_SYMBOL(crc_t10dif_is_optimized); > - > MODULE_DESCRIPTION("CRC-T10DIF using PCLMULQDQ instructions"); > MODULE_LICENSE("GPL"); > diff --git a/include/linux/crc-t10dif.h b/include/linux/crc-t10dif.h > index d0706544fc11..a559fdff3f7e 100644 > --- a/include/linux/crc-t10dif.h > +++ b/include/linux/crc-t10dif.h > @@ -17,15 +17,6 @@ static inline u16 crc_t10dif_update(u16 crc, const u8 *p, size_t len) > static inline u16 crc_t10dif(const u8 *p, size_t len) > { > return crc_t10dif_update(0, p, len); > } > > -#if IS_ENABLED(CONFIG_CRC_T10DIF_ARCH) > -bool crc_t10dif_is_optimized(void); > -#else > -static inline bool crc_t10dif_is_optimized(void) > -{ > - return false; > -} > -#endif > - > #endif > > base-commit: 3dceb9c4f1202d2c374976936ef803bf4b076fa7 > -- > 2.48.1 >