Hi Masami, On Mon, 23 Jun 2014 03:17:12 +0000, Masami Hiramatsu wrote: > Fix to add a missing pair of braces for error path. > Commit 36d789a4d75f (perf probe: Improve error message for > unknown member of data structure) introduced this bug. > > Without this fix, defining an event with global variables > is always failed, because it always returns -ENOENT if > the argument is not a local variable. > > ---- > # perf probe -na "vfs_read smp_found_config" > Error: Failed to add events. > ---- > > With this fix, you can set a global variable for the > argument of new event. > > ---- > # perf probe -na "vfs_read smp_found_config" > Added new event: > probe:vfs_read (on vfs_read with smp_found_config) > > You can now use it in all perf tools, such as: > > perf record -e probe:vfs_read -aR sleep 1 > ---- > > Signed-off-by: Masami Hiramatsu <masami.hiramatsu...@hitachi.com> > Reported-by: Patrick Palka <patr...@parcs.ath.cx> > Cc: Peter Zijlstra <a.p.zijls...@chello.nl> > Cc: Paul Mackerras <pau...@samba.org> > Cc: Ingo Molnar <mi...@redhat.com> > Cc: Arnaldo Carvalho de Melo <a...@kernel.org> > Cc: Jiri Olsa <jo...@kernel.org>
Acked-by: Namhyung Kim <namhy...@kernel.org> Thanks, Namhyung -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
