Re: [PATCH 0/6 v3] perf: Have perf become tracefs aware

[Namhyung Kim <namhyung@kernel.org>]

Hi,

On Mon, Feb 09, 2015 at 10:48:53AM -0500, Steven Rostedt wrote:
> If you already applied v2, here's the patch that changes v3.

In any case, please feel free to add my ACKs to this series.

Thanks,
Namhyung


> 
> -- Steve
> 
> From 55989e0e3e9c15d755a9cf0d8c6a899a0de7d1eb Mon Sep 17 00:00:00 2001
> From: "Steven Rostedt (Red Hat)" <rostedt@goodmis.org>
> Date: Mon, 9 Feb 2015 10:42:59 -0500
> Subject: [PATCH] perf: Fix directory name in error message for tracefs
> 
> If tracefs is configured, but not mounted, the error message should
> ask if it is mounted at /sys/kernel/tracing and not at
> /sys/kernel/debug.
> 
> Link: http://lkml.kernel.org/r/20150208141246.GA13586@danjae
> Reported-by: Namhyung Kim <namhyung@kernel.org>
> Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
> ---
>  tools/perf/tests/open-syscall-all-cpus.c | 2 +-
>  tools/perf/tests/open-syscall.c          | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/perf/tests/open-syscall-all-cpus.c b/tools/perf/tests/open-syscall-all-cpus.c
> index 21969e99ea46..3ec885c48f8f 100644
> --- a/tools/perf/tests/open-syscall-all-cpus.c
> +++ b/tools/perf/tests/open-syscall-all-cpus.c
> @@ -30,7 +30,7 @@ int test__open_syscall_event_on_all_cpus(void)
>  	evsel = perf_evsel__newtp("syscalls", "sys_enter_open");
>  	if (evsel == NULL) {
>  		if (tracefs_configured())
> -			pr_debug("is tracefs mounted on /sys/kernel/debug?\n");
> +			pr_debug("is tracefs mounted on /sys/kernel/tracing?\n");
>  		else if (debugfs_configured())
>  			pr_debug("is debugfs mounted on /sys/kernel/debug?\n");
>  		else
> diff --git a/tools/perf/tests/open-syscall.c b/tools/perf/tests/open-syscall.c
> index 4250e40234d2..07aa319bf334 100644
> --- a/tools/perf/tests/open-syscall.c
> +++ b/tools/perf/tests/open-syscall.c
> @@ -19,7 +19,7 @@ int test__open_syscall_event(void)
>  	evsel = perf_evsel__newtp("syscalls", "sys_enter_open");
>  	if (evsel == NULL) {
>  		if (tracefs_configured())
> -			pr_debug("is tracefs mounted on /sys/kernel/debug?\n");
> +			pr_debug("is tracefs mounted on /sys/kernel/tracing?\n");
>  		else if (debugfs_configured())
>  			pr_debug("is debugfs mounted on /sys/kernel/debug?\n");
>  		else
> -- 
> 2.1.4
>