From: Ingo Molnar <mingo@kernel.org>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: linux-kernel@vger.kernel.org,
Adrian Hunter <adrian.hunter@intel.com>,
Jiri Olsa <jolsa@kernel.org>, Kan Liang <kan.liang@intel.com>,
Namhyung Kim <namhyung@kernel.org>,
pi3orama@163.com, Wang Nan <wangnan0@huawei.com>,
Zefan Li <lizefan@huawei.com>,
Arnaldo Carvalho de Melo <acme@redhat.com>
Subject: Re: [GIT PULL 00/15] perf/urgent fixes
Date: Wed, 16 Sep 2015 09:09:54 +0200 [thread overview]
Message-ID: <20150916070954.GA5034@gmail.com> (raw)
In-Reply-To: <1442329740-18042-1-git-send-email-acme@kernel.org>
* Arnaldo Carvalho de Melo <acme@kernel.org> wrote:
> Hi Ingo,
>
> Please consider pulling, there are quite a few patches, but that is
> because Adrian did the right thing and made his patches very granular, thanks
> Adrian!
>
> - Arnaldo
>
> The following changes since commit 91a4dc9f71e4b8f9b8da547d19f80e7be89d7539:
>
> Merge tag 'perf-urgent-for-mingo' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/urgent (2015-09-14 09:29:22 +0200)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-urgent-for-mingo
>
> for you to fetch changes up to c5e6bd2ed3e81df443e4ae11e95ed71ff77bf9e5:
>
> perf tests: Fix software clock events test setting maps (2015-09-15 11:04:49 -0300)
>
> ----------------------------------------------------------------
> perf/urgent fixes:
>
> User visible:
>
> - Fix segfault pressing -> in 'perf top' with no hist entries (Wang Nan)
>
> E.g:
> perf top -e page-faults --pid 11400 # 11400 generates no page-fault
>
> - Fix propagation of thread and cpu maps, that got broken when doing incomplete
> changes to better support events with a PMU cpu mask, leading to Intel PT to
> fail with an error like:
>
> $ perf record -e intel_pt//u uname
> Error: The sys_perf_event_open() syscall returned with
> 22 (Invalid argument) for event (sched:sched_switch).
>
> Because intel_pt adds that sched:sched_switch evsel to the evlist after the
> thread/cpu maps were propagated to the evsels, fix it (Adrian Hunter)
>
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
>
> ----------------------------------------------------------------
> Adrian Hunter (14):
> perf evlist: Simplify propagate_maps() logic
> perf evlist: Simplify set_maps() logic
> perf evlist: Remove redundant validation from propagate_maps()
> perf evlist: Add has_user_cpus member
> perf evlist: Fix splice_list_tail() not setting evlist
> perf evlist: Fix missing thread_map__put in propagate_maps()
> perf evsel: Add own_cpus member
> perf evlist: Make set_maps() more resilient
> perf evlist: Make create_maps() use set_maps()
> perf evlist: Factor out a function to propagate maps for a single evsel
> perf evlist: Fix add() not propagating maps
> perf evlist: Fix create_syswide_maps() not propagating maps
> perf tests: Fix task exit test setting maps
> perf tests: Fix software clock events test setting maps
>
> Wang Nan (1):
> perf top: Fix segfault pressing -> with no hist entries
>
> tools/perf/tests/sw-clock.c | 18 ++++--
> tools/perf/tests/task-exit.c | 18 ++++--
> tools/perf/ui/browsers/hists.c | 12 +++-
> tools/perf/util/evlist.c | 138 +++++++++++++++++++++++------------------
> tools/perf/util/evlist.h | 9 ++-
> tools/perf/util/evsel.c | 1 +
> tools/perf/util/evsel.h | 1 +
> tools/perf/util/parse-events.c | 7 +--
> 8 files changed, 124 insertions(+), 80 deletions(-)
Pulled, thanks a lot Arnaldo!
Ingo
next prev parent reply other threads:[~2015-09-16 7:10 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-15 15:08 [GIT PULL 00/15] perf/urgent fixes Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 01/15] perf top: Fix segfault pressing -> with no hist entries Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 02/15] perf evlist: Simplify propagate_maps() logic Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 03/15] perf evlist: Simplify set_maps() logic Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 04/15] perf evlist: Remove redundant validation from propagate_maps() Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 05/15] perf evlist: Add has_user_cpus member Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 06/15] perf evlist: Fix splice_list_tail() not setting evlist Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 07/15] perf evlist: Fix missing thread_map__put in propagate_maps() Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 08/15] perf evsel: Add own_cpus member Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 09/15] perf evlist: Make set_maps() more resilient Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 10/15] perf evlist: Make create_maps() use set_maps() Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 11/15] perf evlist: Factor out a function to propagate maps for a single evsel Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 12/15] perf evlist: Fix add() not propagating maps Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 13/15] perf evlist: Fix create_syswide_maps() " Arnaldo Carvalho de Melo
2015-09-15 15:08 ` [PATCH 14/15] perf tests: Fix task exit test setting maps Arnaldo Carvalho de Melo
2015-09-15 15:09 ` [PATCH 15/15] perf tests: Fix software clock events " Arnaldo Carvalho de Melo
2015-09-16 7:09 ` Ingo Molnar [this message]
-- strict thread matches above, loose matches on Subject: below --
2017-06-08 21:59 [GIT PULL 00/15] perf/urgent fixes Arnaldo Carvalho de Melo
2017-06-08 22:42 ` Ingo Molnar
2012-06-01 22:16 Arnaldo Carvalho de Melo
2012-06-01 22:45 ` David Ahern
2012-06-01 23:08 ` Arnaldo Carvalho de Melo
2012-06-06 6:50 ` Ingo Molnar
2012-06-06 22:50 ` David Ahern
2012-06-07 2:19 ` Namhyung Kim
2012-06-07 3:52 ` David Ahern
2012-06-07 16:09 ` Arnaldo Carvalho de Melo
2012-06-07 16:11 ` Steven Rostedt
2012-06-07 16:43 ` Arnaldo Carvalho de Melo
2012-06-07 16:55 ` Borislav Petkov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150916070954.GA5034@gmail.com \
--to=mingo@kernel.org \
--cc=acme@kernel.org \
--cc=acme@redhat.com \
--cc=adrian.hunter@intel.com \
--cc=jolsa@kernel.org \
--cc=kan.liang@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=lizefan@huawei.com \
--cc=namhyung@kernel.org \
--cc=pi3orama@163.com \
--cc=wangnan0@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).