Re: [PATCH 0/3] tracing/function-graph-tracer: signal interrupt triggeringon output

From: Wang Liming
Date: Wed Dec 10 2008 - 04:08:00 EST


FrÃdÃric Weisbecker wrote:
2008/12/10 Steven Rostedt <rostedt@xxxxxxxxxxx>:
On Tue, 9 Dec 2008, Frederic Weisbecker wrote:

This patchset adds all the features to draw hardirq triggering on output
for the function graph tracer.
An example is given on the third patch.

Just one question, on recordmcount.pl:

my %text_sections = (
".text" => 1,
".sched.text" => 1,
".spinlock.text" => 1,

The .sched.text section is included inside .text, it's a subsection, I'm not sure
it's necessary to add it in text_sections. In doubt I added the new .irqentry.text

Matters what objdump -dr shows.

$ objdump -dr kernel/sched.o | grep 'Disassembly'
Disassembly of section .text:
Disassembly of section .cpuinit.text:
Disassembly of section .init.text:
Disassembly of section .sched.text:

Yep, .sched.text is needed. It is not about what gets linked by the
linker. It is about what objdump shows, which is what recordmcount.pl uses
to determine whether or not to record the mcount callers in that section.


Ok. I looked at a disassembly objdump on vmlinux to find those
subsections and didn't find it.
But I didn't see that recordmcount did its work on each build folder,
You can see "scripts/Makefile.build" file, which uses recordmcount.pl to deal with each .o file when building.

Liming Wang
I guess that these sections are
visible on the objects found in the subdirectories....

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


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