1 files changed, 51 insertions, 3 deletions

diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
index 20916dd77aac..8dc98c598b1a 100644
--- a/tools/perf/builtin-trace.c
+++ b/tools/perf/builtin-trace.c
@@ -33,6 +33,7 @@
 #include "util/stat.h"
 #include "trace-event.h"
 #include "util/parse-events.h"
+#include "util/bpf-loader.h"
 
 #include <libaudit.h>
 #include <stdlib.h>
@@ -1724,8 +1725,12 @@ static int trace__read_syscall_info(struct trace *trace, int id)
 
         sc->args = sc->tp_format->format.fields;
         sc->nr_args = sc->tp_format->format.nr_fields;
-        /* drop nr field - not relevant here; does not exist on older kernels */
-        if (sc->args && strcmp(sc->args->name, "nr") == 0) {
+        /*
+         * We need to check and discard the first variable '__syscall_nr'
+         * or 'nr' that mean the syscall number. It is needless here.
+         * So drop '__syscall_nr' or 'nr' field but does not exist on older kernels.
+         */
+        if (sc->args && (!strcmp(sc->args->name, "__syscall_nr") || !strcmp(sc->args->name, "nr"))) {
                 sc->args = sc->args->next;
                 --sc->nr_args;
         }
@@ -2177,6 +2182,37 @@ out_dump:
         return 0;
 }
 
+static void bpf_output__printer(enum binary_printer_ops op,
+                                unsigned int val, void *extra)
+{
+        FILE *output = extra;
+        unsigned char ch = (unsigned char)val;
+
+        switch (op) {
+        case BINARY_PRINT_CHAR_DATA:
+                fprintf(output, "%c", isprint(ch) ? ch : '.');
+                break;
+        case BINARY_PRINT_DATA_BEGIN:
+        case BINARY_PRINT_LINE_BEGIN:
+        case BINARY_PRINT_ADDR:
+        case BINARY_PRINT_NUM_DATA:
+        case BINARY_PRINT_NUM_PAD:
+        case BINARY_PRINT_SEP:
+        case BINARY_PRINT_CHAR_PAD:
+        case BINARY_PRINT_LINE_END:
+        case BINARY_PRINT_DATA_END:
+        default:
+                break;
+        }
+}
+
+static void bpf_output__fprintf(struct trace *trace,
+                                struct perf_sample *sample)
+{
+        print_binary(sample->raw_data, sample->raw_size, 8,
+                     bpf_output__printer, trace->output);
+}
+
 static int trace__event_handler(struct trace *trace, struct perf_evsel *evsel,
                                 union perf_event *event __maybe_unused,
                                 struct perf_sample *sample)
@@ -2189,7 +2225,9 @@ static int trace__event_handler(struct trace *trace, struct perf_evsel *evsel,
 
         fprintf(trace->output, "%s:", evsel->name);
 
-        if (evsel->tp_format) {
+        if (perf_evsel__is_bpf_output(evsel)) {
+                bpf_output__fprintf(trace, sample);
+        } else if (evsel->tp_format) {
                 event_format__fprintf(evsel->tp_format, sample->cpu,
                                       sample->raw_data, sample->raw_size,
                                       trace->output);
@@ -2586,6 +2624,16 @@ static int trace__run(struct trace *trace, int argc, const char **argv)
         if (err < 0)
                 goto out_error_open;
 
+        err = bpf__apply_obj_config();
+        if (err) {
+                char errbuf[BUFSIZ];
+
+                bpf__strerror_apply_obj_config(err, errbuf, sizeof(errbuf));
+                pr_err("ERROR: Apply config to BPF failed: %s\n",
+                         errbuf);
+                goto out_error_open;
+        }
+
         /*
          * Better not use !target__has_task() here because we need to cover the
          * case where no threads were specified in the command line, but a