Re: [PATCH] crypto: x86/chacha - Remove SIMD fallback path

Eric Biggers
2025-04-07T16:48:44Z

On Thu, Apr 03, 2025 at 12:14:50PM +0800, Herbert Xu wrote:
> On Wed, Apr 02, 2025 at 08:59:34PM -0700, Eric Biggers wrote:
> >
> > But in a lot of cases there is also no reason to even add that restriction. I'm
> > not sure why you're so eager to make the library functions harder to use.
>
> I have no intention of making any changes to siphash. It doesn't
> even use SIMD.
>
> All I want to do is get rid of the crypto_simd_usable() fallback
> paths that we currently have in arch/x86/crypto. This code is
> never used in hardirq context (and should never be).
>
> For example:
>
> ---8<---
> Get rid of the fallback path as SIMD is now always usable in softirq
> context.
>
> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
>

It looks like this broken patch already got applied for some reason.

First, there doesn't seem to be agreement yet that the library functions should
have requirements on the calling context.

Second, your patch made unrelated changes that deleted the checks for SSSE3
support. Thus dropping support for CPUs that don't support SSSE3.

- Eric