Merge branch 'perf-core-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip

Pull perf updates from Ingo Molnar:
 "Kernel side changes:

   - Clean up the x86 instruction decoder (Masami Hiramatsu)

   - Add new uprobes optimization for PUSH instructions on x86 (Yonghong
     Song)

   - Add MSR_IA32_THERM_STATUS to the MSR events (Stephane Eranian)

   - Fix misc bugs, update documentation, plus various cleanups (Jiri
     Olsa)

  There's a large number of tooling side improvements:

   - Intel-PT/BTS improvements (Adrian Hunter)

   - Numerous 'perf trace' improvements (Arnaldo Carvalho de Melo)

   - Introduce an errno code to string facility (Hendrik Brueckner)

   - Various build system improvements (Jiri Olsa)

   - Add support for CoreSight trace decoding by making the perf tools
     use the external openCSD (Mathieu Poirier, Tor Jeremiassen)

   - Add ARM Statistical Profiling Extensions (SPE) support (Kim
     Phillips)

   - libtraceevent updates (Steven Rostedt)

   - Intel vendor event JSON updates (Andi Kleen)

   - Introduce 'perf report --mmaps' and 'perf report --tasks' to show
     info present in 'perf.data' (Jiri Olsa, Arnaldo Carvalho de Melo)

   - Add infrastructure to record first and last sample time to the
     perf.data file header, so that when processing all samples in a
     'perf record' session, such as when doing build-id processing, or
     when specifically requesting that that info be recorded, use that
     in 'perf report --time', that also got support for percent slices
     in addition to absolute ones.

     I.e. now it is possible to ask for the samples in the 10%-20% time
     slice of a perf.data file (Jin Yao)

   - Allow system wide 'perf stat --per-thread', sorting the result (Jin
     Yao)

     E.g.:

      [root@jouet ~]# perf stat --per-thread --metrics IPC
      ^C
       Performance counter stats for 'system wide':

                  make-22229  23,012,094,032  inst_retired.any   #  0.8 IPC
                   cc1-22419     692,027,497  inst_retired.any   #  0.8 IPC
                   gcc-22418     328,231,855  inst_retired.any   #  0.9 IPC
                   cc1-22509     220,853,647  inst_retired.any   #  0.8 IPC
                   gcc-22486     199,874,810  inst_retired.any   #  1.0 IPC
                    as-22466     177,896,365  inst_retired.any   #  0.9 IPC
                   cc1-22465     150,732,374  inst_retired.any   #  0.8 IPC
                   gcc-22508     112,555,593  inst_retired.any   #  0.9 IPC
                   cc1-22487     108,964,079  inst_retired.any   #  0.7 IPC
       qemu-system-x86-2697       21,330,550  inst_retired.any   #  0.3 IPC
       systemd-journal-551        20,642,951  inst_retired.any   #  0.4 IPC
       docker-containe-17651       9,552,892  inst_retired.any   #  0.5 IPC
       dockerd-current-9809        7,528,586  inst_retired.any   #  0.5 IPC
                  make-22153  12,504,194,380  inst_retired.any   #  0.8 IPC
               python2-22429  12,081,290,954  inst_retired.any   #  0.8 IPC
      <SNIP>
               python2-22429  15,026,328,103  cpu_clk_unhalted.thread
                   cc1-22419     826,660,193  cpu_clk_unhalted.thread
                   gcc-22418     365,321,295  cpu_clk_unhalted.thread
                   cc1-22509     279,169,362  cpu_clk_unhalted.thread
                   gcc-22486     210,156,950  cpu_clk_unhalted.thread
      <SNIP>

           5.638075538 seconds time elapsed

     [root@jouet ~]#

   - Improve shell auto-completion of perf events (Jin Yao)

   - 'perf probe' improvements (Masami Hiramatsu)

   - Improve PMU infrastructure to support amp64's ThunderX2
     implementation defined core events (Ganapatrao Kulkarni)

   - Various annotation related improvements and fixes (Thomas Richter)

   - Clarify usage of 'overwrite' and 'backward' in the evlist/mmap
     code, removing the 'overwrite' parameter from several functions as
     it was always used it as 'false' (Wang Nan)

   - Fix/improve 'perf record' reverse recording support (Wang Nan)

   - Improve command line options documentation (Sihyeon Jang)

   - Optimize sample parsing for ordering events, where we don't need to
     parse all the PERF_SAMPLE_ bits, just the ones leading to the
     timestamp needed to reorder events (Jiri Olsa)

   - Generalize the annotation code to support other source information
     besides objdump/DWARF obtained ones, starting with python scripts,
     that will is slated to be merged soon (Jiri Olsa)

   - ... and a lot more that I failed to list, see the shortlog and
     changelog for details"

