Subject: Re: linux-next: build failure after merge of the net-next tree

From: David Howells <[email-masked]>
Date: Mon, 26 Jun 2023 08:48:26 +0100

Stephen Rothwell <[email-masked]> wrote:

> After merging the net-next tree, today's linux-next build (native perf)
> failed like this:
> 
> In file included from builtin-trace.c:907:
> trace/beauty/msg_flags.c: In function 'syscall_arg__scnprintf_msg_flags':
> trace/beauty/msg_flags.c:28:21: error: 'MSG_SPLICE_PAGES' undeclared (first use in this function)

I tried applying the attached patch, but it doesn't make any difference.

Any idea what I've missed?  Also, why do we have duplicates of all the kernel
headers in the tools/ directory?

David
---

commit 878ff45f5f746f6773224ff952c490b5812462f2
Author: David Howells <[email-masked]>
Date:   Mon Jun 26 08:08:12 2023 +0100

    tools: Fix MSG_SPLICE_PAGES build error in trace tools
    
    Fix the following error:
    
    In file included from builtin-trace.c:907:
    trace/beauty/msg_flags.c: In function 'syscall_arg__scnprintf_msg_flags':
    trace/beauty/msg_flags.c:28:21: error: 'MSG_SPLICE_PAGES' undeclared (first use in this function)
       28 |         if (flags & MSG_##n) { \
          |                     ^~~~
    trace/beauty/msg_flags.c:50:9: note: in expansion of macro 'P_MSG_FLAG'
       50 |         P_MSG_FLAG(SPLICE_PAGES);
          |         ^~~~~~~~~~
    trace/beauty/msg_flags.c:28:21: note: each undeclared identifier is reported only once for each function it appears in
       28 |         if (flags & MSG_##n) { \
          |                     ^~~~
    trace/beauty/msg_flags.c:50:9: note: in expansion of macro 'P_MSG_FLAG'
       50 |         P_MSG_FLAG(SPLICE_PAGES);
          |         ^~~~~~~~~~
    
    There is no MSG_SPLICE_PAGES in tools/perf/trace/beauty/include/linux/socket.h
    
    Fixes: b848b26c6672 ("net: Kill MSG_SENDPAGE_NOTLAST")
    Reported-by: Stephen Rothwell <[email-masked]>
    Link: https://lore.kernel.org/r/[email-masked]/
    Signed-off-by: David Howells <[email-masked]>
    cc: "David S. Miller" <[email-masked]>
    cc: Eric Dumazet <[email-masked]>
    cc: Jakub Kicinski <[email-masked]>
    cc: Paolo Abeni <[email-masked]>
    cc: Jens Axboe <[email-masked]>
    cc: Matthew Wilcox <[email-masked]>
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]
    cc: [email-masked]

diff --git a/tools/perf/trace/beauty/include/linux/socket.h b/tools/perf/trace/beauty/include/linux/socket.h
index 3bef212a24d7..77cb707a566a 100644
--- a/tools/perf/trace/beauty/include/linux/socket.h
+++ b/tools/perf/trace/beauty/include/linux/socket.h
@@ -326,6 +326,7 @@ struct ucred {
 					  */
 
 #define MSG_ZEROCOPY	0x4000000	/* Use user data in kernel path */
+#define MSG_SPLICE_PAGES 0x8000000	/* Splice the pages from the iterator in sendmsg() */
 #define MSG_FASTOPEN	0x20000000	/* Send data in TCP SYN */
 #define MSG_CMSG_CLOEXEC 0x40000000	/* Set close_on_exec for file
 					   descriptor received through