UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

tracing/ftrace: handle more than one stat file per tracer 

Impact: new API for tracers

Make the stat tracing API reentrant. And also provide the new directory
/debugfs/tracing/trace_stat which will contain all the stat files for the
current active tracer.

Now a tracer will, if desired, want to provide a zero terminated array of
tracer_stat structures.
Each one contains the callbacks necessary for one stat file.
It have to provide at least a name for its stat file, an iterator with
stat_start/start_next callback and an output callback for one stat entry.

Also adapt the branch tracer to this new API.
We create two files "all" and "annotated" inside the /debugfs/tracing/trace_stat
directory, making the both stats simultaneously available instead of needing
to change an option to switch from one stat file to another.

The output of these stats haven't changed.

Changes in v2:

_ Apply the previous memory leak fix (rebase against tip/master)

Changes in v3:

_ Merge the patch that adapted the branch tracer to this Api in this patch to
  not break the kernel build.

Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Signed-off-by: Steven Rostedt <srostedt@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
This commit is contained in:
Frederic Weisbecker 2009-01-08 10:03:56 -08:00 committed by Ingo Molnar
parent 67d347245f
commit 034939b65a
3 changed files with 217 additions and 117 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
kernel/trace/trace.h
View File

@ -334,6 +334,25 @@ struct tracer_flags {
/* Makes more easy to define a tracer opt */ /* Makes more easy to define a tracer opt */
#define TRACER_OPT(s, b) .name = #s, .bit = b #define TRACER_OPT(s, b) .name = #s, .bit = b
/*
* If you want to provide a stat file (one-shot statistics), fill
* an iterator with stat_start/stat_next and a stat_show callbacks.
* The others callbacks are optional.
*/
struct tracer_stat {
/* The name of your stat file */
const char *name;
/* Iteration over statistic entries */
void *(*stat_start)(void);
void *(*stat_next)(void *prev, int idx);
/* Compare two entries for sorting (optional) for stats */
int (*stat_cmp)(void *p1, void *p2);
/* Print a stat entry */
int (*stat_show)(struct seq_file *s, void *p);
/* Print the headers of your stat entries */
int (*stat_headers)(struct seq_file *s);
};
/* /*
* A specific tracer, represented by methods that operate on a trace array: * A specific tracer, represented by methods that operate on a trace array:
*/ */
@ -361,21 +380,7 @@ struct tracer {
struct tracer *next; struct tracer *next;
int print_max; int print_max;
struct tracer_flags *flags; struct tracer_flags *flags;
struct tracer_stat *stats;
/*
* If you change one of the following on tracing runtime, recall
* init_tracer_stat()
*/
/* Iteration over statistic entries */
void *(*stat_start)(void);
void *(*stat_next)(void *prev, int idx);
/* Compare two entries for sorting (optional) for stats */
int (*stat_cmp)(void *p1, void *p2);
/* Print a stat entry */
int (*stat_show)(struct seq_file *s, void *p);
/* Print the headers of your stat entries */
int (*stat_headers)(struct seq_file *s);
}; };
struct trace_seq { struct trace_seq {

69
kernel/trace/trace_branch.c
View File

@ -306,19 +306,6 @@ static int annotated_branch_stat_cmp(void *p1, void *p2)
} }
#ifdef CONFIG_PROFILE_ALL_BRANCHES #ifdef CONFIG_PROFILE_ALL_BRANCHES
enum {
TRACE_BRANCH_OPT_ALL = 0x1
};
static struct tracer_opt branch_opts[] = {
{ TRACER_OPT(stat_all_branch, TRACE_BRANCH_OPT_ALL) },
{ }
};
static struct tracer_flags branch_flags = {
.val = 0,
.opts = branch_opts
};
extern unsigned long __start_branch_profile[]; extern unsigned long __start_branch_profile[];
extern unsigned long __stop_branch_profile[]; extern unsigned long __stop_branch_profile[];
@ -352,28 +339,36 @@ all_branch_stat_next(void *v, int idx)
return p; return p;
} }
static int branch_set_flag(u32 old_flags, u32 bit, int set) static struct tracer_stat branch_stats[] = {
{ {.name = "annotated",
if (bit == TRACE_BRANCH_OPT_ALL) { .stat_start = annotated_branch_stat_start,
if (set) { .stat_next = annotated_branch_stat_next,
branch_trace.stat_headers = all_branch_stat_headers; .stat_cmp = annotated_branch_stat_cmp,
branch_trace.stat_start = all_branch_stat_start; .stat_headers = annotated_branch_stat_headers,
branch_trace.stat_next = all_branch_stat_next; .stat_show = branch_stat_show},
branch_trace.stat_cmp = NULL;
} else {
branch_trace.stat_headers =
annotated_branch_stat_headers;
branch_trace.stat_start = annotated_branch_stat_start;
branch_trace.stat_next = annotated_branch_stat_next;
branch_trace.stat_cmp = annotated_branch_stat_cmp;
}
init_tracer_stat(&branch_trace);
}
return 0;
}
{.name = "all",
.stat_start = all_branch_stat_start,
.stat_next = all_branch_stat_next,
.stat_headers = all_branch_stat_headers,
.stat_show = branch_stat_show},
{ }
};
#else
static struct tracer_stat branch_stats[] = {
{.name = "annotated",
.stat_start = annotated_branch_stat_start,
.stat_next = annotated_branch_stat_next,
.stat_cmp = annotated_branch_stat_cmp,
.stat_headers = annotated_branch_stat_headers,
.stat_show = branch_stat_show},
{ }
};
#endif /* CONFIG_PROFILE_ALL_BRANCHES */ #endif /* CONFIG_PROFILE_ALL_BRANCHES */
static struct tracer branch_trace __read_mostly = static struct tracer branch_trace __read_mostly =
{ {
.name = "branch", .name = "branch",
@ -383,16 +378,8 @@ static struct tracer branch_trace __read_mostly =
#ifdef CONFIG_FTRACE_SELFTEST #ifdef CONFIG_FTRACE_SELFTEST
.selftest = trace_selftest_startup_branch, .selftest = trace_selftest_startup_branch,
#endif /* CONFIG_FTRACE_SELFTEST */ #endif /* CONFIG_FTRACE_SELFTEST */
#endif /* CONFIG_BRANCH_TRACER */
.stat_start = annotated_branch_stat_start,
.stat_next = annotated_branch_stat_next,
.stat_show = branch_stat_show,
.stat_headers = annotated_branch_stat_headers,
.stat_cmp = annotated_branch_stat_cmp,
#ifdef CONFIG_PROFILE_ALL_BRANCHES
.flags = &branch_flags,
.set_flag = branch_set_flag,
#endif #endif
.stats = branch_stats
}; };
__init static int init_branch_trace(void) __init static int init_branch_trace(void)