* 'perf-core-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip: (262 commits)
  perf trace beauty flock: Move to separate object file
  perf evlist: Remove fcntl.h from evlist.h
  perf trace beauty futex: Beautify FUTEX_BITSET_MATCH_ANY
  perf trace: Do not print from time delta for interrupted syscall lines
  perf trace: Add --print-sample
  perf bpf: Remove misplaced __maybe_unused attribute
  MAINTAINERS: Adding entry for CoreSight trace decoding
  perf tools: Add mechanic to synthesise CoreSight trace packets
  perf tools: Add full support for CoreSight trace decoding
  pert tools: Add queue management functionality
  perf tools: Add functionality to communicate with the openCSD decoder
  perf tools: Add support for decoding CoreSight trace data
  perf tools: Add decoder mechanic to support dumping trace data
  perf tools: Add processing of coresight metadata
  perf tools: Add initial entry point for decoder CoreSight traces
  perf tools: Integrating the CoreSight decoding library
  perf vendor events intel: Update IvyTown files to V20
  perf vendor events intel: Update IvyBridge files to V20
  perf vendor events intel: Update BroadwellDE events to V7
  perf vendor events intel: Update SkylakeX events to V1.06
  ...

1 files changed, 61 insertions, 24 deletions

diff --git a/tools/perf/util/probe-event.c b/tools/perf/util/probe-event.c
index b7aaf9b2294d..e1dbc9821617 100644
--- a/tools/perf/util/probe-event.c
+++ b/tools/perf/util/probe-event.c
@@ -1325,27 +1325,30 @@ static int parse_perf_probe_event_name(char **arg, struct perf_probe_event *pev)
 {
         char *ptr;
 
-        ptr = strchr(*arg, ':');
+        ptr = strpbrk_esc(*arg, ":");
         if (ptr) {
                 *ptr = '\0';
                 if (!pev->sdt && !is_c_func_name(*arg))
                         goto ng_name;
-                pev->group = strdup(*arg);
+                pev->group = strdup_esc(*arg);
                 if (!pev->group)
                         return -ENOMEM;
                 *arg = ptr + 1;
         } else
                 pev->group = NULL;
-        if (!pev->sdt && !is_c_func_name(*arg)) {
+
+        pev->event = strdup_esc(*arg);
+        if (pev->event == NULL)
+                return -ENOMEM;
+
+        if (!pev->sdt && !is_c_func_name(pev->event)) {
+                zfree(&pev->event);
 ng_name:
+                zfree(&pev->group);
                 semantic_error("%s is bad for event name -it must "
                                "follow C symbol-naming rule.\n", *arg);
                 return -EINVAL;
         }
-        pev->event = strdup(*arg);
-        if (pev->event == NULL)
-                return -ENOMEM;
-
         return 0;
 }
 
@@ -1373,7 +1376,7 @@ static int parse_perf_probe_point(char *arg, struct perf_probe_event *pev)
                         arg++;
         }
 
