Commit a8d154b0 authored by Steven Rostedt's avatar Steven Rostedt Committed by Steven Rostedt

tracing: create automated trace defines

This patch lowers the number of places a developer must modify to add
new tracepoints. The current method to add a new tracepoint
into an existing system is to write the trace point macro in the
trace header with one of the macros TRACE_EVENT, TRACE_FORMAT or
DECLARE_TRACE, then they must add the same named item into the C file
with the macro DEFINE_TRACE(name) and then add the trace point.

This change cuts out the needing to add the DEFINE_TRACE(name).
Every file that uses the tracepoint must still include the trace/<type>.h
file, but the one C file must also add a define before the including
of that file.

 #define CREATE_TRACE_POINTS
 #include <trace/mytrace.h>

This will cause the trace/mytrace.h file to also produce the C code
necessary to implement the trace point.

Note, if more than one trace/<type>.h is used to create the C code
it is best to list them all together.

 #define CREATE_TRACE_POINTS
 #include <trace/foo.h>
 #include <trace/bar.h>
 #include <trace/fido.h>

Thanks to Mathieu Desnoyers and Christoph Hellwig for coming up with
the cleaner solution of the define above the includes over my first
design to have the C code include a "special" header.

This patch converts sched, irq and lockdep and skb to use this new
method.

Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Neil Horman <nhorman@tuxdriver.com>
Cc: Zhao Lei <zhaolei@cn.fujitsu.com>
Cc: Eduard - Gabriel Munteanu <eduard.munteanu@linux360.ro>
Cc: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: default avatarSteven Rostedt <rostedt@goodmis.org>
parent ea20d929
/*
* Trace files that want to automate creationg of all tracepoints defined
* in their file should include this file. The following are macros that the
* trace file may define:
*
* TRACE_SYSTEM defines the system the tracepoint is for
*
* TRACE_INCLUDE_FILE if the file name is something other than TRACE_SYSTEM.h
* This macro may be defined to tell define_trace.h what file to include.
* Note, leave off the ".h".
*
* TRACE_INCLUDE_PATH if the path is something other than core kernel include/trace
* then this macro can define the path to use. Note, the path is relative to
* define_trace.h, not the file including it. Full path names for out of tree
* modules must be used.
*/
#ifdef CREATE_TRACE_POINTS
/* Prevent recursion */
#undef CREATE_TRACE_POINTS
#include <linux/stringify.h>
#undef TRACE_EVENT
#define TRACE_EVENT(name, proto, args, tstruct, assign, print) \
DEFINE_TRACE(name)
#undef TRACE_FORMAT
#define TRACE_FORMAT(name, proto, args, print) \
DEFINE_TRACE(name)
#undef DECLARE_TRACE
#define DECLARE_TRACE(name, proto, args) \
DEFINE_TRACE(name)
#undef TRACE_INCLUDE
#undef __TRACE_INCLUDE
#ifndef TRACE_INCLUDE_FILE
# define TRACE_INCLUDE_FILE TRACE_SYSTEM
# define UNDEF_TRACE_INCLUDE_FILE
#endif
#ifndef TRACE_INCLUDE_PATH
# define __TRACE_INCLUDE(system) <trace/system.h>
# define UNDEF_TRACE_INCLUDE_FILE
#else
# define __TRACE_INCLUDE(system) __stringify(TRACE_INCLUDE_PATH/system.h)
#endif
# define TRACE_INCLUDE(system) __TRACE_INCLUDE(system)
/* Let the trace headers be reread */
#define TRACE_HEADER_MULTI_READ
#include TRACE_INCLUDE(TRACE_INCLUDE_FILE)
#undef TRACE_HEADER_MULTI_READ
/* Only undef what we defined in this file */
#ifdef UNDEF_TRACE_INCLUDE_FILE
# undef TRACE_INCLUDE_PATH
# undef UNDEF_TRACE_INCLUDE_FILE
#endif
#ifdef UNDEF_TRACE_INCLUDE_FILE
# undef TRACE_INCLUDE_PATH
# undef UNDEF_TRACE_INCLUDE_FILE
#endif
/* We may be processing more files */
#define CREATE_TRACE_POINTS
#endif /* CREATE_TRACE_POINTS */
...@@ -51,4 +51,7 @@ TRACE_FORMAT(softirq_exit, ...@@ -51,4 +51,7 @@ TRACE_FORMAT(softirq_exit,
TP_FMT("softirq=%d action=%s", (int)(h - vec), softirq_to_name[h-vec]) TP_FMT("softirq=%d action=%s", (int)(h - vec), softirq_to_name[h-vec])
); );
#endif #endif /* _TRACE_IRQ_H */
/* This part must be outside protection */
#include <trace/define_trace.h>
...@@ -188,5 +188,7 @@ TRACE_EVENT(kmem_cache_free, ...@@ -188,5 +188,7 @@ TRACE_EVENT(kmem_cache_free,
TP_printk("call_site=%lx ptr=%p", __entry->call_site, __entry->ptr) TP_printk("call_site=%lx ptr=%p", __entry->call_site, __entry->ptr)
); );
#endif /* _TRACE_KMEM_H */
#endif /* This part must be outside protection */
#include <trace/define_trace.h>
...@@ -55,3 +55,6 @@ TRACE_EVENT(lock_acquired, ...@@ -55,3 +55,6 @@ TRACE_EVENT(lock_acquired,
#endif #endif
#endif /* _TRACE_LOCKDEP_H */ #endif /* _TRACE_LOCKDEP_H */
/* This part must be outside protection */
#include <trace/define_trace.h>
...@@ -334,3 +334,6 @@ TRACE_EVENT(sched_signal_send, ...@@ -334,3 +334,6 @@ TRACE_EVENT(sched_signal_send,
); );
#endif /* _TRACE_SCHED_H */ #endif /* _TRACE_SCHED_H */
/* This part must be outside protection */
#include <trace/define_trace.h>
...@@ -35,3 +35,6 @@ TRACE_EVENT(kfree_skb, ...@@ -35,3 +35,6 @@ TRACE_EVENT(kfree_skb,
); );
#endif /* _TRACE_SKB_H */ #endif /* _TRACE_SKB_H */
/* This part must be outside protection */
#include <trace/define_trace.h>
...@@ -56,10 +56,6 @@ ...@@ -56,10 +56,6 @@
#include <asm/mmu_context.h> #include <asm/mmu_context.h>
#include "cred-internals.h" #include "cred-internals.h"
DEFINE_TRACE(sched_process_free);
DEFINE_TRACE(sched_process_exit);
DEFINE_TRACE(sched_process_wait);
static void exit_mm(struct task_struct * tsk); static void exit_mm(struct task_struct * tsk);
static void __unhash_process(struct task_struct *p) static void __unhash_process(struct task_struct *p)
......
...@@ -83,8 +83,6 @@ DEFINE_PER_CPU(unsigned long, process_counts) = 0; ...@@ -83,8 +83,6 @@ DEFINE_PER_CPU(unsigned long, process_counts) = 0;
__cacheline_aligned DEFINE_RWLOCK(tasklist_lock); /* outer */ __cacheline_aligned DEFINE_RWLOCK(tasklist_lock); /* outer */
DEFINE_TRACE(sched_process_fork);
int nr_processes(void) int nr_processes(void)
{ {
int cpu; int cpu;
......
...@@ -17,9 +17,11 @@ ...@@ -17,9 +17,11 @@
#include <linux/kernel_stat.h> #include <linux/kernel_stat.h>
#include <linux/rculist.h> #include <linux/rculist.h>
#include <linux/hash.h> #include <linux/hash.h>
#include <trace/irq.h>
#include <linux/bootmem.h> #include <linux/bootmem.h>
#define CREATE_TRACE_POINTS
#include <trace/irq.h>
#include "internals.h" #include "internals.h"
/* /*
...@@ -348,9 +350,6 @@ static void warn_no_thread(unsigned int irq, struct irqaction *action) ...@@ -348,9 +350,6 @@ static void warn_no_thread(unsigned int irq, struct irqaction *action)
"but no thread function available.", irq, action->name); "but no thread function available.", irq, action->name);
} }
DEFINE_TRACE(irq_handler_entry);
DEFINE_TRACE(irq_handler_exit);
/** /**
* handle_IRQ_event - irq action chain handler * handle_IRQ_event - irq action chain handler
* @irq: the interrupt number * @irq: the interrupt number
......
...@@ -21,9 +21,6 @@ static DEFINE_SPINLOCK(kthread_create_lock); ...@@ -21,9 +21,6 @@ static DEFINE_SPINLOCK(kthread_create_lock);
static LIST_HEAD(kthread_create_list); static LIST_HEAD(kthread_create_list);
struct task_struct *kthreadd_task; struct task_struct *kthreadd_task;
DEFINE_TRACE(sched_kthread_stop);
DEFINE_TRACE(sched_kthread_stop_ret);
struct kthread_create_info struct kthread_create_info
{ {
/* Information passed to kthread() from kthreadd. */ /* Information passed to kthread() from kthreadd. */
......
...@@ -42,12 +42,14 @@ ...@@ -42,12 +42,14 @@
#include <linux/hash.h> #include <linux/hash.h>
#include <linux/ftrace.h> #include <linux/ftrace.h>
#include <linux/stringify.h> #include <linux/stringify.h>
#include <trace/lockdep.h>
#include <asm/sections.h> #include <asm/sections.h>
#include "lockdep_internals.h" #include "lockdep_internals.h"
#define CREATE_TRACE_POINTS
#include <trace/lockdep.h>
#ifdef CONFIG_PROVE_LOCKING #ifdef CONFIG_PROVE_LOCKING
int prove_locking = 1; int prove_locking = 1;
module_param(prove_locking, int, 0644); module_param(prove_locking, int, 0644);
...@@ -2929,8 +2931,6 @@ void lock_set_class(struct lockdep_map *lock, const char *name, ...@@ -2929,8 +2931,6 @@ void lock_set_class(struct lockdep_map *lock, const char *name,
} }
EXPORT_SYMBOL_GPL(lock_set_class); EXPORT_SYMBOL_GPL(lock_set_class);
DEFINE_TRACE(lock_acquire);
/* /*
* We are not always called with irqs disabled - do that here, * We are not always called with irqs disabled - do that here,
* and also avoid lockdep recursion: * and also avoid lockdep recursion:
...@@ -2957,8 +2957,6 @@ void lock_acquire(struct lockdep_map *lock, unsigned int subclass, ...@@ -2957,8 +2957,6 @@ void lock_acquire(struct lockdep_map *lock, unsigned int subclass,
} }
EXPORT_SYMBOL_GPL(lock_acquire); EXPORT_SYMBOL_GPL(lock_acquire);
DEFINE_TRACE(lock_release);
void lock_release(struct lockdep_map *lock, int nested, void lock_release(struct lockdep_map *lock, int nested,
unsigned long ip) unsigned long ip)
{ {
...@@ -3061,8 +3059,6 @@ __lock_contended(struct lockdep_map *lock, unsigned long ip) ...@@ -3061,8 +3059,6 @@ __lock_contended(struct lockdep_map *lock, unsigned long ip)
put_lock_stats(stats); put_lock_stats(stats);
} }
DEFINE_TRACE(lock_acquired);
static void static void
__lock_acquired(struct lockdep_map *lock, unsigned long ip) __lock_acquired(struct lockdep_map *lock, unsigned long ip)
{ {
...@@ -3118,8 +3114,6 @@ __lock_acquired(struct lockdep_map *lock, unsigned long ip) ...@@ -3118,8 +3114,6 @@ __lock_acquired(struct lockdep_map *lock, unsigned long ip)
lock->ip = ip; lock->ip = ip;
} }
DEFINE_TRACE(lock_contended);
void lock_contended(struct lockdep_map *lock, unsigned long ip) void lock_contended(struct lockdep_map *lock, unsigned long ip)
{ {
unsigned long flags; unsigned long flags;
......
...@@ -72,13 +72,15 @@ ...@@ -72,13 +72,15 @@
#include <linux/debugfs.h> #include <linux/debugfs.h>
#include <linux/ctype.h> #include <linux/ctype.h>
#include <linux/ftrace.h> #include <linux/ftrace.h>
#include <trace/sched.h>
#include <asm/tlb.h> #include <asm/tlb.h>
#include <asm/irq_regs.h> #include <asm/irq_regs.h>
#include "sched_cpupri.h" #include "sched_cpupri.h"
#define CREATE_TRACE_POINTS
#include <trace/sched.h>
/* /*
* Convert user-nice values [ -20 ... 0 ... 19 ] * Convert user-nice values [ -20 ... 0 ... 19 ]
* to static priority [ MAX_RT_PRIO..MAX_PRIO-1 ], * to static priority [ MAX_RT_PRIO..MAX_PRIO-1 ],
...@@ -118,12 +120,6 @@ ...@@ -118,12 +120,6 @@
*/ */
#define RUNTIME_INF ((u64)~0ULL) #define RUNTIME_INF ((u64)~0ULL)
DEFINE_TRACE(sched_wait_task);
DEFINE_TRACE(sched_wakeup);
DEFINE_TRACE(sched_wakeup_new);
DEFINE_TRACE(sched_switch);
DEFINE_TRACE(sched_migrate_task);
#ifdef CONFIG_SMP #ifdef CONFIG_SMP
static void double_rq_lock(struct rq *rq1, struct rq *rq2); static void double_rq_lock(struct rq *rq1, struct rq *rq2);
......
...@@ -41,8 +41,6 @@ ...@@ -41,8 +41,6 @@
static struct kmem_cache *sigqueue_cachep; static struct kmem_cache *sigqueue_cachep;
DEFINE_TRACE(sched_signal_send);
static void __user *sig_handler(struct task_struct *t, int sig) static void __user *sig_handler(struct task_struct *t, int sig)
{ {
return t->sighand->action[sig - 1].sa.sa_handler; return t->sighand->action[sig - 1].sa.sa_handler;
......
...@@ -186,9 +186,6 @@ EXPORT_SYMBOL(local_bh_enable_ip); ...@@ -186,9 +186,6 @@ EXPORT_SYMBOL(local_bh_enable_ip);
*/ */
#define MAX_SOFTIRQ_RESTART 10 #define MAX_SOFTIRQ_RESTART 10
DEFINE_TRACE(softirq_entry);
DEFINE_TRACE(softirq_exit);
asmlinkage void __do_softirq(void) asmlinkage void __do_softirq(void)
{ {
struct softirq_action *h; struct softirq_action *h;
......
...@@ -4,9 +4,11 @@ ...@@ -4,9 +4,11 @@
#include <linux/module.h> #include <linux/module.h>
#include <linux/err.h> #include <linux/err.h>
#include <linux/sched.h> #include <linux/sched.h>
#include <linux/tracepoint.h>
#include <asm/uaccess.h> #include <asm/uaccess.h>
#define CREATE_TRACE_POINTS
#include <trace/kmem.h>
/** /**
* kstrdup - allocate space for and copy an existing string * kstrdup - allocate space for and copy an existing string
* @s: the string to duplicate * @s: the string to duplicate
...@@ -239,13 +241,6 @@ int __attribute__((weak)) get_user_pages_fast(unsigned long start, ...@@ -239,13 +241,6 @@ int __attribute__((weak)) get_user_pages_fast(unsigned long start,
EXPORT_SYMBOL_GPL(get_user_pages_fast); EXPORT_SYMBOL_GPL(get_user_pages_fast);
/* Tracepoints definitions. */ /* Tracepoints definitions. */
DEFINE_TRACE(kmalloc);
DEFINE_TRACE(kmem_cache_alloc);
DEFINE_TRACE(kmalloc_node);
DEFINE_TRACE(kmem_cache_alloc_node);
DEFINE_TRACE(kfree);
DEFINE_TRACE(kmem_cache_free);
EXPORT_TRACEPOINT_SYMBOL(kmalloc); EXPORT_TRACEPOINT_SYMBOL(kmalloc);
EXPORT_TRACEPOINT_SYMBOL(kmem_cache_alloc); EXPORT_TRACEPOINT_SYMBOL(kmem_cache_alloc);
EXPORT_TRACEPOINT_SYMBOL(kmalloc_node); EXPORT_TRACEPOINT_SYMBOL(kmalloc_node);
......
...@@ -19,11 +19,11 @@ ...@@ -19,11 +19,11 @@
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <linux/netlink.h> #include <linux/netlink.h>
#include <linux/net_dropmon.h> #include <linux/net_dropmon.h>
#include <trace/skb.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <asm/bitops.h> #include <asm/bitops.h>
#define CREATE_TRACE_POINTS
#include <trace/skb.h>
DEFINE_TRACE(kfree_skb);
EXPORT_TRACEPOINT_SYMBOL_GPL(kfree_skb); EXPORT_TRACEPOINT_SYMBOL_GPL(kfree_skb);
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment