Re: [GIT PULL 0/4] perf/urgent fixes for 'perf sched timehist' and samples/bpf/ testing

Ingo Molnar
2016-12-23T19:24:25Z

* Arnaldo Carvalho de Melo <acme@kernel.org> wrote:

> Hi Ingo,
>
> Please consider pulling,
>
> - Arnaldo
>
> P.S.: Some of the container test builds now include building samples/bpf/ with:
>
> $ make O=/tmp/build/linux allmodconfig
> $ make O=/tmp/build/linux headers_install
> $ make O=/tmp/build/linux samples/bpf/
>
> In distributions having at least clang/llvm 3.8+ with the 'bpf' target.
>
> Examples include debian:experimental, fedora:rawhide and opensuse:tumbleweed,
> more to come as I go over rebuilding the containers looking for more suitable
> environments.
>
> This will help in finding regressions in this area, now using tools/lib/bpf/.
>
> Test results at the end of this message, as usual.
>
> The following changes since commit 1134c2b5cb840409ffd966d8c2a9468f64e6a494:
>
> perf/x86: Fix overlap counter scheduling bug (2016-12-22 17:45:43 +0100)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-urgent-for-mingo-20161222
>
> for you to fetch changes up to bdd75729e5d279d734e8d3fb41ef4818ac1598ab:
>
> perf sched timehist: Fix invalid period calculation (2016-12-22 16:35:46 -0300)
>
> ----------------------------------------------------------------
> perf/urgent fixes:
>
> Fixes for 'perf sched timehist': (Namhyung Kim)
>
> - Define a larger initial alignment value for the COMM column and
> make it be more consistently honoured, for instance in the header.
>
> - Fix invalid period calculation when using the --time option to
> select a time slice, when events outside that slice were being
> considered for the per cpu idle stats summary.
>
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
>
> ----------------------------------------------------------------
> Namhyung Kim (4):
> perf sched timehist: Honour 'comm_width' when aligning the headers
> perf sched timehist: Enlarge default 'comm_width'
> perf sched timehist: Remove hardcoded 'comm_width' check at print_summary
> perf sched timehist: Fix invalid period calculation
>
> tools/perf/builtin-sched.c | 14 ++++++--------
> 1 file changed, 6 insertions(+), 8 deletions(-)

Pulled, thanks a lot Arnaldo!

Ingo