Re: [PATCH 1/2] warn: consolidate warn_slowpath andwarn_on_slowpath

[Ingo Molnar]

* Arjan van de Ven <arjan@xxxxxxxxxxxxxxx> wrote:

> From 7aaa8478fe22e944adf185e465fa07799b87e573 Mon Sep 17 00:00:00 2001
> From: Arjan van de Ven <arjan@xxxxxxxxxxxxxxx>
> Date: Fri, 28 Nov 2008 08:30:46 -0800
> Subject: [PATCH] warn: consolidate warn_slowpath and warn_on_slowpath
> 
> warn_slowpath is a superset of warn_on_slowpath; just have
> warn_on_slowpath call warn_slowpath with a NULL 3rd argument.
> 
> Signed-off-by: Arjan van de Ven <arjan@xxxxxxxxxxxxxxx>
> ---
>  kernel/panic.c |   32 ++++++++++++--------------------
>  1 files changed, 12 insertions(+), 20 deletions(-)
> 
> diff --git a/kernel/panic.c b/kernel/panic.c
> index d1ac662..2b913aa 100644
> --- a/kernel/panic.c
> +++ b/kernel/panic.c
> @@ -324,23 +324,6 @@ void oops_exit(void)
>  }
>  
>  #ifdef WANT_WARN_ON_SLOWPATH
> -void warn_on_slowpath(const char *file, int line)
> -{
> -	char function[KSYM_SYMBOL_LEN];
> -	unsigned long caller = (unsigned long) __builtin_return_address(0);
> -	sprint_symbol(function, caller);
> -
> -	printk(KERN_WARNING "------------[ cut here ]------------\n");
> -	printk(KERN_WARNING "WARNING: at %s:%d %s()\n", file,
> -		line, function);
> -	print_modules();
> -	dump_stack();
> -	print_oops_end_marker();
> -	add_taint(TAINT_WARN);
> -}
> -EXPORT_SYMBOL(warn_on_slowpath);
> -
> -
>  void warn_slowpath(const char *file, int line, const char *fmt, ...)
>  {
>  	va_list args;
> @@ -351,9 +334,12 @@ void warn_slowpath(const char *file, int line, const char *fmt, ...)
>  	printk(KERN_WARNING "------------[ cut here ]------------\n");
>  	printk(KERN_WARNING "WARNING: at %s:%d %s()\n", file,
>  		line, function);
> -	va_start(args, fmt);
> -	vprintk(fmt, args);
> -	va_end(args);
> +
> +	if (fmt) {
> +		va_start(args, fmt);
> +		vprintk(fmt, args);
> +		va_end(args);
> +	}
>  
>  	print_modules();
>  	dump_stack();
> @@ -361,6 +347,12 @@ void warn_slowpath(const char *file, int line, const char *fmt, ...)
>  	add_taint(TAINT_WARN);
>  }
>  EXPORT_SYMBOL(warn_slowpath);
> +
> +void warn_on_slowpath(const char *file, int line)
> +{
> +	warn_slowpath(file, line, NULL);
> +}
> +EXPORT_SYMBOL(warn_on_slowpath);
>  #endif

well spotted! Applied to tip/core/debug, thanks Arjan!

I've also done the change below that goes one step further, and turns 
eliminates warn_on_slowpath() altogether and calls warn_slowpath(file, 
line, NULL) straight from include/asm-generic/bug.h's __WARN() 
definition.

	Ingo

------------->