public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] lib/crc: arm64: Drop check for CONFIG_KERNEL_MODE_NEON
@ 2026-03-14 17:57 Eric Biggers
  2026-03-17 11:09 ` Ard Biesheuvel
  2026-03-17 16:31 ` Eric Biggers
  0 siblings, 2 replies; 3+ messages in thread
From: Eric Biggers @ 2026-03-14 17:57 UTC (permalink / raw)
  To: linux-kernel
  Cc: linux-crypto, Ard Biesheuvel, linux-arm-kernel, Catalin Marinas,
	Will Deacon, Eric Biggers

CONFIG_KERNEL_MODE_NEON is always enabled on arm64, and it always has
been since its introduction in 2013.  Given that and the fact that the
usefulness of kernel-mode NEON has only been increasing over time,
checking for this option in arm64-specific code is unnecessary.  Remove
this check from lib/crc/ to simplify the code and prevent any future
bugs where e.g. code gets disabled due to a typo in this logic.

Signed-off-by: Eric Biggers <ebiggers@kernel.org>
---

This patch is targeting crc-next
(https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git/log/?h=crc-next)

 lib/crc/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/crc/Kconfig b/lib/crc/Kconfig
index cca228879bb5a..52e216f397468 100644
--- a/lib/crc/Kconfig
+++ b/lib/crc/Kconfig
@@ -46,11 +46,11 @@ config CRC_T10DIF
 
 config CRC_T10DIF_ARCH
 	bool
 	depends on CRC_T10DIF && CRC_OPTIMIZATIONS
 	default y if ARM && KERNEL_MODE_NEON
-	default y if ARM64 && KERNEL_MODE_NEON
+	default y if ARM64
 	default y if PPC64 && ALTIVEC
 	default y if RISCV && RISCV_ISA_ZBC
 	default y if X86
 
 config CRC32

base-commit: c13cee2fc7f137dd25ed50c63eddcc578624f204
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] lib/crc: arm64: Drop check for CONFIG_KERNEL_MODE_NEON
  2026-03-14 17:57 [PATCH] lib/crc: arm64: Drop check for CONFIG_KERNEL_MODE_NEON Eric Biggers
@ 2026-03-17 11:09 ` Ard Biesheuvel
  2026-03-17 16:31 ` Eric Biggers
  1 sibling, 0 replies; 3+ messages in thread
From: Ard Biesheuvel @ 2026-03-17 11:09 UTC (permalink / raw)
  To: Eric Biggers, linux-kernel
  Cc: linux-crypto, linux-arm-kernel, Catalin Marinas, Will Deacon



On Sat, 14 Mar 2026, at 18:57, Eric Biggers wrote:
> CONFIG_KERNEL_MODE_NEON is always enabled on arm64, and it always has
> been since its introduction in 2013.  Given that and the fact that the
> usefulness of kernel-mode NEON has only been increasing over time,
> checking for this option in arm64-specific code is unnecessary.  Remove
> this check from lib/crc/ to simplify the code and prevent any future
> bugs where e.g. code gets disabled due to a typo in this logic.
>
> Signed-off-by: Eric Biggers <ebiggers@kernel.org>
> ---
>
> This patch is targeting crc-next
> (https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git/log/?h=crc-next)
>
>  lib/crc/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

Acked-by: Ard Biesheuvel <ardb@kernel.org>

> diff --git a/lib/crc/Kconfig b/lib/crc/Kconfig
> index cca228879bb5a..52e216f397468 100644
> --- a/lib/crc/Kconfig
> +++ b/lib/crc/Kconfig
> @@ -46,11 +46,11 @@ config CRC_T10DIF
> 
>  config CRC_T10DIF_ARCH
>  	bool
>  	depends on CRC_T10DIF && CRC_OPTIMIZATIONS
>  	default y if ARM && KERNEL_MODE_NEON
> -	default y if ARM64 && KERNEL_MODE_NEON
> +	default y if ARM64
>  	default y if PPC64 && ALTIVEC
>  	default y if RISCV && RISCV_ISA_ZBC
>  	default y if X86
> 
>  config CRC32
>
> base-commit: c13cee2fc7f137dd25ed50c63eddcc578624f204
> -- 
> 2.53.0

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] lib/crc: arm64: Drop check for CONFIG_KERNEL_MODE_NEON
  2026-03-14 17:57 [PATCH] lib/crc: arm64: Drop check for CONFIG_KERNEL_MODE_NEON Eric Biggers
  2026-03-17 11:09 ` Ard Biesheuvel
@ 2026-03-17 16:31 ` Eric Biggers
  1 sibling, 0 replies; 3+ messages in thread
From: Eric Biggers @ 2026-03-17 16:31 UTC (permalink / raw)
  To: linux-kernel
  Cc: linux-crypto, Ard Biesheuvel, linux-arm-kernel, Catalin Marinas,
	Will Deacon

On Sat, Mar 14, 2026 at 10:57:44AM -0700, Eric Biggers wrote:
> CONFIG_KERNEL_MODE_NEON is always enabled on arm64, and it always has
> been since its introduction in 2013.  Given that and the fact that the
> usefulness of kernel-mode NEON has only been increasing over time,
> checking for this option in arm64-specific code is unnecessary.  Remove
> this check from lib/crc/ to simplify the code and prevent any future
> bugs where e.g. code gets disabled due to a typo in this logic.
> 
> Signed-off-by: Eric Biggers <ebiggers@kernel.org>
> ---
> 
> This patch is targeting crc-next
> (https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git/log/?h=crc-next)

Applied to https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git/log/?h=crc-next

- Eric

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-03-17 16:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-14 17:57 [PATCH] lib/crc: arm64: Drop check for CONFIG_KERNEL_MODE_NEON Eric Biggers
2026-03-17 11:09 ` Ard Biesheuvel
2026-03-17 16:31 ` Eric Biggers

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox