Re: [PATCH 03/11] perf tools: Do not change lib/api/fs/debugfs directly

2015-08-28 Thread Matt Fleming
On Wed, 26 Aug, at 11:27:58AM, Arnaldo Carvalho de Melo wrote: > > Yeah, probably that was a really bad name to start with, i.e. it > should've been named perhaps --event-definitions-dir or something to > that degree. > > "debugfs", "tracefs" and the next thing to come in this area are just > imp

Re: [PATCH 03/11] perf tools: Do not change lib/api/fs/debugfs directly

2015-08-28 Thread Matt Fleming
On Wed, 26 Aug, at 03:46:45PM, Jiri Olsa wrote: > The tracing_events_path is the variable we want to change > via --debugfs-dir option, not the debugfs_mountpoint. > > Link: http://lkml.kernel.org/n/tip-pj9htv4foum6f6uk7wzi9...@git.kernel.org > Signed-off-by: Jiri Olsa > --- > tools/perf/perf.c

Re: [PATCH 03/11] perf tools: Do not change lib/api/fs/debugfs directly

2015-08-26 Thread Arnaldo Carvalho de Melo
Em Wed, Aug 26, 2015 at 10:06:45AM -0400, Raphaël Beamonte escreveu: > On Aug 26, 2015 9:47 AM, "Jiri Olsa" wrote: > > > > The tracing_events_path is the variable we want to change > > via --debugfs-dir option, not the debugfs_mountpoint. > > > > > perf_debugfs_set_path(

Re: [PATCH 03/11] perf tools: Do not change lib/api/fs/debugfs directly

2015-08-26 Thread Jiri Olsa
On Wed, Aug 26, 2015 at 10:06:45AM -0400, Raphaël Beamonte wrote: > On Aug 26, 2015 9:47 AM, "Jiri Olsa" wrote: > > > > The tracing_events_path is the variable we want to change > > via --debugfs-dir option, not the debugfs_mountpoint. > > > > > perf_debugfs_set_path(cmd

[PATCH 03/11] perf tools: Do not change lib/api/fs/debugfs directly

2015-08-26 Thread Jiri Olsa
The tracing_events_path is the variable we want to change via --debugfs-dir option, not the debugfs_mountpoint. Link: http://lkml.kernel.org/n/tip-pj9htv4foum6f6uk7wzi9...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/perf.c | 2 +- tools/perf/util/util.c | 1 - 2 files changed, 1