On Sat, 23 Nov 2024 at 07:31, Mathieu Desnoyers
<mathieu.desnoy...@efficios.com> wrote:
>
> include/linux/tracepoint.h | 45 ++++++++++----------------------------
> 1 file changed, 12 insertions(+), 33 deletions(-)
Thanks. This looks much more straightforward, and obviously is smaller too.
Side note: I realize I was the one suggesting "scoped_guard()", but
looking at the patch I do think that just unnecessarily added another
level of indentation. Since you already wrote the
if (cond) {
..
}
part as a block statement, there's no upside to the guard having its
own scoped block, so instead of
if (cond) { \
scoped_guard(preempt_notrace) \
__DO_TRACE_CALL(name, TP_ARGS(args)); \
}
this might be simpler as just a plain "guard()" and one less indentation:
if (cond) { \
guard(preempt_notrace); \
__DO_TRACE_CALL(name, TP_ARGS(args)); \
}
but by now this is just an unimportant detail.
I think I suggested scoped_guard() mainly because that would then just
make the "{ }" in the if-statement superfluous, but that's such a
random reason that it *really* doesn't matter.
Linus
