Thread (44 messages) 44 messages, 6 authors, 2021-06-24

Re: [PATCH RESEND x3 v9 1/9] iov_iter: add copy_struct_from_iter()

From: Linus Torvalds <torvalds@linux-foundation.org>
Date: 2021-06-24 17:52:40
Also in: linux-btrfs, linux-fsdevel

On Wed, Jun 23, 2021 at 11:15 PM Omar Sandoval [off-list ref] wrote:
On Thu, Jun 24, 2021 at 03:00:39AM +0100, Matthew Wilcox wrote:
quoted
Does that work for O_DIRECT and the required 512-byte alignment?
I suppose the kernel could pad the encoded_iov structure with zeroes to
the next sector boundary, since zeroes are effectively noops for
encoded_iov.
Ugh.

I really think the whole "embed the control structure in the stream"
is wrong. The alignment issue is just another sign of that.

Separating it out is the right thing to do. At least the "first iov
entry" thing did separate the control structure from the actual data.
I detest the whole "embed the two together".

            Linus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help