perf trace: Skip unknown syscalls when expanding strace like syscall groups

We have $INSTALL_DIR/share/perf-core/strace/groups/string files with
syscalls that should be selected when 'string' is used, meaning, in this
case, syscalls that receive as one of its arguments a string, like a
pathname.

But those were first selected and tested on x86_64, and end up failing
in architectures where some of those syscalls are not available, like
the 'access' syscall on arm64, which makes using 'perf trace -e string'
in such archs to fail.

Since this the routine doing the validation is used only when reading
such files, do not fail when some syscall is not found in the
syscalltbl, instead just use pr_debug() to register that in case people
are suspicious of problems.

Now using 'perf trace -e string' should work on arm64, selecting only
the syscalls that have a string and are available on that architecture.

Reported-by: Leo Yan <leo.yan@linaro.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Alexei Starovoitov <ast@kernel.org>
Cc: Daniel Borkmann <daniel@iogearbox.net>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Martin KaFai Lau <kafai@fb.com>
Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Mike Leach <mike.leach@linaro.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Song Liu <songliubraving@fb.com>
Cc: Suzuki K Poulose <suzuki.poulose@arm.com>
Cc: Yonghong Song <yhs@fb.com>
Link: https://lkml.kernel.org/r/20190610184754.GU21245@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
index 1a2a605cf068124fe5fb3dcf1d28a007c48a7b2e..eb70a4b717557eb4d1c705ce2cf77e753ff7d928 100644 (file)
--- a/tools/perf/builtin-trace.c
+++ b/tools/perf/builtin-trace.c
@@ -1529,6 +1529,7 @@ static int trace__read_syscall_info(struct trace *trace, int id)
 static int trace__validate_ev_qualifier(struct trace *trace)
 {
        int err = 0, i;
+       bool printed_invalid_prefix = false;
        size_t nr_allocated;
        struct str_node *pos;
 
@@ -1555,14 +1556,15 @@ static int trace__validate_ev_qualifier(struct trace *trace)
                        if (id >= 0)
                                goto matches;
 
-                       if (err == 0) {
-                               fputs("Error:\tInvalid syscall ", trace->output);
-                               err = -EINVAL;
+                       if (!printed_invalid_prefix) {
+                               pr_debug("Skipping unknown syscalls: ");
+                               printed_invalid_prefix = true;
                        } else {
-                               fputs(", ", trace->output);
+                               pr_debug(", ");
                        }
 
-                       fputs(sc, trace->output);
+                       pr_debug("%s", sc);
+                       continue;
                }
 matches:
                trace->ev_qualifier_ids.entries[i++] = id;
@@ -1591,15 +1593,14 @@ matches:
                }
        }
 
-       if (err < 0) {
-               fputs("\nHint:\ttry 'perf list syscalls:sys_enter_*'"
-                     "\nHint:\tand: 'man syscalls'\n", trace->output);
-out_free:
-               zfree(&trace->ev_qualifier_ids.entries);
-               trace->ev_qualifier_ids.nr = 0;
-       }
 out:
+       if (printed_invalid_prefix)
+               pr_debug("\n");
        return err;
+out_free:
+       zfree(&trace->ev_qualifier_ids.entries);
+       trace->ev_qualifier_ids.nr = 0;
+       goto out;
 }
 
 /*