Re: [PATCH 3/4] tracing: move binary buffers into per cpu directory

From: Andrew Morton
Date: Fri Mar 13 2009 - 01:21:28 EST


On Fri, 13 Mar 2009 01:04:49 -0400 Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> if (!entry)
> pr_warning("Could not create debugfs 'trace' entry\n");
> +
> + entry = debugfs_create_file("trace_pipe_raw", 0444, d_cpu,
> + (void *) cpu, &tracing_buffers_fops);
> + if (!entry)
> + pr_warning("Could not create debugfs 'trace_pipe_raw' entry\n");

Maybe we should just do the warning in debugfs_create_file(). It's hard to
think of a scenario where we wouldn't want this failure to be reported.

If there is such a scenario then debugfs_create_file_nowarn() beckons.
--
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/