Commit cba2e47a authored by Toshi Kani's avatar Toshi Kani Committed by Ross Zwisler

pmem: fix BUG() error in pmem.h:48 on X86_32

After 'commit fc0c2028 ("x86, pmem: use memcpy_mcsafe()
for memcpy_from_pmem()")', probing a PMEM device hits the BUG()
error below on X86_32 kernel.

 kernel BUG at include/linux/pmem.h:48!

memcpy_from_pmem() calls arch_memcpy_from_pmem(), which is
unimplemented since CONFIG_ARCH_HAS_PMEM_API is undefined on
X86_32.

Fix the BUG() error by adding default_memcpy_from_pmem().
Acked-by: default avatarDan Williams <dan.j.williams@intel.com>
Signed-off-by: default avatarToshi Kani <toshi.kani@hpe.com>
Signed-off-by: default avatarRoss Zwisler <ross.zwisler@linux.intel.com>
Cc: Dan Williams <dan.j.williams@intel.com>
Cc: Ross Zwisler <ross.zwisler@linux.intel.com>
parent bf162006
...@@ -72,6 +72,18 @@ static inline void arch_invalidate_pmem(void __pmem *addr, size_t size) ...@@ -72,6 +72,18 @@ static inline void arch_invalidate_pmem(void __pmem *addr, size_t size)
} }
#endif #endif
static inline bool arch_has_pmem_api(void)
{
return IS_ENABLED(CONFIG_ARCH_HAS_PMEM_API);
}
static inline int default_memcpy_from_pmem(void *dst, void __pmem const *src,
size_t size)
{
memcpy(dst, (void __force *) src, size);
return 0;
}
/* /*
* memcpy_from_pmem - read from persistent memory with error handling * memcpy_from_pmem - read from persistent memory with error handling
* @dst: destination buffer * @dst: destination buffer
...@@ -83,12 +95,10 @@ static inline void arch_invalidate_pmem(void __pmem *addr, size_t size) ...@@ -83,12 +95,10 @@ static inline void arch_invalidate_pmem(void __pmem *addr, size_t size)
static inline int memcpy_from_pmem(void *dst, void __pmem const *src, static inline int memcpy_from_pmem(void *dst, void __pmem const *src,
size_t size) size_t size)
{ {
return arch_memcpy_from_pmem(dst, src, size); if (arch_has_pmem_api())
} return arch_memcpy_from_pmem(dst, src, size);
else
static inline bool arch_has_pmem_api(void) return default_memcpy_from_pmem(dst, src, size);
{
return IS_ENABLED(CONFIG_ARCH_HAS_PMEM_API);
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment