[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4E2031D8.3090902@hitachi.com>
Date: Fri, 15 Jul 2011 21:26:00 +0900
From: Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>
To: Steven Rostedt <rostedt@...dmis.org>
Cc: Arnaldo Carvalho de Melo <acme@...hat.com>,
Ingo Molnar <mingo@...e.hu>,
Frederic Weisbecker <fweisbec@...il.com>,
Peter Zijlstra <peterz@...radead.org>,
linux-kernel@...r.kernel.org, yrl.pp-manager.tt@...achi.com
Subject: Re: [PATCH -tip 00/13]tracing/kprobes: Dynamic events on module
support
(2011/07/15 20:28), Steven Rostedt wrote:
> On Mon, 2011-06-27 at 16:26 +0900, Masami Hiramatsu wrote:
>> Hi Steven,
>>
>> Here, I'd like to push updates of dynamic events and perf-probe
>> for supporting dynamic events on module.
>>
>> The main part of this series is 1-5th patches for kprobes and
>> dynamic event tracer, which allows us to add events on offline
>> kernel module. This means we can put events even on the init
>> functions of the drivers before loading. The requirement of
>> this feature comes from my recent experience of a driver
>> debugging. It had a timing bug in the initialization routine,
>> and it was hard to catch them without tracing the behavior.
>>
>> I added some perf-probe updates (which including some resent
>> patches), And it's OK to pull only the main part (kprobes and
>> ftrace patches) without perf updates.
>>
>
> OK I pulled in everything but patch 10, as that seemed to have an issue
> with the message that David Ahern brought up. I left that one out as it
> really wasn't necessary for the other patches. If you want to just
> resend that one patch with an update, I can pull that one in too.
Thanks, and patch 10, I've found that that actually fixed a bug
of perf probe itself and have sent it to Ingo as another bugfix,
so feel free to drop it from your tree.
Thank you,
>
> -- Steve
>
>>
>> Thank you,
>>
>> ---
>>
>> Masami Hiramatsu (13):
>> perf probe: Support adding probes on offline kernel modules
>> perf probe: Add probed module in front of function
>> [RESEND]perf probe: Introduce debuginfo to encapsulate dwarf information
>> [RESEND]perf probe: Warn when more than two lines are given
>> [RESEND][CLEANUP]perf-probe: Move dwarf library routines to dwarf-aux.{c,h}
>> [RESEND][CLEANUP]perf probe: Remove redundant dwarf functions
>> [RESEND][CLEANUP]perf probe: Move strtailcmp to string.c
>> [RESEND][CLEANUP]perf probe: Rename DIE_FIND_CB_FOUND to DIE_FIND_CB_END
>> tracing/kprobe: Update symbol reference when loading module
>> tracing/kprobes: Support module init function probing
>> kprobes: Return -ENOENT if probe point doesn't exist
>> [CLEANUP]tracing/kprobes: merge trace probe enable/disable functions
>> [CLEANUP]tracing/kprobes: Rename probe_* to trace_probe_*
>>
>>
>> Documentation/trace/kprobetrace.txt | 9
>> kernel/kprobes.c | 33 +
>> kernel/trace/trace_kprobe.c | 314 +++++++++----
>> tools/perf/Documentation/perf-probe.txt | 6
>> tools/perf/Makefile | 2
>> tools/perf/builtin-probe.c | 14 -
>> tools/perf/util/dwarf-aux.c | 663 +++++++++++++++++++++++++++
>> tools/perf/util/dwarf-aux.h | 100 ++++
>> tools/perf/util/probe-event.c | 165 +++++--
>> tools/perf/util/probe-event.h | 1
>> tools/perf/util/probe-finder.c | 752 ++++---------------------------
>> tools/perf/util/probe-finder.h | 43 +-
>> tools/perf/util/string.c | 19 +
>> tools/perf/util/util.h | 1
>> 14 files changed, 1304 insertions(+), 818 deletions(-)
>> create mode 100644 tools/perf/util/dwarf-aux.c
>> create mode 100644 tools/perf/util/dwarf-aux.h
>>
>
>
> --
> 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/
--
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu.pt@...achi.com
--
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/
Powered by blists - more mailing lists