Re: [PATCH] tracing/function-return-tracer: set a more human readable output

view
thread

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

From: Ingo Molnar

Subject: Re: [PATCH] tracing/function-return-tracer: set a more human readable output

Date: Wednesday, November 26, 2008 - 4:38 am

* Frédéric Weisbecker <fweisbec@gmail.com> wrote:

quoted text> 2008/11/26 Ingo Molnar <mingo@elte.hu>:
> >
> > the regex filter doesnt work well yet when used with the graph tracer:
> >
> > [ 2994.079079] ------------[ cut here ]------------
> > [ 2994.079079] WARNING: at kernel/trace/ftrace.c:373 ftrace_bug+0x8c/0x1ce()
> > [ 2994.079079] Modules linked in:
> > [ 2994.079079] Pid: 12, comm: kstop/0 Not tainted 2.6.28-rc6-tip #2056
> > [ 2994.079079] Call Trace:
> > [ 2994.079079]  [<c02273dd>] warn_on_slowpath+0x46/0x60
> > [ 2994.079079]  [<c0268308>] ? trace_graph_return+0x89/0xa7
> > [ 2994.079079]  [<c0268308>] ? trace_graph_return+0x89/0xa7
> > [ 2994.079079]  [<c0213e85>] ? ftrace_return_to_handler+0x6b/0x74
> > [ 2994.079079]  [<c0357f66>] ? memcpy+0xe/0x31
> > [ 2994.079079]  [<c02633e6>] ? rb_commit+0x29/0x2d
> > [ 2994.079079]  [<c0264c8b>] ? ring_buffer_unlock_commit+0x15/0x1a
> > [ 2994.079079]  [<c0213ece>] ? ftrace_modify_code+0x21/0xa3
> > [ 2994.079079]  [<c0213e85>] ? ftrace_return_to_handler+0x6b/0x74
> > [ 2994.079079]  [<c026c3ce>] ? probe_kernel_read+0xe/0x5c
> > [ 2994.079079]  [<c0261ec7>] ? trace_selftest_dynamic_test_func+0x3/0xc
> > [ 2994.079079]  [<c0203926>] ? return_to_handler+0xa/0x14
> > [ 2994.079079]  [<c0261ec7>] ? trace_selftest_dynamic_test_func+0x3/0xc
> > [ 2994.079079]  [<c0213ee5>] ? ftrace_modify_code+0x38/0xa3
> > [ 2994.079079]  [<c0261ec7>] ? trace_selftest_dynamic_test_func+0x3/0xc
> > [ 2994.079079]  [<c0262749>] ftrace_bug+0x8c/0x1ce
> > [ 2994.079079]  [<c026299c>] ftrace_replace_code+0x111/0x13d
> > [ 2994.079079]  [<c0253b53>] ? stop_cpu+0x0/0xa5
> > [ 2994.079079]  [<c02629e6>] __ftrace_modify_code+0x1e/0x32
> > [ 2994.079079]  [<c0253bbe>] stop_cpu+0x6b/0xa5
> > [ 2994.079079]  [<c020391c>] return_to_handler+0x0/0x14
> > [ 2994.079079]  [<c020391c>] return_to_handler+0x0/0x14
> > [ 2994.079079]  [<c0237fc1>] ? autoremove_wake_function+0x0/0x35
> > [ 2994.079079]  [<c023584b>] ? worker_thread+0x0/0xc7
> > [ 2994.079079]  [<c0237eba>] kthread+0x40/0x66
> > [ 2994.079079]  [<c0237e7a>] ? kthread+0x0/0x66
> > [ 2994.079079]  [<c02038cb>] kernel_thread_helper+0x7/0x10
> > [ 2994.079079] ---[ end trace bff97d149b80ecde ]---
> > [ 2994.079079] ftrace failed to modify [<c0261ec7>] trace_selftest_dynamic_test_func+0x3/0xc
> > [ 2994.079079]  actual: e8:0c:1a:fa:ff
> >
> > produced via:
> >
> >  # cd /debug/tracing/
> >  # cat set_ftrace_filter
> >  # echo schedule > set_ftrace_filter
> >  # cat set_ftrace_filter
> >  cat: set_ftrace_filter: No such device
> 
> 
> I didn't met this problem by the past with dynamic filtering on 
> return tracer. Did it occur only with schedule ? Do the last patches 
> from Steve correct the issue?

