summaryrefslogtreecommitdiffstats
path: root/tools/perf/builtin-stat.c
diff options
context:
space:
mode:
authorYong Wang <yong.y.wang@linux.intel.com>2009-06-24 04:13:24 +0200
committerIngo Molnar <mingo@elte.hu>2009-06-24 10:51:24 +0200
commitc14dab5c0782ef632742963a66276a195418a63c (patch)
tree441e3da9c7346f010b129a8826d21323b7d41d0e /tools/perf/builtin-stat.c
parentperf_counter tools: Fix strbuf_fread() error path handling (diff)
downloadlinux-c14dab5c0782ef632742963a66276a195418a63c.tar.xz
linux-c14dab5c0782ef632742963a66276a195418a63c.zip
perf_counter, x86: Set global control MSR correctly
Previous code made an assumption that the power on value of global control MSR has enabled all fixed and general purpose counters properly. However, this is not the case for certain Intel processors, such as Atom - and it might also be firmware dependent. Each enable bit in IA32_PERF_GLOBAL_CTRL is AND'ed with the enable bits for all privilege levels in the respective IA32_PERFEVTSELx or IA32_PERF_FIXED_CTR_CTRL MSRs to start/stop the counting of respective counters. Counting is enabled if the AND'ed results is true; counting is disabled when the result is false. The end result is that all fixed counters are always disabled on Atom processors because the assumption is just invalid. Fix this by not initializing the ctrl-mask out of the global MSR, but setting it to perf_counter_mask. Reported-by: Stephane Eranian <eranian@googlemail.com> Signed-off-by: Yong Wang <yong.y.wang@intel.com> Cc: Arjan van de Ven <arjan@infradead.org> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Mike Galbraith <efault@gmx.de> Cc: Paul Mackerras <paulus@samba.org> LKML-Reference: <20090624021324.GA2788@ywang-moblin2.bj.intel.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools/perf/builtin-stat.c')
0 files changed, 0 insertions, 0 deletions