Re: [RFC PATCH]debugfs:Fix terminology inconsistency of dir nameto mount debugfs filesystem.

From: Greg KH
Date: Wed May 27 2009 - 15:17:17 EST


On Wed, May 27, 2009 at 02:22:37PM -0400, Steven Rostedt wrote:
>
> On Wed, 27 May 2009, Jaswinder Singh Rajput wrote:
>
> > Hello Steve,
> >
> > On Tue, 2009-05-26 at 22:38 -0400, Steven Rostedt wrote:
> >
> > > If you look in Documentation/trace/ftrace.txt I have:
> > >
> > > ===
> > > To mount the debugfs system:
> > >
> > > # mkdir /debug
> > > # mount -t debugfs nodev /debug
> > >
> > > ( Note: it is more common to mount at /sys/kernel/debug, but for
> > > simplicity this document will use /debug)
> > > ===
> > >
> >
> > Please standardize Documentation/trace/ftrace.txt and avoid confusion to
> > users:
> >
> > ===
> > To mount the debugfs system:
> >
> > # mount -t debugfs nodev /sys/kernel/debug
> > ===
>
> I really don't want to do that. Then I would need to show
>
> # echo function_graph > /sys/kernel/debug/current_trace
>
> Which I really do not like. I state that the common place to mount it is
> in /sys/kernel/debug, But my examples are much easier to understand if
> I just do: # echo function_graph > /debug/current_trace

It's not just the "common" place, it is the DOCUMENTED PLACE.

I'm sorry you have to type a long path, perhaps you wish to invest in a
tab key for auto-completion :)

Again, distros already mount debugfs at this location, please don't
confuse people just because you are annoyed at the length of the path.

thanks,

greg k-h
--
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/