Thread (10 messages) 10 messages, 2 authors, 2025-01-16

Re: [PATCH] tracing: gfp: Fix the GFP enum values shown for user space tracing tools

flat view

From: Steven Rostedt <rostedt@goodmis.org>
Date: 2025-01-16 19:30:15
Also in: linux-mm, linux-perf-users, lkml

On Thu, 16 Jan 2025 11:19:20 -0800
Linus Torvalds [off-list ref] wrote:
On Thu, 16 Jan 2025 at 11:17, Steven Rostedt [off-list ref] wrote:
quoted
But they are only defines, they are not created until used. Are you worried
that it will slow down the compile?  
Yes, and the whole "pointless noise" thing.

I *look* at header files sometimes. I don't want to see pointless
noise that isn't relevant to the context.

This is not relevant to non-tracing users.
Would it at least be OK to add a comment there to update the tracing code
if a new GFP flag is added?

/*
 * If a new GFP flag is added, a new TRACE_DEFINE_ENUM needs to be added
 * in path/to/where/define/is/made.h
 */


?

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