linux-kernel - Re: [PATCH v2] tracing: Dump either the oops's cpu source or all cpus buffers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20100418192902.f3fad119.randy.dunlap@oracle.com>
Date:	Sun, 18 Apr 2010 19:29:02 -0700
From:	Randy Dunlap <randy.dunlap@...cle.com>
To:	Frederic Weisbecker <fweisbec@...il.com>
Cc:	Steven Rostedt <rostedt@...dmis.org>,
	LKML <linux-kernel@...r.kernel.org>,
	Thomas Gleixner <tglx@...utronix.de>,
	Li Zefan <lizf@...fujitsu.com>,
	Lai Jiangshan <laijs@...fujitsu.com>,
	Ingo Molnar <mingo@...e.hu>
Subject: Re: [PATCH v2] tracing: Dump either the oops's cpu source or all
 cpus buffers

On Mon, 19 Apr 2010 03:15:40 +0200 Frederic Weisbecker wrote:

>  Documentation/kernel-parameters.txt |    6 +++-
>  Documentation/trace/ftrace.txt      |    6 +++-
>  drivers/char/sysrq.c                |    2 +-
>  include/linux/ftrace.h              |    4 ++-
>  include/linux/kernel.h              |   11 ++++++-
>  kernel/trace/trace.c                |   51 ++++++++++++++++++++++++++--------
>  6 files changed, 61 insertions(+), 19 deletions(-)
> 
> diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
> index e4cbca5..f6c68e8 100644
> --- a/Documentation/kernel-parameters.txt
> +++ b/Documentation/kernel-parameters.txt
> @@ -789,8 +789,12 @@ and is between 256 and 4096 characters. It is defined in the file
>  			as early as possible in order to facilitate early
>  			boot debugging.
>  
> -	ftrace_dump_on_oops
> +	ftrace_dump_on_oops[=orig_cpu]
>  			[FTRACE] will dump the trace buffers on oops.
> +			If no parameter is passed, ftrace will dump
> +			every cpu buffers, but if you pass orig_cpu, it will

			buffers of all CPUs,

> +			dump only the buffer of the cpu that triggered the

			                            CPU

> +			oops.
>  
>  	ftrace_filter=[function-list]
>  			[FTRACE] Limit the functions traced by the function
> diff --git a/Documentation/trace/ftrace.txt b/Documentation/trace/ftrace.txt
> index 03485bf..d242bc0 100644
> --- a/Documentation/trace/ftrace.txt
> +++ b/Documentation/trace/ftrace.txt
> @@ -1337,12 +1337,14 @@ ftrace_dump_on_oops must be set. To set ftrace_dump_on_oops, one
>  can either use the sysctl function or set it via the proc system
>  interface.
>  
> -  sysctl kernel.ftrace_dump_on_oops=1
> +  sysctl kernel.ftrace_dump_on_oops=n
>  
>  or
>  
> -  echo 1 > /proc/sys/kernel/ftrace_dump_on_oops
> +  echo n > /proc/sys/kernel/ftrace_dump_on_oops
>  
> +If n = 1, ftrace will dump every cpu buffers, if n = 2 ftrace will

                              buffers of all CPUs,

> +only dump the buffer of the cpu that triggered the oops.

                               CPU

>  
>  Here's an example of such a dump after a null pointer
>  dereference in a kernel module:
> diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c
> index bed83ca..4252ffe 100644
> --- a/kernel/trace/trace.c
> +++ b/kernel/trace/trace.c
> @@ -117,9 +117,12 @@ static cpumask_var_t __read_mostly	tracing_buffer_mask;
>   *
>   * It is default off, but you can enable it with either specifying
>   * "ftrace_dump_on_oops" in the kernel command line, or setting
> - * /proc/sys/kernel/ftrace_dump_on_oops to true.
> + * /proc/sys/kernel/ftrace_dump_on_oops
> + * Set 1 if you want to dump every cpu buffers

                                buffers of all CPUs

> + * Set 2 if you want to dump the buffer of the cpu that triggered oops

                                                  CPU

>   */
> -int ftrace_dump_on_oops;
> +
> +enum ftrace_dump_mode ftrace_dump_on_oops;
>  
>  static int tracing_set_tracer(const char *buf);
>  
> @@ -4429,12 +4442,25 @@ static void __ftrace_dump(bool disable_tracing)
>  	/* don't look at user memory in panic mode */
>  	trace_flags &= ~TRACE_ITER_SYM_USEROBJ;
>  
> -	printk(KERN_TRACE "Dumping ftrace buffer:\n");
> -
>  	/* Simulate the iterator */
>  	iter.tr = &global_trace;
>  	iter.trace = current_trace;
> -	iter.cpu_file = TRACE_PIPE_ALL_CPU;
> +
> +	switch (oops_dump_mode) {
> +	case DUMP_ALL:
> +		iter.cpu_file = TRACE_PIPE_ALL_CPU;
> +		break;
> +	case DUMP_ORIG:
> +		iter.cpu_file = raw_smp_processor_id();
> +		break;
> +	case DUMP_NONE:
> +		goto out_enable;
> +	default:
> +		printk(KERN_TRACE "Bad dumping mode, switching to all cpu dump\n");

		                                                      CPUs

> +		iter.cpu_file = TRACE_PIPE_ALL_CPU;
> +	}
> +
> +	printk(KERN_TRACE "Dumping ftrace buffer:\n");
>  
>  	/*
>  	 * We need to stop all tracing on all CPUS to read the


---
~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/