login
Login
/
Register
Search
Search this site:
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2008
»
December
»
14
Re: [PATCH] tracing/ftrace: add the printk-msg-only option
view
thread
Previous message: [
thread
] [
date
] [
author
]
Next message: [thread] [
date
] [
author
]
[view in full thread]
From: =?ISO-8859-1?Q?Fr=E9d=E9ric_Weisbecker?=
Subject:
Re: [PATCH] tracing/ftrace: add the printk-msg-only option
Date: Sunday, December 14, 2008 - 10:52 am
2008/12/14 Steven Rostedt <rostedt@goodmis.org>:
quoted text
> > On Sat, 13 Dec 2008, Frederic Weisbecker wrote: > >> Impact: display ftrace_printk messages "as is" >> >> By default, ftrace_printk() messages find their output with some other >> informations like pid, caller, ... >> Sometimes a developer just want to have the ftrace_printk left "as is", without >> other information. >> >> This is done by providing a default-off option called printk-msg-only. >> To enable it, just do `echo printk-msg-only > /debugfs/tracing/trace_options` >> >> Before the patch: >> >> <...>-2739 [000] 145.692153: __might_sleep: I'm an ftrace_printk msg in __might_sleep >> <...>-2739 [000] 145.692155: __might_sleep: I'm another ftrace_printk msg in __might_sleep >> >> After the patch and the printk-msg-only option enabled: >> >> I'm an ftrace_printk msg in __might_sleep >> I'm another ftrace_printk msg in __might_sleep >> >> Cc: Steven Rostedt <rostedt@goodmis.org> >> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com> >> --- >> diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c >> index 8fd3c5a..cd4dbf2 100644 >> --- a/kernel/trace/trace.c >> +++ b/kernel/trace/trace.c >> @@ -287,6 +287,7 @@ static const char *trace_options[] = { >> "annotate", >> "userstacktrace", >> "sym-userobj", >> + "printk-msg-only", > > Hmm, I'm a bit apprehensive about this approach. We are adding a lot of > options that I'm not sure will be used. Is this needed because you added a > ftrace printk that you want to not have the header, or because there are > ftrace printks that exist that you want to turn on and off. Or perhaps, > you want to see the header sometimes and sometimes you do not? > > If it is the later two, then sure we could have this option. But if it is > the former, then we probably should have another type of ftrace_printk > that takes an option to make another event type to tell the output not to > print? > > Oh well, I have patches in mind to clean up the output code anyway. > > -- Steve >
Actually, it's not for my needs but Ingo requested it to have only the message output without the headers while debugging with ftrace_printk. --
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [thread] [
date
] [
author
]
Messages in current thread:
[PATCH] tracing/ftrace: add the printk-msg-only option
, Frederic Weisbecker
, (Sat Dec 13, 12:18 pm)
Re: [PATCH] tracing/ftrace: add the printk-msg-only option
, Steven Rostedt
, (Sun Dec 14, 10:04 am)
Re: [PATCH] tracing/ftrace: add the printk-msg-only option
, =?ISO-8859-1?Q?Fr=E9 ...
, (Sun Dec 14, 10:52 am)
Navigation
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Paul Turner
[tg_shares_up rewrite v4 11/11] sched: update tg->shares after cpu.shares write
Matthew Garrett
Re: [PATCH] Enable speedstep for sonoma processors.
Mauro Carvalho Chehab
Re: [PATCH 1/2] media: Add timberdale video-in driver
Peter Zijlstra
[PATCH 23/30] netvm: skb processing
Greg Kroah-Hartman
[PATCH 21/28] cgroupfs: create /sys/fs/cgroup to mount cgroupfs on
git
:
Jan Hudec
Re: GIT push to sftp (feature request)
Steffen Prohaska
[PATCH 0/4] core.ignorecase
Johannes Schindelin
Re: Git checkout preserve timestamp?
Linus Torvalds
[PATCH 1/7] Make unpack_trees_options bit flags actual bitfields
Johan Herland
Re: What's cooking in git.git (Oct 2010, #01; Wed, 13)
linux-netdev
:
David Miller
Re: [PATCH 1/3] f_phonet: dev_kfree_skb instead of dev_kfree_skb_any in TX callback
Richard Cochran
Re: [PATCH v3 3/3] ptp: Added a clock that uses the eTSEC found on the MPC85xx.
Jan Engelhardt
Re: [PATCH] Fix netfilter xt_time's time_mt()'s use of do_div()
Herbert Xu
Re: [RFC PATCH 00/17] virtual-bus
Jeff Kirsher
Re: [net-next-2.6 PATCH] e1000e: don't inadvertently re-set INTX_DISABLE
git-commits-head
:
Linux Kernel Mailing List
ALSA: hda - Enable beep on Realtek codecs with PCI SSID override
Linux Kernel Mailing List
Use path_put() in a few places instead of {mnt,d}put()
Linux Kernel Mailing List
mv643xx_eth: use sw csum for big packets
Linux Kernel Mailing List
arm: fix HAVE_CLK merge goof
Linux Kernel Mailing List
arm: convert pcm037 platform to use smsc911x
freebsd-current
:
David Wolfskill
"interrupt storm..."; seems associated with an0 NIC
Andriy Gapon
Re: letting glabel recognise a media change
Garrett Cooper
Re: Only display ACPI bootmenu key if ACPI is present
Pyun YongHyeon
CFT: msk(4) Rx checksum offloading support
FreeBSD Tinderbox
[head tinderbox] failure on sparc64/sparc64
Colocation donated by:
Syndicate