From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752459AbdBGBN3 (ORCPT ); Mon, 6 Feb 2017 20:13:29 -0500 Received: from mail.kernel.org ([198.145.29.136]:38796 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751588AbdBGBN2 (ORCPT ); Mon, 6 Feb 2017 20:13:28 -0500 Date: Tue, 7 Feb 2017 10:13:17 +0900 From: Masami Hiramatsu To: Ravi Bangoria Cc: linux-kernel@vger.kernel.org, mingo@redhat.com, acme@redhat.com, alexis.berlemont@gmail.com, peterz@infradead.org, alexander.shishkin@linux.intel.com, mpe@ellerman.id.au, naveen.n.rao@linux.vnet.ibm.com, mhiramat@kernel.org, maddy@linux.vnet.ibm.com Subject: Re: [PATCH v2] perf/sdt: Show proper hint Message-Id: <20170207101317.70b2faa505e6db4c01debac0@kernel.org> In-Reply-To: <20170203102642.17258-1-ravi.bangoria@linux.vnet.ibm.com> References: <20170202162051.GA2266@redhat.com> <20170203102642.17258-1-ravi.bangoria@linux.vnet.ibm.com> X-Mailer: Sylpheed 3.5.0 (GTK+ 2.24.31; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 3 Feb 2017 15:56:42 +0530 Ravi Bangoria wrote: > All events from 'perf list', except SDT events, can be directly recorded > with 'perf record'. But, the flow is little different for SDT events. > Probe point for SDT event needs to be created using 'perf probe' before > recording it using 'perf record'. Perf shows misleading hint when user > tries to record SDT event without creating a probe point. Show proper > hint there. > > Before patch: > $ perf record -a -e sdt_glib:idle__add > event syntax error: 'sdt_glib:idle__add' > \___ unknown tracepoint > > Error: File /sys/kernel/debug/tracing/events/sdt_glib/idle__add not found. > Hint: Perhaps this kernel misses some CONFIG_ setting to enable this feature?. > ... > > After patch: > $ perf record -a -e sdt_glib:idle__add > event syntax error: 'sdt_glib:idle__add' > \___ unknown tracepoint > > Error: File /sys/kernel/debug/tracing/events/sdt_glib/idle__add not found. > Hint: SDT event cannot be directly recorded on. Please use 'perf probe sdt_glib:idle__add' before recording it. > ... > > $ perf probe sdt_glib:idle__add > Added new event: > sdt_glib:idle__add (on %idle__add in /usr/lib64/libglib-2.0.so.0.5000.2) > > You can now use it in all perf tools, such as: > > perf record -e sdt_glib:idle__add -aR sleep 1 > > $ perf record -a -e sdt_glib:idle__add > [ perf record: Woken up 1 times to write data ] > [ perf record: Captured and wrote 0.175 MB perf.data ] > Looks good to me:) Acked-by: Masami Hiramatsu Thanks! > Suggested-by: Ingo Molnar > Signed-off-by: Ravi Bangoria > --- > Changes in v2: > - More precise hint > > tools/lib/api/fs/tracing_path.c | 31 +++++++++++++++++-------------- > 1 file changed, 17 insertions(+), 14 deletions(-) > > diff --git a/tools/lib/api/fs/tracing_path.c b/tools/lib/api/fs/tracing_path.c > index 251b7c3..aaafc99 100644 > --- a/tools/lib/api/fs/tracing_path.c > +++ b/tools/lib/api/fs/tracing_path.c > @@ -86,9 +86,13 @@ void put_tracing_file(char *file) > free(file); > } > > -static int strerror_open(int err, char *buf, size_t size, const char *filename) > +int tracing_path__strerror_open_tp(int err, char *buf, size_t size, > + const char *sys, const char *name) > { > char sbuf[128]; > + char filename[PATH_MAX]; > + > + snprintf(filename, PATH_MAX, "%s/%s", sys, name ?: "*"); > > switch (err) { > case ENOENT: > @@ -99,10 +103,18 @@ static int strerror_open(int err, char *buf, size_t size, const char *filename) > * - jirka > */ > if (debugfs__configured() || tracefs__configured()) { > - 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); > + /* 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. Please 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); > + } > break; > } > snprintf(buf, size, "%s", > @@ -125,12 +137,3 @@ static int strerror_open(int err, char *buf, size_t size, const char *filename) > > return 0; > } > - > -int tracing_path__strerror_open_tp(int err, char *buf, size_t size, const char *sys, const char *name) > -{ > - char path[PATH_MAX]; > - > - snprintf(path, PATH_MAX, "%s/%s", sys, name ?: "*"); > - > - return strerror_open(err, buf, size, path); > -} > -- > 2.9.3 > -- Masami Hiramatsu