diff options
| author |   Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com> | 2019-09-05 21:16:01 +0530 |
|---|---|---|
| committer |   Dan Williams <dan.j.williams@intel.com> | 2019-09-05 16:11:14 -0700 |
| commit | 047e0eff1f150220d615c35c36af225f3e1ed057 (patch) | |
| tree | 6653fe86108ad22521742799dfb05d7fbb849896 /drivers/nvdimm/label.c | |
| parent | libnvdimm/pfn_dev: Add page size and struct page size to pfn superblock (diff) | |
| download | linux-dev-047e0eff1f150220d615c35c36af225f3e1ed057.tar.xz linux-dev-047e0eff1f150220d615c35c36af225f3e1ed057.zip | |
libnvdimm/label: Remove the dpa align check
There's no strict requirement why slot_valid() needs to check for page alignment
and it would seem to actively hurt cross-page-size compatibility. Let's
delete the check and rely on checksum validation.
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com>
Link: https://lore.kernel.org/r/20190905154603.10349-6-aneesh.kumar@linux.ibm.com
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/nvdimm/label.c')
| -rw-r--r-- | drivers/nvdimm/label.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/nvdimm/label.c b/drivers/nvdimm/label.c index 73e197babc2f..47a4828b8b31 100644 --- a/drivers/nvdimm/label.c +++ b/drivers/nvdimm/label.c @@ -353,11 +353,6 @@ static bool slot_valid(struct nvdimm_drvdata *ndd, if (slot != __le32_to_cpu(nd_label->slot)) return false; - /* check that DPA allocations are page aligned */ - if ((__le64_to_cpu(nd_label->dpa) - | __le64_to_cpu(nd_label->rawsize)) % SZ_4K) - return false; - /* check checksum */ if (namespace_label_has(ndd, checksum)) { u64 sum, sum_save; |