yep. Will re-report it if it occurs again.

	Ingo
--
unsubscribe noticeTo 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/function-return-tracer: set a more human r ..., Frederic Weisbecker, (Tue Nov 25, 4:57 pm)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Ingo Molnar, (Tue Nov 25, 5:39 pm)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Ingo Molnar, (Tue Nov 25, 7:04 pm)

[PATCH] tracing: function graph tracer, fix, Ingo Molnar, (Tue Nov 25, 7:10 pm)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Ingo Molnar, (Tue Nov 25, 7:12 pm)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Ingo Molnar, (Tue Nov 25, 7:14 pm)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Steven Rostedt, (Tue Nov 25, 7:19 pm)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Steven Rostedt, (Tue Nov 25, 7:21 pm)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Ingo Molnar, (Tue Nov 25, 7:42 pm)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Steven Rostedt, (Tue Nov 25, 7:51 pm)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Frédéric Weisbecker, (Wed Nov 26, 4:11 am)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Frédéric Weisbecker, (Wed Nov 26, 4:19 am)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Frédéric Weisbecker, (Wed Nov 26, 4:32 am)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Ingo Molnar, (Wed Nov 26, 4:38 am)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Steven Rostedt, (Wed Nov 26, 5:21 am)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Frédéric Weisbecker, (Wed Nov 26, 6:22 am)

Re: [PATCH] tracing/function-return-tracer: set a more hum ..., Steven Rostedt, (Wed Nov 26, 10:09 am)

Navigation

Popular discussions


linux-kernel:
Frederic Weisbecker	[PATCH v2] struct sort_entry has a callback named snprintf that turns an entry int...
FUJITA Tomonori	Re: [Scst-devel] Integration of SCST in the mainstream Linux kernel
Andrew Morton	Re: [PATCH v3 0/4] Introduce hardware spinlock framework
Ingo Molnar	Re: [GIT PULL] time.c - respin
Jens Axboe	Re: [BUG] Linux 2.6.25-rc2 - Regression from 2.6.24-rc1-git1 softlockup while bo...
git:
Mark Junker	git on MacOSX and files with decomposed utf-8 file names
Junio C Hamano	Re: git-svnimport
Johannes Schindelin	Re: [PATCH] Fix approxidate("never") to always return 0
Michal Sojka	[PATCHv5 1/2] filter-branch: Fix to allow replacing submodules with another content
A Large Angry SCM	Re: [RFC] origin link for cherry-pick and revert
linux-netdev:
Arnaldo Carvalho de Melo	Re: [PATCH 06/37] dccp: Limit feature negotiation to connection setup phase
Gerrit Renker	[PATCH 1/5] dccp: Initialisation framework for feature negotiation
Ursula Braun	[patch 2/8] [PATCH] af_iucv: sync sk shutdown flag if iucv path is quiesced
Daniel Lezcano	getsockopt(TCP_DEFER_ACCEPT) value change
David Miller	Re: 2.6.27.18: bnx2/tg3: BUG: "scheduling while atomic" trying to ifenslave a seco...
git-commits-head:
Linux Kernel Mailing List	sata_mv: drop unncessary EH callback resetting
Linux Kernel Mailing List	ath9k_htc: Allocate URBs properly
Linux Kernel Mailing List	ARM: S3C64XX: DMA: Callback with correct buffer pointer
Linux Kernel Mailing List	timer: Try to survive timer callback preempt_count leak
Linux Kernel Mailing List	powerpc/kexec: Add support for FSL-BookE
openbsd-misc:
Rene Maroufi	smtpd: Aliases only work with for local alias aliases
Stephen J. Bevan	GRE over IPsec
Christophe Rioux	Implementation example of snmp
Darrin Chandler	Re: strange output on openbsd C code
Nick Holland	Re: booting openbsd on eee without cd-rom

Colocation donated by:

Syndicate