[PATCH 18/30] notifier: Show function names on notifier routines if DEBUG_NOTIFIERS is set

Guilherme G. Piccoli gpiccoli at igalia.com
Mon May 16 09:14:17 PDT 2022


On 10/05/2022 14:29, Steven Rostedt wrote:
> [...]
> Also, don't sprinkle #ifdef in C code. Instead:
> 
> 	if (IS_ENABLED(CONFIG_DEBUG_NOTIFIERS))
> 		pr_info("notifers: regsiter %ps()\n",
> 			n->notifer_call);
> 
> 
> Or define a print macro at the start of the C file that is a nop if it is
> not defined, and use the macro.

Thanks, I'll go with the IS_ENABLED() idea in V2 - appreciate the hint.
Cheers,


Guilherme



More information about the linux-um mailing list