Re: [GIT PULL] perf/urgent fixes

From: Ingo Molnar
Date: Wed Mar 04 2020 - 05:54:53 EST



* Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:

> Hi Ingo/Thomas,
>
> Please consider pulling, mostly from mostly valid complaints by
> the most recent gcc version in my container farm,
>
> Best regards,
>
> - Arnaldo
>
> Test results at the end of this message, as usual.
>
> The following changes since commit e0560ba6d92f06dbe13e9d11c921a60c07ea6fcc:
>
> perf annotate: Fix segfault with source toggle (2020-02-27 11:47:23 -0300)
>
> are available in the Git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-urgent-for-mingo-5.6-20200303
>
> for you to fetch changes up to b5c0951860ba98cfe1936b5c0739450875d51451:
>
> perf symbols: Don't try to find a vmlinux file when looking for kernel modules (2020-03-03 16:20:01 -0300)
>
> ----------------------------------------------------------------
> perf/urgent fixes:
>
> perf symbols:
>
> Arnaldo Carvalho de Melo:
>
> - Don't try to find a vmlinux file when looking for kernel modules,
> fixing symbol resolution in systems with compressed kernel modules.
>
> perf env:
>
> Arnaldo Carvalho de Melo:
>
> - Do not return pointers to local variables, fixing valid warning from
> gcc 10 for corner case that stops the build due to -Werror.
>
> perf tests:
>
> Arnaldo Carvalho de Melo:
>
> - Make global variable static in the bp_account entry to fix build
> with gcc 10.
>
> perf parse-events:
>
> Arnaldo Carvalho de Melo:
>
> - Use asprintf() instead of strncpy() to read tracepoint files, addressing
> compiler warning that stops the build as we use -Werror.
>
> perf bench:
>
> Arnaldo Carvalho de Melo:
>
> - Share some global variables to fix build with gcc 10.
>
> Signed-off-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
>
> ----------------------------------------------------------------
> Arnaldo Carvalho de Melo (5):
> perf tests bp_account: Make global variable static
> perf env: Do not return pointers to local variables
> perf parse-events: Use asprintf() instead of strncpy() to read tracepoint files
> perf bench: Share some global variables to fix build with gcc 10
> perf symbols: Don't try to find a vmlinux file when looking for kernel modules
>
> tools/perf/bench/bench.h | 4 ++++
> tools/perf/bench/epoll-ctl.c | 7 +++----
> tools/perf/bench/epoll-wait.c | 11 +++++------
> tools/perf/bench/futex-hash.c | 12 ++++++------
> tools/perf/bench/futex-lock-pi.c | 11 +++++------
> tools/perf/tests/bp_account.c | 2 +-
> tools/perf/util/env.c | 4 ++--
> tools/perf/util/parse-events.c | 10 ++--------
> tools/perf/util/symbol.c | 13 ++++++-------
> 9 files changed, 34 insertions(+), 40 deletions(-)

Pulled, thanks a lot Arnaldo!

Ingo