diff options
| author |  Thomas Gleixner <tglx@linutronix.de>
| 2024-03-04 11:12:20 +0100 |
|---|---|---|
| committer |  Ingo Molnar <mingo@kernel.org>
| 2024-03-04 12:01:54 +0100 |
| commit | 5323922f50ecdf9d10cdd2fabd06507e5b4f3feb (patch) | |
| tree | 2fb52aeab30c15ae779290dd5775c420a30e8c45 /arch/x86/lib/msr.c | |
| parent | x86/msr: Prepare for including <linux/percpu.h> into <asm/msr.h> (diff) | |
| download | linux-5323922f50ecdf9d10cdd2fabd06507e5b4f3feb.tar.gz linux-5323922f50ecdf9d10cdd2fabd06507e5b4f3feb.tar.bz2 linux-5323922f50ecdf9d10cdd2fabd06507e5b4f3feb.zip | |
x86/msr: Add missing __percpu annotations
Sparse rightfully complains about using a plain pointer for per CPU
accessors:
msr-smp.c:15:23: sparse: warning: incorrect type in initializer (different address spaces)
msr-smp.c:15:23: sparse: expected void const [noderef] __percpu *__vpp_verify
msr-smp.c:15:23: sparse: got struct msr *
Add __percpu annotations to the related datastructure and function
arguments to cure this. This also cures the related sparse warnings at the
callsites in drivers/edac/amd64_edac.c.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lore.kernel.org/r/20240304005104.513181735@linutronix.de
Diffstat (limited to 'arch/x86/lib/msr.c')
| -rw-r--r-- | arch/x86/lib/msr.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/arch/x86/lib/msr.c b/arch/x86/lib/msr.c index 47fd9bd6b91d..4bf4fad5b148 100644 --- a/arch/x86/lib/msr.c +++ b/arch/x86/lib/msr.c @@ -6,9 +6,9 @@ #define CREATE_TRACE_POINTS #include <asm/msr-trace.h> -struct msr *msrs_alloc(void) +struct msr __percpu *msrs_alloc(void) { - struct msr *msrs = NULL; + struct msr __percpu *msrs = NULL; msrs = alloc_percpu(struct msr); if (!msrs) { @@ -20,7 +20,7 @@ struct msr *msrs_alloc(void) } EXPORT_SYMBOL(msrs_alloc); -void msrs_free(struct msr *msrs) +void msrs_free(struct msr __percpu *msrs) { free_percpu(msrs); } |