From: Simon Richter <Simon.Richter@hogyros.de>
To: Eric Biggers <ebiggers@kernel.org>
Cc: Christophe Leroy <christophe.leroy@csgroup.eu>,
linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org,
Ard Biesheuvel <ardb@kernel.org>,
"Jason A . Donenfeld" <Jason@zx2c4.com>,
linux-mips@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,
sparclinux@vger.kernel.org
Subject: Re: Crypto use cases
Date: Tue, 5 Aug 2025 16:17:49 +0900 [thread overview]
Message-ID: <854efc41-c40f-46c9-b8ae-84bda9d17faa@hogyros.de> (raw)
In-Reply-To: <20250805045846.GA10695@sol>
Hi,
On 8/5/25 13:58, Eric Biggers wrote:
> What does this have to do with this thread, which is about the PowerPC
> optimized MD5 code?
Hence the new subject. It is still related to removal of code, but asks
about the bigger picture.
The code removal changes you've been pushing lately absolutely make
sense in the context of "the crypto subsystem is for internal use by the
kernel, and all known users will only ever submit small work items."
However, there is also the userspace API, and hardware accelerators also
register with the crypto subsystem, so it is *also* the way for
userspace to use specialized hardware.
If these are separate, then it makes sense to acknowledge that the
kernel will never use asynchronous transforms for anything, simplify the
internal API, and move all the hardware support to a separate registry
that is for userspace applications only.
However if we don't want separate registries, then it makes no sense for
the kernel to set policy for userspace here; it should offer all the
alternatives. The kernel can, and should, define policy for itself, and
I wholeheartedly agree that offloading small requests does not make
sense, unless we're on battery power.
I'm also not convinced that fscrypt cannot ever learn to submit a single
large request or a large batch of small requests if it is asked to
decrypt a large file, so in my opinion the common registry makes more sense.
Making sure that hardware support actually works and is tested is the
responsibility of the driver and port maintainers. We understand that
subsystem maintainers do not have all the hardware available, but the
same goes for all the other subsystems -- the DRM maintainers routinely
merge drivers for hardware they do not own, because the changes come
from people who *do* own the hardware, and have tested the changes.
The latter is a project management issue, mostly: if there is a lack of
working relationships with driver and port maintainers, then that needs
to be fixed, not assumed to be an unchangeable part of the environment
that technical decisions are made in.
Simon
next prev parent reply other threads:[~2025-08-05 7:17 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-03 20:44 [PATCH 0/7] MD5 library functions Eric Biggers
2025-08-03 20:44 ` [PATCH 1/7] mips: cavium-octeon: Remove Octeon optimized MD5 code Eric Biggers
2025-08-03 20:44 ` [PATCH 2/7] mips: cavium-octeon: Move octeon-crypto.c into parent dir Eric Biggers
2025-08-03 20:44 ` [PATCH 3/7] crypto: powerpc/md5 - Remove PowerPC optimized MD5 code Eric Biggers
2025-08-03 22:07 ` Segher Boessenkool
2025-08-03 22:14 ` Eric Biggers
2025-08-03 22:27 ` Segher Boessenkool
2025-08-03 22:56 ` Eric Biggers
2025-08-04 17:42 ` Christophe Leroy
2025-08-04 18:09 ` Eric Biggers
2025-08-04 19:02 ` Christophe Leroy
2025-08-04 22:59 ` Eric Biggers
2025-08-04 23:09 ` Eric Biggers
2025-08-05 6:21 ` Christophe Leroy
2025-08-05 4:49 ` Crypto use cases (was: Remove PowerPC optimized MD5 code) Simon Richter
2025-08-05 4:58 ` Eric Biggers
2025-08-05 7:17 ` Simon Richter [this message]
2025-08-05 17:15 ` Crypto use cases Eric Biggers
2025-08-05 6:27 ` [PATCH 3/7] crypto: powerpc/md5 - Remove PowerPC optimized MD5 code Christophe Leroy
2025-08-05 16:16 ` Eric Biggers
2025-08-03 20:44 ` [PATCH 4/7] crypto: sparc/md5 - Remove SPARC64 " Eric Biggers
2025-08-04 4:44 ` Simon Richter
2025-08-04 6:07 ` Eric Biggers
2025-08-03 20:44 ` [PATCH 5/7] lib/crypto: md5: Add MD5 and HMAC-MD5 library functions Eric Biggers
2025-08-03 20:44 ` [PATCH 6/7] crypto: md5 - Wrap library and add HMAC support Eric Biggers
2025-08-03 20:44 ` [PATCH 7/7] lib/crypto: tests: Add KUnit tests for MD5 and HMAC-MD5 Eric Biggers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=854efc41-c40f-46c9-b8ae-84bda9d17faa@hogyros.de \
--to=simon.richter@hogyros.de \
--cc=Jason@zx2c4.com \
--cc=ardb@kernel.org \
--cc=christophe.leroy@csgroup.eu \
--cc=ebiggers@kernel.org \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=sparclinux@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®