[PATCH v4 7/7] perf/sdt: Remove stale warning
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--- 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
[PATCH v4 7/7] perf/sdt: Remove stale warning
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 --- 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