From: Andi Kleen <a...@linux.intel.com> __get_ibs_caps is used by both oprofile and perf events. This causes oprofile to be dependent on perf.
__get_ibs_caps is actually only a few lines, so we can easily move that to the standard AMD CPU initialization file. It will be always compiled in this way, but it's far better than requiring perf always to be compiled in for this. Cc: b...@suse.de Signed-off-by: Andi Kleen <a...@linux.intel.com> --- arch/x86/include/asm/perf_event.h | 2 ++ arch/x86/kernel/cpu/amd.c | 27 +++++++++++++++++++++++++++ arch/x86/kernel/cpu/perf_event_amd_ibs.c | 23 ----------------------- arch/x86/oprofile/op_model_amd.c | 4 +++- 4 files changed, 32 insertions(+), 24 deletions(-) diff --git a/arch/x86/include/asm/perf_event.h b/arch/x86/include/asm/perf_event.h index 8249df4..866b0f3 100644 --- a/arch/x86/include/asm/perf_event.h +++ b/arch/x86/include/asm/perf_event.h @@ -198,6 +198,8 @@ struct x86_pmu_capability { #define IBS_OP_MAX_CNT_EXT 0x007FFFFFULL /* not a register bit mask */ #define IBS_RIP_INVALID (1ULL<<38) +extern u32 __get_ibs_caps(void); + #ifdef CONFIG_X86_LOCAL_APIC extern u32 get_ibs_caps(void); #else diff --git a/arch/x86/kernel/cpu/amd.c b/arch/x86/kernel/cpu/amd.c index 903a264..887ad1e 100644 --- a/arch/x86/kernel/cpu/amd.c +++ b/arch/x86/kernel/cpu/amd.c @@ -16,6 +16,8 @@ # include <asm/cacheflush.h> #endif +#include <asm/perf_event.h> + #include "cpu.h" static inline int rdmsrl_amd_safe(unsigned msr, unsigned long long *p) @@ -909,3 +911,28 @@ static bool cpu_has_amd_erratum(struct cpuinfo_x86 *cpu, const int *erratum) return false; } + + +/* IBS - apic initialization, for perf and oprofile */ + +u32 __get_ibs_caps(void) +{ + u32 caps; + unsigned int max_level; + + if (!boot_cpu_has(X86_FEATURE_IBS)) + return 0; + + /* check IBS cpuid feature flags */ + max_level = cpuid_eax(0x80000000); + if (max_level < IBS_CPUID_FEATURES) + return IBS_CAPS_DEFAULT; + + caps = cpuid_eax(IBS_CPUID_FEATURES); + if (!(caps & IBS_CAPS_AVAIL)) + /* cpuid flags not valid */ + return IBS_CAPS_DEFAULT; + + return caps; +} +EXPORT_SYMBOL(__get_ibs_caps); diff --git a/arch/x86/kernel/cpu/perf_event_amd_ibs.c b/arch/x86/kernel/cpu/perf_event_amd_ibs.c index e09f0bf..c4fed71 100644 --- a/arch/x86/kernel/cpu/perf_event_amd_ibs.c +++ b/arch/x86/kernel/cpu/perf_event_amd_ibs.c @@ -664,29 +664,6 @@ static __init int perf_event_ibs_init(void) { return 0; } #endif -/* IBS - apic initialization, for perf and oprofile */ - -static __init u32 __get_ibs_caps(void) -{ - u32 caps; - unsigned int max_level; - - if (!boot_cpu_has(X86_FEATURE_IBS)) - return 0; - - /* check IBS cpuid feature flags */ - max_level = cpuid_eax(0x80000000); - if (max_level < IBS_CPUID_FEATURES) - return IBS_CAPS_DEFAULT; - - caps = cpuid_eax(IBS_CPUID_FEATURES); - if (!(caps & IBS_CAPS_AVAIL)) - /* cpuid flags not valid */ - return IBS_CAPS_DEFAULT; - - return caps; -} - u32 get_ibs_caps(void) { return ibs_caps; diff --git a/arch/x86/oprofile/op_model_amd.c b/arch/x86/oprofile/op_model_amd.c index 50d86c0..964171f 100644 --- a/arch/x86/oprofile/op_model_amd.c +++ b/arch/x86/oprofile/op_model_amd.c @@ -26,6 +26,8 @@ #include <asm/processor.h> #include <asm/cpufeature.h> +#include <asm/perf_event.h> /* for __get_ibs_caps */ + #include "op_x86_model.h" #include "op_counter.h" @@ -446,7 +448,7 @@ static void op_amd_stop(struct op_msrs const * const msrs) static void init_ibs(void) { - ibs_caps = get_ibs_caps(); + ibs_caps = __get_ibs_caps(); if (!ibs_caps) return; -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/