Re: [PATCH 1/3] ftrace: add function tracing to single thread




On Tue, 25 Nov 2008, Andrew Morton wrote:

On Tue, 25 Nov 2008 18:01:06 -0500 (EST)
Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:


On Tue, 25 Nov 2008, Andrew Morton wrote:

On Tue, 25 Nov 2008 17:34:22 -0500
Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

This patch adds the ability to function trace a single thread.
The file:

/debugfs/tracing/set_ftrace_pid

contains the pid to trace.

What happens if the same pid exists in two or more pid namespaces?

I think we had this discussion before.

Oh. What did we end up concluding?

It tests current->pid, would that
be different among the name spaces?

I think those are non-unique. containers@xxxxxxxxxxxxxx would have
better ideas..

Added list in CC.

I think the end result was, if this file can only be changed by root, then
we do not need to worry about namespaces. This file is a privileged file
that can only be modified by root.

If someday we decide to let non admin users touch this file, then we would
need to care about this. This file may actually be modified in the future
by users, so this may become an issue.

-- Steve

--
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/

Relevant Pages