diff options
author | Dan Williams <dan.j.williams@intel.com> | 2015-06-30 12:07:17 -0400 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2015-06-30 12:07:17 -0400 |
commit | 31f02455455d405320e2f749696bef4e02903b35 (patch) | |
tree | 300b811ed1e6a13d58d83f3117b37bceda92c800 | |
parent | 88793e5c774ec69351ef6b5200bb59f532e41bca (diff) | |
download | op-kernel-dev-31f02455455d405320e2f749696bef4e02903b35.zip op-kernel-dev-31f02455455d405320e2f749696bef4e02903b35.tar.gz |
sparse: fix misplaced __pmem definition
Move the definition of __pmem outside of CONFIG_SPARSE_RCU_POINTER to fix:
drivers/nvdimm/pmem.c:198:17: sparse: too many arguments for function __builtin_expect
drivers/nvdimm/pmem.c:36:33: sparse: expected ; at end of declaration
drivers/nvdimm/pmem.c:48:21: sparse: void declaration
...due to __pmem failing to be defined in some configurations when
CONFIG_SPARSE_RCU_POINTER=y.
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
-rw-r--r-- | include/linux/compiler.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/compiler.h b/include/linux/compiler.h index 26fc8bc..d8fbd50 100644 --- a/include/linux/compiler.h +++ b/include/linux/compiler.h @@ -17,11 +17,11 @@ # define __release(x) __context__(x,-1) # define __cond_lock(x,c) ((c) ? ({ __acquire(x); 1; }) : 0) # define __percpu __attribute__((noderef, address_space(3))) +# define __pmem __attribute__((noderef, address_space(5))) #ifdef CONFIG_SPARSE_RCU_POINTER # define __rcu __attribute__((noderef, address_space(4))) #else # define __rcu -# define __pmem __attribute__((noderef, address_space(5))) #endif extern void __chk_user_ptr(const volatile void __user *); extern void __chk_io_ptr(const volatile void __iomem *); |