[PATCH v2] ftrace: Make ftrace_graph_is_dead() a static branch
Steven Rostedt
rostedt at goodmis.org
Thu Mar 31 07:19:50 AEDT 2022
On Wed, 30 Mar 2022 06:55:26 +0000
Christophe Leroy <christophe.leroy at csgroup.eu> wrote:
> > Small nit. Please order the includes in "upside-down x-mas tree" fashion:
> >
> > #include <linux/jump_label.h>
> > #include <linux/suspend.h>
> > #include <linux/ftrace.h>
> > #include <linux/slab.h>
> >
>
> That's the first time I get such a request. Usually people request
> #includes to be in alphabetical order so when I see a file that has
> headers in alphabetical order I try to not break it, but here that was
> not the case so I put it at the end of the list.
This is something that Ingo Molnar started back in 2009 or so. And I do
find it easier on the eyes ;-) I may be the only one today trying to keep
it (albeit poorly).
It's not a hard requirement, but I find it makes the code look more like
art, which it is :-D
>
> I'll send v3
Thanks,
-- Steve
More information about the Linuxppc-dev
mailing list