diff options
author | Robert Richter <robert.richter@amd.com> | 2009-06-03 20:36:44 -0400 |
---|---|---|
committer | Robert Richter <robert.richter@amd.com> | 2009-06-11 13:42:16 -0400 |
commit | 217d3cfb959756cb493fc03106c0253baa420ce8 (patch) | |
tree | 05b5531feba327812b7c791c1077a7225c526fbd /arch/x86/oprofile/op_model_ppro.c | |
parent | dea3766ca052a4f572b16a23a322553c064d75af (diff) |
x86/oprofile: replace CTR*_IS_RESERVED macros
The patch replaces all CTR*_IS_RESERVED macros.
Signed-off-by: Robert Richter <robert.richter@amd.com>
Diffstat (limited to 'arch/x86/oprofile/op_model_ppro.c')
-rw-r--r-- | arch/x86/oprofile/op_model_ppro.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/arch/x86/oprofile/op_model_ppro.c b/arch/x86/oprofile/op_model_ppro.c index 566b43f0b6c6..0a261a5c696e 100644 --- a/arch/x86/oprofile/op_model_ppro.c +++ b/arch/x86/oprofile/op_model_ppro.c | |||
@@ -82,7 +82,7 @@ static void ppro_setup_ctrs(struct op_x86_model_spec const *model, | |||
82 | 82 | ||
83 | /* clear all counters */ | 83 | /* clear all counters */ |
84 | for (i = 0 ; i < num_counters; ++i) { | 84 | for (i = 0 ; i < num_counters; ++i) { |
85 | if (unlikely(!CTRL_IS_RESERVED(msrs, i))) | 85 | if (unlikely(!msrs->controls[i].addr)) |
86 | continue; | 86 | continue; |
87 | rdmsrl(msrs->controls[i].addr, val); | 87 | rdmsrl(msrs->controls[i].addr, val); |
88 | val &= model->reserved; | 88 | val &= model->reserved; |
@@ -91,14 +91,14 @@ static void ppro_setup_ctrs(struct op_x86_model_spec const *model, | |||
91 | 91 | ||
92 | /* avoid a false detection of ctr overflows in NMI handler */ | 92 | /* avoid a false detection of ctr overflows in NMI handler */ |
93 | for (i = 0; i < num_counters; ++i) { | 93 | for (i = 0; i < num_counters; ++i) { |
94 | if (unlikely(!CTR_IS_RESERVED(msrs, i))) | 94 | if (unlikely(!msrs->counters[i].addr)) |
95 | continue; | 95 | continue; |
96 | wrmsrl(msrs->counters[i].addr, -1LL); | 96 | wrmsrl(msrs->counters[i].addr, -1LL); |
97 | } | 97 | } |
98 | 98 | ||
99 | /* enable active counters */ | 99 | /* enable active counters */ |
100 | for (i = 0; i < num_counters; ++i) { | 100 | for (i = 0; i < num_counters; ++i) { |
101 | if ((counter_config[i].enabled) && (CTR_IS_RESERVED(msrs, i))) { | 101 | if (counter_config[i].enabled && msrs->counters[i].addr) { |
102 | reset_value[i] = counter_config[i].count; | 102 | reset_value[i] = counter_config[i].count; |
103 | wrmsrl(msrs->counters[i].addr, -reset_value[i]); | 103 | wrmsrl(msrs->counters[i].addr, -reset_value[i]); |
104 | rdmsrl(msrs->controls[i].addr, val); | 104 | rdmsrl(msrs->controls[i].addr, val); |
@@ -181,11 +181,11 @@ static void ppro_shutdown(struct op_msrs const * const msrs) | |||
181 | int i; | 181 | int i; |
182 | 182 | ||
183 | for (i = 0 ; i < num_counters ; ++i) { | 183 | for (i = 0 ; i < num_counters ; ++i) { |
184 | if (CTR_IS_RESERVED(msrs, i)) | 184 | if (msrs->counters[i].addr) |
185 | release_perfctr_nmi(MSR_P6_PERFCTR0 + i); | 185 | release_perfctr_nmi(MSR_P6_PERFCTR0 + i); |
186 | } | 186 | } |
187 | for (i = 0 ; i < num_counters ; ++i) { | 187 | for (i = 0 ; i < num_counters ; ++i) { |
188 | if (CTRL_IS_RESERVED(msrs, i)) | 188 | if (msrs->controls[i].addr) |
189 | release_evntsel_nmi(MSR_P6_EVNTSEL0 + i); | 189 | release_evntsel_nmi(MSR_P6_EVNTSEL0 + i); |
190 | } | 190 | } |
191 | if (reset_value) { | 191 | if (reset_value) { |