On 4/29/2025 2:45 AM, Ilpo Järvinen wrote:
arch/x86/events/msr.c | 3 +++
arch/x86/events/perf_event.h | 1 +
arch/x86/events/probe.c | 2 ++
Under arch/x86/events/ a few files seem to be missing the include?
Most C files in arch/x86/events/ include arch/x86/events/perf_event.h,
thus they don't need to include <asm/msr.h> directly once
arch/x86/events/perf_event.h includes <asm/msr.h>, and this patch does
that.
The following files include arch/x86/events/intel/uncore.h which
includes arch/x86/events/perf_event.h, thus no change needed:
arch/x86/events/intel/uncore.c
arch/x86/events/intel/uncore_discovery.c
arch/x86/events/intel/uncore_nhmex.c
arch/x86/events/intel/uncore_snb.c
arch/x86/events/intel/uncore_snbep.c
The following 2 files don't include arch/x86/events/perf_event.h so they
include <asm/msr.h> directly with this patch:
arch/x86/events/msr.c
arch/x86/events/probe.c
arch/x86/events/amd/uncore.c doesn't include
arch/x86/events/perf_event.h but includes <asm/msr.h> already.
So we are good in this directory, but it should be a separate patch with
the above explanation then.