This is the mail archive of the
systemtap@sourceware.org
mailing list for the systemtap project.
Re: [PATCH -tip v8 03/26] kprobes: Prohibit probing on .entry.text code
- From: Steven Rostedt <rostedt at goodmis dot org>
- To: Masami Hiramatsu <masami dot hiramatsu dot pt at hitachi dot com>
- Cc: linux-kernel at vger dot kernel dot org, Ingo Molnar <mingo at kernel dot org>, Andi Kleen <andi at firstfloor dot org>, Ananth N Mavinakayanahalli <ananth at in dot ibm dot com>, Peter Zijlstra <peterz at infradead dot org>, Frederic Weisbecker <fweisbec at gmail dot com>, x86 at kernel dot org, Sandeepa Prabhu <sandeepa dot prabhu at linaro dot org>, fche at redhat dot com, mingo at redhat dot com, Al Viro <viro at zeniv dot linux dot org dot uk>, systemtap at sourceware dot org, "H. Peter Anvin" <hpa at zytor dot com>, Thomas Gleixner <tglx at linutronix dot de>, Seiji Aguchi <seiji dot aguchi at hds dot com>
- Date: Mon, 24 Mar 2014 14:53:51 -0400
- Subject: Re: [PATCH -tip v8 03/26] kprobes: Prohibit probing on .entry.text code
- Authentication-results: sourceware.org; auth=none
- References: <20140305115843 dot 22766 dot 8355 dot stgit at ltc230 dot yrl dot intra dot hitachi dot co dot jp> <20140305115904 dot 22766 dot 57607 dot stgit at ltc230 dot yrl dot intra dot hitachi dot co dot jp> <20140321180400 dot 0c1b2969 at gandalf dot local dot home> <532F8ED2 dot 2030704 at hitachi dot com>
On Mon, 24 Mar 2014 10:48:02 +0900
Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> wrote:
> (2014/03/22 7:04), Steven Rostedt wrote:
> > On Wed, 05 Mar 2014 20:59:04 +0900
> > Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> wrote:
> >
> >
> >> struct kprobe_insn_cache {
> >> struct mutex mutex;
> >> diff --git a/kernel/kprobes.c b/kernel/kprobes.c
> >> index ceeadfc..5b5ac76 100644
> >> --- a/kernel/kprobes.c
> >> +++ b/kernel/kprobes.c
> >> @@ -96,9 +96,6 @@ static raw_spinlock_t *kretprobe_table_lock_ptr(unsigned long hash)
> >> static struct kprobe_blackpoint kprobe_blacklist[] = {
> >> {"preempt_schedule",},
> >> {"native_get_debugreg",},
> >> - {"irq_entries_start",},
> >> - {"common_interrupt",},
> >> - {"mcount",}, /* mcount can be called from everywhere */
> >
> > Is mcount in the entry.text section? Also, what about ftrace_caller and
> > friends.
>
> I saw mcount(and ftrace_caller) in arch/x86/kernel/entry_*.S, and
> almost all code in it are in ".entry.text" section as far as I can see,
> or do I miss something?
>
Ah, checking the System.map I guess we are OK.
-- Steve