diff options
| author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2009-11-20 22:19:49 +0100 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2009-11-21 14:11:37 +0100 |
| commit | abf4868b8548cae18d4fe8bbfb4e207443be01be (patch) | |
| tree | 0a8900bb5413826bc1f3ac9ccc435c4750dfedab /kernel/sched_fair.c | |
| parent | f6d9dd237da400effb265f3554c64413f8a3e7b4 (diff) | |
| download | linux-abf4868b8548cae18d4fe8bbfb4e207443be01be.tar.gz linux-abf4868b8548cae18d4fe8bbfb4e207443be01be.tar.bz2 linux-abf4868b8548cae18d4fe8bbfb4e207443be01be.zip | |
perf: Fix PERF_FORMAT_GROUP scale info
As Corey reported, the total_enabled and total_running times
could occasionally be 0, even though there were events counted.
It turns out this is because we record the times before reading
the counter while the latter updates the times.
This patch corrects that.
While looking at this code I found that there is a lot of
locking iffyness around, the following patches correct most of
that.
Reported-by: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
LKML-Reference: <20091120212508.685559857@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/sched_fair.c')
0 files changed, 0 insertions, 0 deletions