diff options
author | Cyrill Gorcunov <gorcunov@gmail.com> | 2010-03-18 17:12:56 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-03-18 17:17:46 -0400 |
commit | 9c8c6bad3137112d2c7bf3d215b736ee4215fa74 (patch) | |
tree | 82590bcada0d7fed60ce18c8b29b3c3837cecfc3 /arch/x86 | |
parent | 4b24a88b35e15e04bd8f2c5dda65b5dc8ebca05f (diff) |
x86, perf: Fix few cosmetic dabs for P4 pmu (comments and constantify)
- A few ESCR have escaped fixing at previous attempt.
- p4_escr_map is read only, make it const.
Nothing serious.
Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Cc: Lin Ming <ming.m.lin@intel.com>
LKML-Reference: <20100318211256.GH5062@lenovo>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/x86')
-rw-r--r-- | arch/x86/include/asm/perf_event_p4.h | 4 | ||||
-rw-r--r-- | arch/x86/kernel/cpu/perf_event_p4.c | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/arch/x86/include/asm/perf_event_p4.h b/arch/x86/include/asm/perf_event_p4.h index 871249cf4d2b..2a1a57f71539 100644 --- a/arch/x86/include/asm/perf_event_p4.h +++ b/arch/x86/include/asm/perf_event_p4.h | |||
@@ -401,13 +401,13 @@ static inline u32 p4_default_escr_conf(int cpu, int exclude_os, int exclude_usr) | |||
401 | #define P4_RETIRED_MISPRED_BRANCH_TYPE P4_EVENT_PACK(0x05, 0x02) | 401 | #define P4_RETIRED_MISPRED_BRANCH_TYPE P4_EVENT_PACK(0x05, 0x02) |
402 | /* | 402 | /* |
403 | * MSR_P4_TBPU_ESCR0: 4, 5 | 403 | * MSR_P4_TBPU_ESCR0: 4, 5 |
404 | * MSR_P4_TBPU_ESCR0: 6, 7 | 404 | * MSR_P4_TBPU_ESCR1: 6, 7 |
405 | */ | 405 | */ |
406 | 406 | ||
407 | #define P4_RETIRED_BRANCH_TYPE P4_EVENT_PACK(0x04, 0x02) | 407 | #define P4_RETIRED_BRANCH_TYPE P4_EVENT_PACK(0x04, 0x02) |
408 | /* | 408 | /* |
409 | * MSR_P4_TBPU_ESCR0: 4, 5 | 409 | * MSR_P4_TBPU_ESCR0: 4, 5 |
410 | * MSR_P4_TBPU_ESCR0: 6, 7 | 410 | * MSR_P4_TBPU_ESCR1: 6, 7 |
411 | */ | 411 | */ |
412 | 412 | ||
413 | #define P4_RESOURCE_STALL P4_EVENT_PACK(0x01, 0x01) | 413 | #define P4_RESOURCE_STALL P4_EVENT_PACK(0x01, 0x01) |
diff --git a/arch/x86/kernel/cpu/perf_event_p4.c b/arch/x86/kernel/cpu/perf_event_p4.c index b7bf9911198c..b8a811ab7609 100644 --- a/arch/x86/kernel/cpu/perf_event_p4.c +++ b/arch/x86/kernel/cpu/perf_event_p4.c | |||
@@ -545,7 +545,7 @@ static void p4_pmu_swap_config_ts(struct hw_perf_event *hwc, int cpu) | |||
545 | } | 545 | } |
546 | 546 | ||
547 | /* ESCRs are not sequential in memory so we need a map */ | 547 | /* ESCRs are not sequential in memory so we need a map */ |
548 | static unsigned int p4_escr_map[ARCH_P4_TOTAL_ESCR] = { | 548 | static const unsigned int p4_escr_map[ARCH_P4_TOTAL_ESCR] = { |
549 | MSR_P4_ALF_ESCR0, /* 0 */ | 549 | MSR_P4_ALF_ESCR0, /* 0 */ |
550 | MSR_P4_ALF_ESCR1, /* 1 */ | 550 | MSR_P4_ALF_ESCR1, /* 1 */ |
551 | MSR_P4_BPU_ESCR0, /* 2 */ | 551 | MSR_P4_BPU_ESCR0, /* 2 */ |