Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

Vlastimil Babka
2015-12-10T09:52:16Z

On 12/10/2015 04:51 AM, Steven Rostedt wrote:
> I should have been Cc'd on this as I'm the maintainer of a few of the files
> here that is being modified.

Sorry about that.

>> --- a/include/linux/trace_events.h
>> +++ b/include/linux/trace_events.h
>> @@ -15,16 +15,6 @@ struct tracer;
>> struct dentry;
>> struct bpf_prog;
>>
>> -struct trace_print_flags {
>> - unsigned long mask;
>> - const char *name;
>> -};
>> -
>> -struct trace_print_flags_u64 {
>> - unsigned long long mask;
>> - const char *name;
>> -};
>> -
>
> Ingo took some patches from Andi Kleen that creates a tracepoint-defs.h file
> If anything, these should be moved there. That code is currently in tip.

Yeah I noticed that yesterday and seems like a good idea. Rasmus
suggested types.h but these didn't seem general enough for that one. Thanks.