Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932855AbaDVOyQ (ORCPT ); Tue, 22 Apr 2014 10:54:16 -0400 Received: from fw-tnat.austin.arm.com ([217.140.110.23]:58047 "EHLO collaborate-mta1.arm.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932661AbaDVOyK (ORCPT ); Tue, 22 Apr 2014 10:54:10 -0400 Date: Tue, 22 Apr 2014 07:53:58 -0700 From: Drew Richardson To: Ingo Molnar Cc: Ingo Molnar , Peter Zijlstra , "linux-kernel@vger.kernel.org" , Pawel Moll Subject: Re: [PATCH] Export sched tracepoints for use by kernel modules Message-ID: <20140422145338.GA4591@dreric01-Precision-T1600> References: <20140422142313.GB21121@dreric01-Precision-T1600> <20140422144503.GA15892@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140422144503.GA15892@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Apr 22, 2014 at 03:45:04PM +0100, Ingo Molnar wrote: > > * Drew Richardson wrote: > > > After commit de7b2973903c6cc50b31ee5682a69b2219b9919d ("tracepoint: > > Use struct pointer instead of name hash for reg/unreg tracepoints"), > > any tracepoints used in a kernel module must be exported. > > > > Signed-off-by: Drew Richardson > > Acked-by: Pawel Moll > > --- > > kernel/sched/core.c | 25 +++++++++++++++++++++++++ > > 1 file changed, 25 insertions(+) > > > > diff --git a/kernel/sched/core.c b/kernel/sched/core.c > > index 268a45ea238c..cdb8e92ff24a 100644 > > --- a/kernel/sched/core.c > > +++ b/kernel/sched/core.c > > @@ -90,6 +90,31 @@ > > #define CREATE_TRACE_POINTS > > #include > > > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_kthread_stop); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_kthread_stop_ret); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_wakeup); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_wakeup_new); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_switch); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_migrate_task); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_process_free); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_process_exit); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_wait_task); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_process_wait); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_process_fork); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_process_exec); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_stat_wait); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_stat_sleep); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_stat_iowait); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_stat_blocked); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_stat_runtime); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_pi_setprio); > > +#ifdef CONFIG_DETECT_HUNG_TASK > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_process_hang); > > +#endif /* CONFIG_DETECT_HUNG_TASK */ > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_move_numa); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_stick_numa); > > +EXPORT_TRACEPOINT_SYMBOL_GPL(sched_swap_numa); > > Which in-tree kernel module is using these? > > Thanks, > > Ingo > Gator (http://git.linaro.org/arm/ds5/gator.git), an out of tree kernel module, uses sched_process_fork, sched_switch and sched_process_free. Along with exporting those tracepoints I thought it would be helpful to export all the tracepoints from trace/events/sched.h. Thanks, Drew -- To 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/