Merge tag 'perf-urgent-for-mingo' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/urgent

Pull perf/urgent fixes from Arnaldo Carvalho de Melo:

  - Add missing newlines to some pr_err() calls (Arnaldo Carvalho de Melo)

  - Print full source file paths when using
    'perf annotate --print-line --full-paths' (Michael Petlan)

  - Fix 'perf probe -d' when just one out of uprobes and kprobes is
    enabled (Wang Nan)

  - Add compiler.h to list.h to fix 'make perf-tar-src-pkg' generated
    tarballs, i.e. out of tree building (Arnaldo Carvalho de Melo)

  - Add the llvm-src-base.c and llvm-src-kbuild.c files, generated by the
    'perf test' LLVM entries, when running it in-tree, to .gitignore (Yunlong Song)

Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>

5 files changed, 14 insertions, 4 deletions

diff --git a/tools/include/linux/list.h b/tools/include/linux/list.h
index 76b014c96893..a017f1595676 100644
--- a/tools/include/linux/list.h
+++ b/tools/include/linux/list.h
@@ -1,3 +1,4 @@
+#include <linux/compiler.h>
 #include <linux/kernel.h>
 #include <linux/types.h>
 
diff --git a/tools/perf/tests/.gitignore b/tools/perf/tests/.gitignore
new file mode 100644
index 000000000000..489fc9ffbcb0
--- /dev/null
+++ b/tools/perf/tests/.gitignore
@@ -0,0 +1,2 @@
+llvm-src-base.c
+llvm-src-kbuild.c
diff --git a/tools/perf/util/annotate.c b/tools/perf/util/annotate.c
index f2974da0185a..1dd1949b0e79 100644
--- a/tools/perf/util/annotate.c
+++ b/tools/perf/util/annotate.c
@@ -1620,6 +1620,7 @@ int symbol__tty_annotate(struct symbol *sym, struct map *map,
         len = symbol__size(sym);
 
         if (print_lines) {
+                srcline_full_filename = full_paths;
                 symbol__get_source_line(sym, map, evsel, &source_line, len);
                 print_summary(&source_line, dso->long_name);
         }
diff --git a/tools/perf/util/probe-file.c b/tools/perf/util/probe-file.c
index 89dbeb92c68e..e3b3b92e4458 100644
--- a/tools/perf/util/probe-file.c
+++ b/tools/perf/util/probe-file.c
@@ -138,6 +138,9 @@ struct strlist *probe_file__get_rawlist(int fd)
         char *p;
         struct strlist *sl;
 
+        if (fd < 0)
+                return NULL;
+
         sl = strlist__new(NULL, NULL);
 
         fp = fdopen(dup(fd), "r");
@@ -271,6 +274,9 @@ int probe_file__get_events(int fd, struct strfilter *filter,
         const char *p;
         int ret = -ENOENT;
 
+        if (!plist)
+                return -EINVAL;
+
         namelist = __probe_file__get_namelist(fd, true);
         if (!namelist)
                 return -ENOENT;
diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c
index 428149bc64d2..c35ffdd360fe 100644
--- a/tools/perf/util/session.c
+++ b/tools/perf/util/session.c
@@ -29,7 +29,7 @@ static int perf_session__open(struct perf_session *session)
         struct perf_data_file *file = session->file;
 
         if (perf_session__read_header(session) < 0) {
-                pr_err("incompatible file format (rerun with -v to learn more)");
+                pr_err("incompatible file format (rerun with -v to learn more)\n");
                 return -1;
         }
 
@@ -37,17 +37,17 @@ static int perf_session__open(struct perf_session *session)
                 return 0;
 
         if (!perf_evlist__valid_sample_type(session->evlist)) {
-                pr_err("non matching sample_type");
+                pr_err("non matching sample_type\n");
                 return -1;
         }
 
         if (!perf_evlist__valid_sample_id_all(session->evlist)) {
-                pr_err("non matching sample_id_all");
+                pr_err("non matching sample_id_all\n");
                 return -1;
         }
 
         if (!perf_evlist__valid_read_format(session->evlist)) {
-                pr_err("non matching read_format");
+                pr_err("non matching read_format\n");
                 return -1;
         }