[<prev] [next>] [day] [month] [year] [list]
Message-ID: <160996560856.3973505.10433205523993096439.stgit@djiang5-desk3.ch.intel.com>
Date: Wed, 06 Jan 2021 13:40:08 -0700
From: Dave Jiang <dave.jiang@...el.com>
To: bp@...en8.de, x86@...nel.org
Cc: kernel test robot <lkp@...el.com>,
Ben Widawsky <ben.widawsky@...el.com>,
dan.j.williams@...el.com, ben.widawsky@...el.com,
tglx@...utronix.de, mingo@...hat.com, hpa@...or.com,
linux-kernel@...r.kernel.org
Subject: [PATCH] x86: fix enqcmds() sparse warning
Add missing __iomem anotation to address sparse warning.
"sparse warnings: (new ones prefixed by >>)"
drivers/dma/idxd/submit.c: note: in included file (through include/linux/io.h, include/linux/pci.h):
arch/x86/include/asm/io.h:422:27: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void *dst @@ got void [noderef] __iomem *dst @@
arch/x86/include/asm/io.h:422:27: sparse: expected void *dst
arch/x86/include/asm/io.h:422:27: sparse: got void [noderef] __iomem *dst
drivers/dma/idxd/submit.c: note: in included file (through arch/x86/include/asm/processor.h, arch/x86/include/asm/timex.h, include/linux/timex.h, ...):
>> arch/x86/include/asm/special_insns.h:289:41: sparse: sparse: incorrect type in initializer (different address spaces) @@ expected struct <noident> *__dst @@ got void [noderef] __iomem *dst @@
arch/x86/include/asm/special_insns.h:289:41: sparse: expected struct <noident> *__dst
arch/x86/include/asm/special_insns.h:289:41: sparse: got void [noderef] __iomem *dst
Fixes: 7f5933f81bd8 ("x86/asm: Add an enqcmds() wrapper for the ENQCMDS instruction")
Reported-by: kernel test robot <lkp@...el.com>
Reviewed-by: Ben Widawsky <ben.widawsky@...el.com>
Signed-off-by: Dave Jiang <dave.jiang@...el.com>
---
arch/x86/include/asm/special_insns.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/include/asm/special_insns.h b/arch/x86/include/asm/special_insns.h
index 4e234645f0c6..1d3cbaef4bb7 100644
--- a/arch/x86/include/asm/special_insns.h
+++ b/arch/x86/include/asm/special_insns.h
@@ -286,7 +286,7 @@ static inline void movdir64b(void __iomem *dst, const void *src)
static inline int enqcmds(void __iomem *dst, const void *src)
{
const struct { char _[64]; } *__src = src;
- struct { char _[64]; } *__dst = dst;
+ struct { char _[64]; } __iomem *__dst = dst;
int zf;
/*
Powered by blists - more mailing lists