Thread (8 messages) 8 messages, 4 authors, 2023-08-15

Re: [RFC PATCH] iov_iter: Convert iterate*() to inline funcs

From: Linus Torvalds <torvalds@linux-foundation.org>
Date: 2023-08-11 16:39:11
Also in: linux-fsdevel, linux-mm, lkml

On Fri, 11 Aug 2023 at 07:40, David Howells [off-list ref] wrote:
Convert the iov_iter iteration macros to inline functions to make the code
easier to follow.
I like this generally, the code generation deprovement worries me a
bit, but from a quick look on a test-branch it didn't really look all
that bad (but the changes are too big to usefully show up as asm
diffs)

I do note that maybe you should just also mark
copy_to/from/page_user_iter as being always-inlines. clang actually
seems to do that without prompting, gcc apparently not.

Or at *least* do the memcpy_to/from_iter functions, which are only
wrappers around memcpy and are just completely noise. I'm surprised
gcc didn't already inline that. Strange.

            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