diff options
| author | Peter Zijlstra <[email protected]> | 2017-09-05 14:26:44 +0000 |
|---|---|---|
| committer | Ingo Molnar <[email protected]> | 2017-10-27 08:31:59 +0000 |
| commit | 0c1cbc18df9e38182a0604b15535699c84d7342a (patch) | |
| tree | 85c7d42284fd594af32ca8d176aa8fc44e618a60 /tools/perf/builtin-script.c | |
| parent | perf/core: Remove wrong barrier (diff) | |
| download | kernel-0c1cbc18df9e38182a0604b15535699c84d7342a.tar.gz kernel-0c1cbc18df9e38182a0604b15535699c84d7342a.zip | |
perf/core: Fix perf_event_read()
perf_event_read() has a number of issues regarding the timekeeping bits.
- The IPI didn't update group times when it found INACTIVE
- The direct call would not re-check ->state after taking ctx->lock
which can result in ->count and timestamps getting out of sync.
And we can make use of the ordering introduced for perf_event_stop()
to make it more accurate for ACTIVE.
Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
Cc: Linus Torvalds <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Signed-off-by: Ingo Molnar <[email protected]>
Diffstat (limited to 'tools/perf/builtin-script.c')
0 files changed, 0 insertions, 0 deletions