diff options
| author | 2022-09-02 18:48:55 +0200 | |
|---|---|---|
| committer | 2022-09-07 21:54:01 +0200 | |
| commit | f3c0eba287049237b23d1300376768293eb89e69 (patch) | |
| tree | ceb5cab8cd9b1733b4504a639a63454c4d4c746c /tools/testing/selftests/exec/git:/ssh:/git@git.zx2c4.com | |
| parent | x86/perf: Assert all platform event flags are within PERF_EVENT_FLAG_ARCH (diff) | |
| download | linux-dev-f3c0eba287049237b23d1300376768293eb89e69.tar.xz linux-dev-f3c0eba287049237b23d1300376768293eb89e69.zip | |
perf: Add a few assertions
While auditing 6b959ba22d34 ("perf/core: Fix reentry problem in
perf_output_read_group()") a few spots were found that wanted
assertions.
Notable for_each_sibling_event() relies on exclusion from
modification. This would normally be holding either ctx->lock or
ctx->mutex, however due to how things are constructed disabling IRQs
is a valid and sufficient substitute for ctx->lock.
Another possible site to add assertions would be the various
pmu::{add,del,read,..}() methods, but that's not trivially expressable
in C -- the best option is wrappers, but those are easy enough to
forget.
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Diffstat (limited to 'tools/testing/selftests/exec/git:/ssh:/git@git.zx2c4.com')
0 files changed, 0 insertions, 0 deletions
