On Fri, 21 Feb 2020 18:44:19 +0100 Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> wrote: > diff --git a/include/linux/trace_events.h b/include/linux/trace_events.h > index f3b1ef07e4a5f..c8b3f06cb5ed7 100644 > --- a/include/linux/trace_events.h > +++ b/include/linux/trace_events.h > @@ -62,9 +62,9 @@ struct trace_entry { > unsigned char flags; > unsigned char preempt_count; > int pid; > - unsigned short migrate_disable; > - unsigned short padding; > + unsigned char migrate_disable; > unsigned char preempt_lazy_count; > + unsigned short padding; > }; Do we really need the padding here? -- Steve