Re: [PATCH 08/12] perf: export some syscall metadata

From: Frederic Weisbecker
Date: Tue Feb 23 2010 - 16:45:01 EST


On Wed, Jan 27, 2010 at 02:27:59AM -0600, Tom Zanussi wrote:
> Create and export a list of syscall_nr:syscall_name pairs from the
> data in the syscalls_metadata, for use initially by perf trace.
>
> Signed-off-by: Tom Zanussi <tzanussi@xxxxxxxxx>



I would have much more preferred we inject a kind of "syscall_map"
event for that. Unfortunately the event injection is still
in discussion and not yet ready.


I guess we can take this temporary solution and remove that
later in profit of injected events. It means we'll need to
keep support for that file though.


Hmm...



> ---
> kernel/trace/trace_syscalls.c | 87 +++++++++++++++++++++++++++++++++++++++++
> 1 files changed, 87 insertions(+), 0 deletions(-)
>
> diff --git a/kernel/trace/trace_syscalls.c b/kernel/trace/trace_syscalls.c
> index f6b0712..e037486 100644
> --- a/kernel/trace/trace_syscalls.c
> +++ b/kernel/trace/trace_syscalls.c
> @@ -626,5 +626,92 @@ void prof_sysexit_disable(struct ftrace_event_call *call)
> mutex_unlock(&syscall_trace_lock);
> }
>
> +struct syscall_metadata **meta_skip(struct syscall_metadata **meta,
> + loff_t *pos)
> +{
> + struct syscall_metadata **end = syscalls_metadata + NR_syscalls;
> +
> + do {
> + meta++;
> + (*pos)++;
> + } while (meta < end && *meta == NULL);
> +
> + if (meta >= end)
> + meta = NULL;
> +
> + return meta;
> +}
> +
> +static void *syscall_map_start(struct seq_file *s, loff_t *pos)
> +{
> + struct syscall_metadata **end = syscalls_metadata + NR_syscalls;
> + struct syscall_metadata **meta = syscalls_metadata + *pos;
> +
> + if (meta >= end)
> + return NULL;
> +
> + if (*meta == NULL)
> + meta = meta_skip(meta, pos);
> +
> + return meta;
> +}
> +
> +static void *syscall_map_next(struct seq_file *s, void *v, loff_t *pos)
> +{
> + (*pos)++;
> +
> + return syscall_map_start(s, pos);
> +}
> +
> +static int syscall_map_show(struct seq_file *s, void *v)
> +{
> + const struct syscall_metadata **meta = v;
> + const struct syscall_metadata *m = *meta;
> +
> + seq_printf(s, "%d:", m->syscall_nr);
> + seq_printf(s, "%s\n", m->name);
> +
> + return 0;
> +}
> +
> +static void syscall_map_stop(struct seq_file *m, void *p)
> +{
> +}
> +
> +static const struct seq_operations show_syscall_map_seq_ops = {
> + .start = syscall_map_start,
> + .next = syscall_map_next,
> + .show = syscall_map_show,
> + .stop = syscall_map_stop,
> +};
> +
> +static int syscall_map_open(struct inode *inode, struct file *file)
> +{
> + return seq_open(file, &show_syscall_map_seq_ops);
> +}
> +
> +static const struct file_operations syscall_map_fops = {
> + .open = syscall_map_open,
> + .read = seq_read,
> + .llseek = seq_lseek,
> + .release = seq_release,
> +};
> +
> +static __init int init_syscall_map(void)
> +{
> + struct dentry *d_tracer;
> +
> + d_tracer = tracing_init_dentry();
> + if (!d_tracer)
> + return 0;
> +
> + trace_create_file("syscall_map", 0444, d_tracer,
> + NULL, &syscall_map_fops);
> +
> + return 0;
> +}
> +
> +fs_initcall(init_syscall_map);
> +
> #endif /* CONFIG_PERF_EVENTS */
>
> --
> 1.6.4.GIT
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/