Skip to content

Add support for memcpy_mcsafe

Balbir Singh linux-nvdimm linuxppc-dev nvdimm 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest3 patches 4 repliesIn review
  2. v1 posting 2/23 patches 14 repliesSuperseded
  3. v1 RFC posting 1/23 patches 0 repliesSuperseded

Changes in v2 from the cover letter

- Fix the logic of shifting in addr_to_pfn
- Use shift consistently instead of PAGE_SHIFT
- Fix a typo in patch1

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3powerpc/mce: Bug fixes for MCE handling in kernel spacearch/powerpc/kernel/mce_power.c+16 -101 reply
2/3powerpc/memcpy: Add memcpy_mcsafe for pmemarch/powerpc/include/asm/string.h arch/powerpc/lib/Makefile arch/powerpc/lib/memcpy_mcsafe_64.S+215 -12 replies
3/3powerpc/mce: Handle memcpy_mcsafearch/powerpc/include/asm/mce.h arch/powerpc/kernel/mce.c+77 -31 reply

Reviews trailers per patch

1 of 3 patches has a Reviewed-by or Acked-by.

PatchNicholas Piggin
1/3
2/3A-b
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/3] Add support for memcpy_mcsafe

memcpy_mcsafe() is an API currently used by the pmem subsystem to convert errors while doing a memcpy (machine check exception errors) to a return value. This patchset consists of three patches

1. The first patch is a bug fix to handle machine check errors correctly while walking the page tables in kernel mode, due to huge pmd/pud sizes
2. The second patch adds memcpy_mcsafe() support, this is largely derived from existing code
3. The third patch registers for callbacks on machine check exceptions and in them uses specialized knowledge of the type of page to decide whether to handle the MCE as is or to return to a fixup address present in memcpy_mcsafe(). If a fixup address is used, then we return an error value of -EFAULT to the caller.

Testing

A large part of the testing was done under a simulator by selectively inserting machine check exceptions in a test driver doing memcpy_mcsafe via ioctls.

Changelog v2
- Fix the logic of shifting in addr_to_pfn
- Use shift consistently instead of PAGE_SHIFT
- Fix a typo in patch1

Balbir Singh (3):
powerpc/mce: Bug fixes for MCE handling in kernel space powerpc/memcpy: Add memcpy_mcsafe for pmem powerpc/mce: Handle memcpy_mcsafe

 arch/powerpc/include/asm/mce.h      |   3 +-
 arch/powerpc/include/asm/string.h   |   2 +
 arch/powerpc/kernel/mce.c           |  77 ++++++++++++-
 arch/powerpc/kernel/mce_power.c     |  26 +++--
 arch/powerpc/lib/Makefile           |   2 +-
 arch/powerpc/lib/memcpy_mcsafe_64.S | 212 ++++++++++++++++++++++++++++++++++++
 6 files changed, 308 insertions(+), 14 deletions(-)
 create mode 100644 arch/powerpc/lib/memcpy_mcsafe_64.S
Signature · 6 lines
2.13.6

_______________________________________________
Linux-nvdimm mailing list
[email protected]
https://lists.01.org/mailman/listinfo/linux-nvdimm