From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 2053 invoked by alias); 17 Apr 2014 14:40:49 -0000 Mailing-List: contact systemtap-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: systemtap-owner@sourceware.org Received: (qmail 2042 invoked by uid 89); 17 Apr 2014 14:40:48 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.6 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.2 X-HELO: relay3-d.mail.gandi.net Received: from relay3-d.mail.gandi.net (HELO relay3-d.mail.gandi.net) (217.70.183.195) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Thu, 17 Apr 2014 14:40:47 +0000 Received: from mfilter32-d.gandi.net (mfilter32-d.gandi.net [217.70.178.163]) by relay3-d.mail.gandi.net (Postfix) with ESMTP id 53503A80EB; Thu, 17 Apr 2014 16:40:43 +0200 (CEST) Received: from relay3-d.mail.gandi.net ([217.70.183.195]) by mfilter32-d.gandi.net (mfilter32-d.gandi.net [10.0.15.180]) (amavisd-new, port 10024) with ESMTP id JIOODKb3UCo0; Thu, 17 Apr 2014 16:40:41 +0200 (CEST) Received: from leaf (static-50-43-32-211.bvtn.or.frontiernet.net [50.43.32.211]) (Authenticated sender: josh@joshtriplett.org) by relay3-d.mail.gandi.net (Postfix) with ESMTPSA id 2B360A8100; Thu, 17 Apr 2014 16:40:31 +0200 (CEST) Date: Thu, 17 Apr 2014 14:40:00 -0000 From: Josh Triplett To: Masami Hiramatsu Cc: linux-kernel@vger.kernel.org, Ingo Molnar , Andi Kleen , Ananth N Mavinakayanahalli , Sandeepa Prabhu , Frederic Weisbecker , x86@kernel.org, Steven Rostedt , fche@redhat.com, mingo@redhat.com, systemtap@sourceware.org, "H. Peter Anvin" , Thomas Gleixner , "Paul E. McKenney" Subject: Re: [PATCH -tip v9 17/26] notifier: Use NOKPROBE_SYMBOL macro in notifier Message-ID: <20140417144030.GA23089@leaf> References: <20140417081636.26341.87858.stgit@ltc230.yrl.intra.hitachi.co.jp> <20140417081835.26341.56128.stgit@ltc230.yrl.intra.hitachi.co.jp> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140417081835.26341.56128.stgit@ltc230.yrl.intra.hitachi.co.jp> User-Agent: Mutt/1.5.23 (2014-03-12) X-SW-Source: 2014-q2/txt/msg00072.txt.bz2 On Thu, Apr 17, 2014 at 05:18:35PM +0900, Masami Hiramatsu wrote: > Use NOKPROBE_SYMBOL macro to protect functions from > kprobes instead of __kprobes annotation in notifier. > > Signed-off-by: Masami Hiramatsu > Reviewed-by: Steven Rostedt > Cc: Josh Triplett > Cc: "Paul E. McKenney" Reviewed-by: Josh Triplett > kernel/notifier.c | 22 +++++++++++++--------- > 1 file changed, 13 insertions(+), 9 deletions(-) > > diff --git a/kernel/notifier.c b/kernel/notifier.c > index db4c8b0..4803da6 100644 > --- a/kernel/notifier.c > +++ b/kernel/notifier.c > @@ -71,9 +71,9 @@ static int notifier_chain_unregister(struct notifier_block **nl, > * @returns: notifier_call_chain returns the value returned by the > * last notifier function called. > */ > -static int __kprobes notifier_call_chain(struct notifier_block **nl, > - unsigned long val, void *v, > - int nr_to_call, int *nr_calls) > +static int notifier_call_chain(struct notifier_block **nl, > + unsigned long val, void *v, > + int nr_to_call, int *nr_calls) > { > int ret = NOTIFY_DONE; > struct notifier_block *nb, *next_nb; > @@ -102,6 +102,7 @@ static int __kprobes notifier_call_chain(struct notifier_block **nl, > } > return ret; > } > +NOKPROBE_SYMBOL(notifier_call_chain); > > /* > * Atomic notifier chain routines. Registration and unregistration > @@ -172,9 +173,9 @@ EXPORT_SYMBOL_GPL(atomic_notifier_chain_unregister); > * Otherwise the return value is the return value > * of the last notifier function called. > */ > -int __kprobes __atomic_notifier_call_chain(struct atomic_notifier_head *nh, > - unsigned long val, void *v, > - int nr_to_call, int *nr_calls) > +int __atomic_notifier_call_chain(struct atomic_notifier_head *nh, > + unsigned long val, void *v, > + int nr_to_call, int *nr_calls) > { > int ret; > > @@ -184,13 +185,15 @@ int __kprobes __atomic_notifier_call_chain(struct atomic_notifier_head *nh, > return ret; > } > EXPORT_SYMBOL_GPL(__atomic_notifier_call_chain); > +NOKPROBE_SYMBOL(__atomic_notifier_call_chain); > > -int __kprobes atomic_notifier_call_chain(struct atomic_notifier_head *nh, > - unsigned long val, void *v) > +int atomic_notifier_call_chain(struct atomic_notifier_head *nh, > + unsigned long val, void *v) > { > return __atomic_notifier_call_chain(nh, val, v, -1, NULL); > } > EXPORT_SYMBOL_GPL(atomic_notifier_call_chain); > +NOKPROBE_SYMBOL(atomic_notifier_call_chain); > > /* > * Blocking notifier chain routines. All access to the chain is > @@ -527,7 +530,7 @@ EXPORT_SYMBOL_GPL(srcu_init_notifier_head); > > static ATOMIC_NOTIFIER_HEAD(die_chain); > > -int notrace __kprobes notify_die(enum die_val val, const char *str, > +int notrace notify_die(enum die_val val, const char *str, > struct pt_regs *regs, long err, int trap, int sig) > { > struct die_args args = { > @@ -540,6 +543,7 @@ int notrace __kprobes notify_die(enum die_val val, const char *str, > }; > return atomic_notifier_call_chain(&die_chain, val, &args); > } > +NOKPROBE_SYMBOL(notify_die); > > int register_die_notifier(struct notifier_block *nb) > { > >