Skip to content

Mark the namespace disabled on pfn superblock mismatch

Aneesh Kumar K.V linux-mm linux-nvdimm nvdimm 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v9 Latest6 of 7 patches 1 replyIn review

Changes in v9 from the cover letter

* updated patch 7 for addressing review feedback

6 of 7 patches are in this archive. The cover letter announces 7.

Patches in v9 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/7libnvdimm/region: Rewrite _probe_success() to _advance_seeds()drivers/nvdimm/bus.c drivers/nvdimm/namespace_devs.c drivers/nvdimm/nd-core.h +1 more+41 -710 replies
2/7libnvdimm/pmem: Advance namespace seed for specific probe errorsdrivers/nvdimm/bus.c drivers/nvdimm/pmem.c+27 -50 replies
3/7libnvdimm/pfn_dev: Add a build check to make sure we notice when struct page size changedrivers/nvdimm/nd.h drivers/nvdimm/pfn_devs.c+13 -10 replies
4/7libnvdimm/pfn_dev: Add page size and struct page size to pfn superblockdrivers/nvdimm/pfn.h drivers/nvdimm/pfn_devs.c+28 -20 replies
5/7libnvdimm/label: Remove the dpa align checkdrivers/nvdimm/label.c+0 -50 replies
6/7libnvdimm: Use PAGE_SIZE instead of SZ_4K for align checkdrivers/nvdimm/namespace_devs.c drivers/nvdimm/region_devs.c+7 -70 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v9 yet.

Read the v9 thread

Cover letter [PATCH v9 0/7] Mark the namespace disabled on pfn superblock mismatch

We add new members to pfn superblock (PAGE_SIZE and struct page size) in this series. This is now checked while initializing the namespace. If we find a mismatch we mark the namespace disabled.

This series also handle configs where hugepage support is not enabled by default. This can result in different align restrictions for dax namespace. We mark the dax namespace disabled if we find the alignment not supported.

Changes from v8:
* updated patch 7 for addressing review feedback

Changes from v6:
* Formatting changes

Changes from v5:
* Split patch 3
* Update commit message
* Add MAX_STRUCT_PAGE_SIZE with value 64 and use that when allocating reserve block
* Add BUILD_BUG_ON if we find sizeof(struct page) > 64

Aneesh Kumar K.V (6):
libnvdimm/pmem: Advance namespace seed for specific probe errors libnvdimm/pfn_dev: Add a build check to make sure we notice when
struct page size change
libnvdimm/pfn_dev: Add page size and struct page size to pfn
superblock
libnvdimm/label: Remove the dpa align check
libnvdimm: Use PAGE_SIZE instead of SZ_4K for align check libnvdimm/dax: Pick the right alignment default when creating dax
devices

Dan Williams (1):
libnvdimm/region: Rewrite _probe_success() to _advance_seeds()

 drivers/nvdimm/bus.c            |   8 +--
 drivers/nvdimm/label.c          |   5 --
 drivers/nvdimm/namespace_devs.c |  40 +++++++++---
 drivers/nvdimm/nd-core.h        |   3 +-
 drivers/nvdimm/nd.h             |  10 +--
 drivers/nvdimm/pfn.h            |   5 +-
 drivers/nvdimm/pfn_devs.c       | 110 +++++++++++++++++++++++++-------
 drivers/nvdimm/pmem.c           |  29 +++++++--
 drivers/nvdimm/region_devs.c    |  76 ++++------------------
 include/linux/huge_mm.h         |   7 +-
 10 files changed, 176 insertions(+), 117 deletions(-)
Signature · 1 line
2.21.0