diff options
author | Ingo Molnar <mingo@kernel.org> | 2017-10-03 12:30:24 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2017-10-03 12:30:24 -0400 |
commit | 4b50239a769e78bbe362fc92c3f8bd5415a4a5f5 (patch) | |
tree | 951832e213945ede19cd676a04e5218e68afd074 /tools/perf/tests/attr.py | |
parent | a47ba4d77e1236d214e5116b5631bc4c2d6e6369 (diff) | |
parent | f6a9820d572bd8384d982357cbad214b3a6c04bb (diff) |
Merge tag 'perf-core-for-mingo-4.15-20171003' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/core
Pull perf/core improvements and fixes from Arnaldo Carvalho de Melo:
- Multithread the synthesizing of PERF_RECORD_ events for pre-existing
threads in 'perf top', speeding up that phase, greatly improving the
user experience in systems such as Intel's Knights Mill (Kan Liang)
- 'perf test' fixes for the perf_event_attr test case (Jiri Olsa, Thomas Richter)
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'tools/perf/tests/attr.py')
-rw-r--r-- | tools/perf/tests/attr.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tools/perf/tests/attr.py b/tools/perf/tests/attr.py index 6bb50e82a3e3..a13cd780148e 100644 --- a/tools/perf/tests/attr.py +++ b/tools/perf/tests/attr.py | |||
@@ -237,6 +237,7 @@ class Test(object): | |||
237 | # events in result. Fail if there's not any. | 237 | # events in result. Fail if there's not any. |
238 | for exp_name, exp_event in expect.items(): | 238 | for exp_name, exp_event in expect.items(): |
239 | exp_list = [] | 239 | exp_list = [] |
240 | res_event = {} | ||
240 | log.debug(" matching [%s]" % exp_name) | 241 | log.debug(" matching [%s]" % exp_name) |
241 | for res_name, res_event in result.items(): | 242 | for res_name, res_event in result.items(): |
242 | log.debug(" to [%s]" % res_name) | 243 | log.debug(" to [%s]" % res_name) |
@@ -253,7 +254,10 @@ class Test(object): | |||
253 | if exp_event.optional(): | 254 | if exp_event.optional(): |
254 | log.debug(" %s does not match, but is optional" % exp_name) | 255 | log.debug(" %s does not match, but is optional" % exp_name) |
255 | else: | 256 | else: |
256 | exp_event.diff(res_event) | 257 | if not res_event: |
258 | log.debug(" res_event is empty"); | ||
259 | else: | ||
260 | exp_event.diff(res_event) | ||
257 | raise Fail(self, 'match failure'); | 261 | raise Fail(self, 'match failure'); |
258 | 262 | ||
259 | match[exp_name] = exp_list | 263 | match[exp_name] = exp_list |