Messages in this thread | | | From | Ard Biesheuvel <> | Date | Fri, 18 Aug 2017 14:54:31 +0100 | Subject | Re: [PATCH v2 6/6] kernel: tracepoints: add support for relative references |
| |
On 18 August 2017 at 14:52, Steven Rostedt <rostedt@goodmis.org> wrote: > On Fri, 18 Aug 2017 14:44:15 +0100 > Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote: > >> >> It appears the stuff above needs to be move inside the double-include >> >> guard (which oddly enough does not cover the entire file) >> > >> > Why was this moved to the header file? To fulfill some checkpatch >> > warning? >> > >> >> Yes. > > My preference is to ignore that checkpatch warning. The section > variables are created by linker magic, and not normal "extern" > variables. They are only used in one location, and I like to keep them > where they are used, and not be something other places might think they > can be used. In other words, keep them by the C code, and out of > headers. > > Tracepoints and linker/asm work always triggers a lot of bogus > checkpatch warnings. Which is unfortunate. :-/ >
Actually, I couldn't agree more. I will backpedal on the checkpatch appeasement in v3 in general.
Thanks, Ard.
| |