x86, perf: Fix few cosmetic dabs for P4 pmu (comments and constantify)
authorCyrill Gorcunov <gorcunov@gmail.com>
Thu, 18 Mar 2010 21:12:56 +0000 (00:12 +0300)
committerIngo Molnar <mingo@elte.hu>
Thu, 18 Mar 2010 21:17:46 +0000 (22:17 +0100)
- 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>
arch/x86/include/asm/perf_event_p4.h
arch/x86/kernel/cpu/perf_event_p4.c

index 871249c..2a1a57f 100644 (file)
@@ -401,13 +401,13 @@ static inline u32 p4_default_escr_conf(int cpu, int exclude_os, int exclude_usr)
 #define P4_RETIRED_MISPRED_BRANCH_TYPE P4_EVENT_PACK(0x05, 0x02)
        /*
         * MSR_P4_TBPU_ESCR0:   4, 5
-        * MSR_P4_TBPU_ESCR0:   6, 7
+        * MSR_P4_TBPU_ESCR1:   6, 7
         */
 
 #define P4_RETIRED_BRANCH_TYPE         P4_EVENT_PACK(0x04, 0x02)
        /*
         * MSR_P4_TBPU_ESCR0:   4, 5
-        * MSR_P4_TBPU_ESCR0:   6, 7
+        * MSR_P4_TBPU_ESCR1:   6, 7
         */
 
 #define P4_RESOURCE_STALL              P4_EVENT_PACK(0x01, 0x01)
index b7bf991..b8a811a 100644 (file)
@@ -545,7 +545,7 @@ static void p4_pmu_swap_config_ts(struct hw_perf_event *hwc, int cpu)
 }
 
 /* ESCRs are not sequential in memory so we need a map */
-static unsigned int p4_escr_map[ARCH_P4_TOTAL_ESCR] = {
+static const unsigned int p4_escr_map[ARCH_P4_TOTAL_ESCR] = {
        MSR_P4_ALF_ESCR0,       /*  0 */
        MSR_P4_ALF_ESCR1,       /*  1 */
        MSR_P4_BPU_ESCR0,       /*  2 */