230
kernel/trace/trace_stat.c
View File

@ -21,37 +21,87 @@ struct trace_stat_list {
void *stat; void *stat;
}; };
static LIST_HEAD(stat_list); /* A stat session is the stats output in one file */
struct tracer_stat_session {
struct tracer_stat *ts;
struct list_head stat_list;
struct mutex stat_mutex;
};
/* /* All of the sessions currently in use. Each stat file embeed one session */
* This is a copy of the current tracer to avoid racy static struct tracer_stat_session **all_stat_sessions;
* and dangerous output while the current tracer is static int nb_sessions;
* switched. static struct dentry *stat_dir, **stat_files;
*/
static struct tracer current_tracer;
/*
* Protect both the current tracer and the global
* stat list.
*/
static DEFINE_MUTEX(stat_list_mutex);
static void reset_stat_list(void) static void reset_stat_session(struct tracer_stat_session *session)
{ {
struct trace_stat_list *node, *next; struct trace_stat_list *node, *next;
list_for_each_entry_safe(node, next, &stat_list, list) list_for_each_entry_safe(node, next, &session->stat_list, list)
kfree(node); kfree(node);
INIT_LIST_HEAD(&stat_list); INIT_LIST_HEAD(&session->stat_list);
} }
void init_tracer_stat(struct tracer *trace) /* Called when a tracer is initialized */
static int init_all_sessions(int nb, struct tracer_stat *ts)
{ {
mutex_lock(&stat_list_mutex); int i, j;
current_tracer = *trace; struct tracer_stat_session *session;
mutex_unlock(&stat_list_mutex);
nb_sessions = 0;
if (all_stat_sessions) {
for (i = 0; i < nb_sessions; i++) {
session = all_stat_sessions[i];
reset_stat_session(session);
mutex_destroy(&session->stat_mutex);
kfree(session);
}
}
all_stat_sessions = kmalloc(sizeof(struct tracer_stat_session *) * nb,
GFP_KERNEL);
if (!all_stat_sessions)
return -ENOMEM;
for (i = 0; i < nb; i++) {
session = kmalloc(sizeof(struct tracer_stat_session) * nb,
GFP_KERNEL);
if (!session)
goto free_sessions;
INIT_LIST_HEAD(&session->stat_list);
mutex_init(&session->stat_mutex);
session->ts = &ts[i];
all_stat_sessions[i] = session;
}
nb_sessions = nb;
return 0;
free_sessions:
for (j = 0; j < i; j++)
kfree(all_stat_sessions[i]);
kfree(all_stat_sessions);
all_stat_sessions = NULL;
return -ENOMEM;
}
static int basic_tracer_stat_checks(struct tracer_stat *ts)
{
int i;
if (!ts)
return 0;
for (i = 0; ts[i].name; i++) {
if (!ts[i].stat_start || !ts[i].stat_next || !ts[i].stat_show)
return -EBUSY;
}
return i;
} }
/* /*
@ -69,22 +119,19 @@ static int dummy_cmp(void *p1, void *p2)
* All of these copies and sorting are required on all opening * All of these copies and sorting are required on all opening
* since the stats could have changed between two file sessions. * since the stats could have changed between two file sessions.
*/ */
static int stat_seq_init(void) static int stat_seq_init(struct tracer_stat_session *session)
{ {
struct trace_stat_list *iter_entry, *new_entry; struct trace_stat_list *iter_entry, *new_entry;
struct tracer_stat *ts = session->ts;
void *prev_stat; void *prev_stat;
int ret = 0; int ret = 0;
int i; int i;
mutex_lock(&stat_list_mutex); mutex_lock(&session->stat_mutex);
reset_stat_list(); reset_stat_session(session);
if (!current_tracer.stat_start || !current_tracer.stat_next || if (!ts->stat_cmp)
!current_tracer.stat_show) ts->stat_cmp = dummy_cmp;
goto exit;
if (!current_tracer.stat_cmp)
current_tracer.stat_cmp = dummy_cmp;
/* /*
* The first entry. Actually this is the second, but the first * The first entry. Actually this is the second, but the first
@ -97,9 +144,10 @@ static int stat_seq_init(void)
} }
INIT_LIST_HEAD(&new_entry->list); INIT_LIST_HEAD(&new_entry->list);
list_add(&new_entry->list, &stat_list);
new_entry->stat = current_tracer.stat_start();
list_add(&new_entry->list, &session->stat_list);
new_entry->stat = ts->stat_start();
prev_stat = new_entry->stat; prev_stat = new_entry->stat;
/* /*
@ -114,15 +162,16 @@ static int stat_seq_init(void)
} }
INIT_LIST_HEAD(&new_entry->list); INIT_LIST_HEAD(&new_entry->list);
new_entry->stat = current_tracer.stat_next(prev_stat, i); new_entry->stat = ts->stat_next(prev_stat, i);
/* End of insertion */ /* End of insertion */
if (!new_entry->stat) if (!new_entry->stat)
break; break;
list_for_each_entry(iter_entry, &stat_list, list) { list_for_each_entry(iter_entry, &session->stat_list, list) {
/* Insertion with a descendent sorting */ /* Insertion with a descendent sorting */
if (current_tracer.stat_cmp(new_entry->stat, if (ts->stat_cmp(new_entry->stat,
iter_entry->stat) > 0) { iter_entry->stat) > 0) {
list_add_tail(&new_entry->list, list_add_tail(&new_entry->list,
@ -131,7 +180,7 @@ static int stat_seq_init(void)
/* The current smaller value */ /* The current smaller value */
} else if (list_is_last(&iter_entry->list, } else if (list_is_last(&iter_entry->list,
&stat_list)) { &session->stat_list)) {
list_add(&new_entry->list, &iter_entry->list); list_add(&new_entry->list, &iter_entry->list);
break; break;
} }
@ -140,49 +189,49 @@ static int stat_seq_init(void)
prev_stat = new_entry->stat; prev_stat = new_entry->stat;
} }
exit: exit:
mutex_unlock(&stat_list_mutex); mutex_unlock(&session->stat_mutex);
return ret; return ret;
exit_free_list: exit_free_list:
reset_stat_list(); reset_stat_session(session);
mutex_unlock(&stat_list_mutex); mutex_unlock(&session->stat_mutex);
return ret; return ret;
} }
static void *stat_seq_start(struct seq_file *s, loff_t *pos) static void *stat_seq_start(struct seq_file *s, loff_t *pos)
{ {
struct list_head *l = (struct list_head *)s->private; struct tracer_stat_session *session = s->private;
/* Prevent from tracer switch or stat_list modification */ /* Prevent from tracer switch or stat_list modification */
mutex_lock(&stat_list_mutex); mutex_lock(&session->stat_mutex);
/* If we are in the beginning of the file, print the headers */ /* If we are in the beginning of the file, print the headers */
if (!*pos && current_tracer.stat_headers) if (!*pos && session->ts->stat_headers)
current_tracer.stat_headers(s); session->ts->stat_headers(s);
return seq_list_start(l, *pos); return seq_list_start(&session->stat_list, *pos);
} }
static void *stat_seq_next(struct seq_file *s, void *p, loff_t *pos) static void *stat_seq_next(struct seq_file *s, void *p, loff_t *pos)
{ {
struct list_head *l = (struct list_head *)s->private; struct tracer_stat_session *session = s->private;
return seq_list_next(p, l, pos); return seq_list_next(p, &session->stat_list, pos);
} }
static void stat_seq_stop(struct seq_file *m, void *p) static void stat_seq_stop(struct seq_file *s, void *p)
{ {
mutex_unlock(&stat_list_mutex); struct tracer_stat_session *session = s->private;
mutex_unlock(&session->stat_mutex);
} }
static int stat_seq_show(struct seq_file *s, void *v) static int stat_seq_show(struct seq_file *s, void *v)
{ {
struct trace_stat_list *entry; struct tracer_stat_session *session = s->private;
struct trace_stat_list *l = list_entry(v, struct trace_stat_list, list);
entry = list_entry(v, struct trace_stat_list, list); return session->ts->stat_show(s, l->stat);
return current_tracer.stat_show(s, entry->stat);
} }
static const struct seq_operations trace_stat_seq_ops = { static const struct seq_operations trace_stat_seq_ops = {
@ -192,15 +241,18 @@ static const struct seq_operations trace_stat_seq_ops = {
.show = stat_seq_show .show = stat_seq_show
}; };
/* The session stat is refilled and resorted at each stat file opening */
static int tracing_stat_open(struct inode *inode, struct file *file) static int tracing_stat_open(struct inode *inode, struct file *file)
{ {
int ret; int ret;
struct tracer_stat_session *session = inode->i_private;
ret = seq_open(file, &trace_stat_seq_ops); ret = seq_open(file, &trace_stat_seq_ops);
if (!ret) { if (!ret) {
struct seq_file *m = file->private_data; struct seq_file *m = file->private_data;
m->private = &stat_list; m->private = session;
ret = stat_seq_init(); ret = stat_seq_init(session);
} }
return ret; return ret;
@ -212,9 +264,12 @@ static int tracing_stat_open(struct inode *inode, struct file *file)
*/ */
static int tracing_stat_release(struct inode *i, struct file *f) static int tracing_stat_release(struct inode *i, struct file *f)
{ {
mutex_lock(&stat_list_mutex); struct tracer_stat_session *session = i->i_private;
reset_stat_list();
mutex_unlock(&stat_list_mutex); mutex_lock(&session->stat_mutex);
reset_stat_session(session);
mutex_unlock(&session->stat_mutex);
return 0; return 0;
} }
@ -225,17 +280,70 @@ static const struct file_operations tracing_stat_fops = {
.release = tracing_stat_release .release = tracing_stat_release
}; };
static void destroy_trace_stat_files(void)
{
int i;
if (stat_files) {
for (i = 0; i < nb_sessions; i++)
debugfs_remove(stat_files[i]);
kfree(stat_files);
stat_files = NULL;
}
}
static void init_trace_stat_files(void)
{
int i;
if (!stat_dir || !nb_sessions)
return;
stat_files = kmalloc(sizeof(struct dentry *) * nb_sessions, GFP_KERNEL);
if (!stat_files) {
pr_warning("trace stat: not enough memory\n");
return;
}
for (i = 0; i < nb_sessions; i++) {
struct tracer_stat_session *session = all_stat_sessions[i];
stat_files[i] = debugfs_create_file(session->ts->name, 0644,
stat_dir,
session, &tracing_stat_fops);
if (!stat_files[i])
pr_warning("cannot create %s entry\n",
session->ts->name);
}
}
void init_tracer_stat(struct tracer *trace)
{
int nb = basic_tracer_stat_checks(trace->stats);
destroy_trace_stat_files();
if (nb < 0) {
pr_warning("stat tracing: missing stat callback on %s\n",
trace->name);
return;
}
if (!nb)
return;
init_all_sessions(nb, trace->stats);
init_trace_stat_files();
}
static int __init tracing_stat_init(void) static int __init tracing_stat_init(void)
{ {
struct dentry *d_tracing; struct dentry *d_tracing;
struct dentry *entry;
d_tracing = tracing_init_dentry(); d_tracing = tracing_init_dentry();
entry = debugfs_create_file("trace_stat", 0444, d_tracing, stat_dir = debugfs_create_dir("trace_stat", d_tracing);
NULL, if (!stat_dir)
&tracing_stat_fops);
if (!entry)
pr_warning("Could not create debugfs " pr_warning("Could not create debugfs "
"'trace_stat' entry\n"); "'trace_stat' entry\n");
return 0; return 0;