Re: [PATCH v6 09/13] iov_iter: Add a kernel-type iterator-only iteration function

From: Linus Torvalds
Date: Fri Sep 22 2023 - 14:51:43 EST


On Fri, 22 Sept 2023 at 05:03, David Howells <dhowells@xxxxxxxxxx> wrote:
>
> Add an iteration function that can only iterate over kernel internal-type
> iterators (ie. BVEC, KVEC, XARRAY) and not user-backed iterators (ie. UBUF
> and IOVEC). This allows for smaller iterators to be built when it is known
> the caller won't have a user-backed iterator.

This one is pretty ugly, and has no actual users.

Without even explaining why we'd care about this abomination, NAK.

If we actyually have some static knowledge of "this will only use
iterators X/Y/Z", then we should probably pass that in as a constant
bitmask to the thing, instead of this kind of "kernel only" special
case.

But even then, we'd want to have actual explicit use-cases, not a
hypothetical "if you have this situation here's this function".

Linus