The title of this patch seems not correct.

On Tue, 14 Mar 2017 20:36:58 +0530
Ravi Bangoria <ravi.bango...@linux.vnet.ibm.com> wrote:

> Perf was showing warning if user tries to record sdt event without
> creating a probepoint. Now we are allowing direct record on sdt
> events, remove this stale warning/hint.
> 
> Signed-off-by: Ravi Bangoria <ravi.bango...@linux.vnet.ibm.com>
> ---
>  tools/lib/api/fs/tracing_path.c | 17 ++++-------------
>  1 file changed, 4 insertions(+), 13 deletions(-)
> 
> diff --git a/tools/lib/api/fs/tracing_path.c b/tools/lib/api/fs/tracing_path.c
> index 3e606b9..fa52e67 100644
> --- a/tools/lib/api/fs/tracing_path.c
> +++ b/tools/lib/api/fs/tracing_path.c
> @@ -103,19 +103,10 @@ int tracing_path__strerror_open_tp(int err, char *buf, 
> size_t size,
>                * - jirka
>                */
>               if (debugfs__configured() || tracefs__configured()) {
> -                     /* sdt markers */
> -                     if (!strncmp(filename, "sdt_", 4)) {
> -                             snprintf(buf, size,
> -                                     "Error:\tFile %s/%s not found.\n"
> -                                     "Hint:\tSDT event cannot be directly 
> recorded on.\n"
> -                                     "\tPlease first use 'perf probe %s:%s' 
> before recording it.\n",
> -                                     tracing_events_path, filename, sys, 
> name);
> -                     } else {
> -                             snprintf(buf, size,
> -                                      "Error:\tFile %s/%s not found.\n"
> -                                      "Hint:\tPerhaps this kernel misses 
> some CONFIG_ setting to enable this feature?.\n",
> -                                      tracing_events_path, filename);
> -                     }
> +                     snprintf(buf, size,
> +                              "Error:\tFile %s/%s not found.\n"
> +                              "Hint:\tPerhaps this kernel misses some 
> CONFIG_ setting to enable this feature?.\n",
> +                              tracing_events_path, filename);
>                       break;
>               }
>               snprintf(buf, size, "%s",
> -- 
> 2.9.3
> 


-- 
Masami Hiramatsu <mhira...@kernel.org>

Reply via email to