-        ptr = strpbrk(arg, ";=@+%");
+        ptr = strpbrk_esc(arg, ";=@+%");
         if (pev->sdt) {
                 if (ptr) {
                         if (*ptr != '@') {
@@ -1387,7 +1390,7 @@ static int parse_perf_probe_point(char *arg, struct perf_probe_event *pev)
                                 pev->target = build_id_cache__origname(tmp);
                                 free(tmp);
                         } else
-                                pev->target = strdup(ptr + 1);
+                                pev->target = strdup_esc(ptr + 1);
                         if (!pev->target)
                                 return -ENOMEM;
                         *ptr = '\0';
@@ -1421,13 +1424,14 @@ static int parse_perf_probe_point(char *arg, struct perf_probe_event *pev)
          *
          * Otherwise, we consider arg to be a function specification.
          */
-        if (!strpbrk(arg, "+@%") && (ptr = strpbrk(arg, ";:")) != NULL) {
+        if (!strpbrk_esc(arg, "+@%")) {
+                ptr = strpbrk_esc(arg, ";:");
                 /* This is a file spec if it includes a '.' before ; or : */
-                if (memchr(arg, '.', ptr - arg))
+                if (ptr && memchr(arg, '.', ptr - arg))
                         file_spec = true;
         }
 
-        ptr = strpbrk(arg, ";:+@%");
+        ptr = strpbrk_esc(arg, ";:+@%");
         if (ptr) {
                 nc = *ptr;
                 *ptr++ = '\0';
@@ -1436,7 +1440,7 @@ static int parse_perf_probe_point(char *arg, struct perf_probe_event *pev)
         if (arg[0] == '\0')
                 tmp = NULL;
         else {
-                tmp = strdup(arg);
+                tmp = strdup_esc(arg);
                 if (tmp == NULL)
                         return -ENOMEM;
         }
@@ -1469,12 +1473,12 @@ static int parse_perf_probe_point(char *arg, struct perf_probe_event *pev)
                 arg = ptr;
                 c = nc;
                 if (c == ';') { /* Lazy pattern must be the last part */
-                        pp->lazy_line = strdup(arg);
+                        pp->lazy_line = strdup(arg); /* let leave escapes */
                         if (pp->lazy_line == NULL)
                                 return -ENOMEM;
                         break;
                 }
-                ptr = strpbrk(arg, ";:+@%");
+                ptr = strpbrk_esc(arg, ";:+@%");
                 if (ptr) {
                         nc = *ptr;
                         *ptr++ = '\0';
@@ -1501,7 +1505,7 @@ static int parse_perf_probe_point(char *arg, struct perf_probe_event *pev)
                                 semantic_error("SRC@SRC is not allowed.\n");
                                 return -EINVAL;
                         }
-                        pp->file = strdup(arg);
+                        pp->file = strdup_esc(arg);
                         if (pp->file == NULL)
                                 return -ENOMEM;
                         break;
@@ -2573,7 +2577,8 @@ int show_perf_probe_events(struct strfilter *filter)
 }
 
 static int get_new_event_name(char *buf, size_t len, const char *base,
-                              struct strlist *namelist, bool allow_suffix)
+                              struct strlist *namelist, bool ret_event,
+                              bool allow_suffix)
 {
         int i, ret;
         char *p, *nbase;
@@ -2584,13 +2589,13 @@ static int get_new_event_name(char *buf, size_t len, const char *base,
         if (!nbase)
                 return -ENOMEM;
 
-        /* Cut off the dot suffixes (e.g. .const, .isra)*/
-        p = strchr(nbase, '.');
+        /* Cut off the dot suffixes (e.g. .const, .isra) and version suffixes */
+        p = strpbrk(nbase, ".@");
         if (p && p != nbase)
                 *p = '\0';
 
         /* Try no suffix number */
-        ret = e_snprintf(buf, len, "%s", nbase);
+        ret = e_snprintf(buf, len, "%s%s", nbase, ret_event ? "__return" : "");
         if (ret < 0) {
                 pr_debug("snprintf() failed: %d\n", ret);
                 goto out;
@@ -2625,6 +2630,14 @@ static int get_new_event_name(char *buf, size_t len, const char *base,
 
 out:
         free(nbase);
+
+        /* Final validation */
+        if (ret >= 0 && !is_c_func_name(buf)) {
+                pr_warning("Internal error: \"%s\" is an invalid event name.\n",
+                           buf);
+                ret = -EINVAL;
+        }
+
         return ret;
 }
 
@@ -2681,8 +2694,8 @@ static int probe_trace_event__set_name(struct probe_trace_event *tev,
                 group = PERFPROBE_GROUP;
 
         /* Get an unused new event name */
-        ret = get_new_event_name(buf, 64, event,
-                                 namelist, allow_suffix);
+        ret = get_new_event_name(buf, 64, event, namelist,
+                                 tev->point.retprobe, allow_suffix);
         if (ret < 0)
                 return ret;
 
@@ -2792,16 +2805,40 @@ static int find_probe_functions(struct map *map, char *name,
         int found = 0;
         struct symbol *sym;
         struct rb_node *tmp;
+        const char *norm, *ver;
+        char *buf = NULL;
+        bool cut_version = true;
 
         if (map__load(map) < 0)
                 return 0;
 
+        /* If user gives a version, don't cut off the version from symbols */
+        if (strchr(name, '@'))
+                cut_version = false;
+
         map__for_each_symbol(map, sym, tmp) {
-                if (strglobmatch(sym->name, name)) {
+                norm = arch__normalize_symbol_name(sym->name);
+                if (!norm)
+                        continue;
+
+                if (cut_version) {
+                        /* We don't care about default symbol or not */
+                        ver = strchr(norm, '@');
+                        if (ver) {
+                                buf = strndup(norm, ver - norm);
+                                if (!buf)
+                                        return -ENOMEM;
+                                norm = buf;
+                        }
+                }
+
+                if (strglobmatch(norm, name)) {
                         found++;
                         if (syms && found < probe_conf.max_probes)
                                 syms[found - 1] = sym;
                 }
+                if (buf)
+                        zfree(&buf);
         }
 
         return found;
@@ -2847,7 +2884,7 @@ static int find_probe_trace_events_from_map(struct perf_probe_event *pev,
          * same name but different addresses, this lists all the symbols.
          */
         num_matched_functions = find_probe_functions(map, pp->function, syms);
-        if (num_matched_functions == 0) {
+        if (num_matched_functions <= 0) {
                 pr_err("Failed to find symbol %s in %s\n", pp->function,
                         pev->target ? : "kernel");
                 ret = -ENOENT;