diff options
author | Robert Richter <robert.richter@amd.com> | 2008-10-12 15:12:34 -0400 |
---|---|---|
committer | Robert Richter <robert.richter@amd.com> | 2009-06-10 16:09:24 -0400 |
commit | e419294ed3c98cccc145202e4fe165bfd8099d63 (patch) | |
tree | 24eced48e81cc31765ac1a3a87eb75bed583e1fc /arch/x86/oprofile/op_model_ppro.c | |
parent | 849620fab413355eff48232eac5a8c53c57615c5 (diff) |
x86/oprofile: moving arch_perfmon counter setup to op_x86_model_spec.init
The function arch_perfmon_init() in nmi_int.c is model specific. This
patch moves it to op_model_ppro.c by using the init function pointer
in struct op_x86_model_spec.
Cc: Andi Kleen <ak@linux.intel.com>
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 | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/arch/x86/oprofile/op_model_ppro.c b/arch/x86/oprofile/op_model_ppro.c index 2a123990a84c..ae5811966883 100644 --- a/arch/x86/oprofile/op_model_ppro.c +++ b/arch/x86/oprofile/op_model_ppro.c | |||
@@ -233,7 +233,7 @@ struct op_x86_model_spec const op_ppro_spec = { | |||
233 | * the specific CPU. | 233 | * the specific CPU. |
234 | */ | 234 | */ |
235 | 235 | ||
236 | void arch_perfmon_setup_counters(void) | 236 | static void arch_perfmon_setup_counters(void) |
237 | { | 237 | { |
238 | union cpuid10_eax eax; | 238 | union cpuid10_eax eax; |
239 | 239 | ||
@@ -253,7 +253,14 @@ void arch_perfmon_setup_counters(void) | |||
253 | op_arch_perfmon_spec.num_controls = num_counters; | 253 | op_arch_perfmon_spec.num_controls = num_counters; |
254 | } | 254 | } |
255 | 255 | ||
256 | static int arch_perfmon_init(struct oprofile_operations *ignore) | ||
257 | { | ||
258 | arch_perfmon_setup_counters(); | ||
259 | return 0; | ||
260 | } | ||
261 | |||
256 | struct op_x86_model_spec op_arch_perfmon_spec = { | 262 | struct op_x86_model_spec op_arch_perfmon_spec = { |
263 | .init = &arch_perfmon_init, | ||
257 | /* num_counters/num_controls filled in at runtime */ | 264 | /* num_counters/num_controls filled in at runtime */ |
258 | .fill_in_addresses = &ppro_fill_in_addresses, | 265 | .fill_in_addresses = &ppro_fill_in_addresses, |
259 | /* user space does the cpuid check for available events */ | 266 | /* user space does the cpuid check for available events */ |