[GIT,PULL] nvdimm fixes v5.19-rc5

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm.git tags/libnvdimm-fixes-5.19-rc5

Commit Message

Verma, Vishal L July 1, 2022, 11:07 p.m. UTC

Hi Linus, please pull from:

  git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm
tags/libnvdimm-fixes-5.19-rc5

...to receive a fix for v5.19-rc5. It has been in -next for
a week with no reported issues.

---

The following changes since commit a111daf0c53ae91e71fd2bfe7497862d14132e3e:

  Linux 5.19-rc3 (2022-06-19 15:06:47 -0500)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm.git tags/libnvdimm-fixes-5.19-rc5

for you to fetch changes up to ef9102004a87cb3f8b26e000a095a261fc0467d3:

  nvdimm: Fix badblocks clear off-by-one error (2022-06-24 11:57:19 -0700)

----------------------------------------------------------------
libnvdimm fixes for v5.19-rc5

- Fix a bug in the libnvdimm 'BTT' (Block Translation Table) driver
  where accounting for poison blocks to be cleared was off by one,
  causing a failure to clear the the last badblock in an nvdimm region.

----------------------------------------------------------------
Chris Ye (1):
      nvdimm: Fix badblocks clear off-by-one error

 drivers/nvdimm/bus.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

[e-mail] July 2, 2022, 12:02 a.m. UTC | #1

The pull request you sent on Fri, 1 Jul 2022 23:07:26 +0000:

> git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm tags/libnvdimm-fixes-5.19-rc5

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/089866061428ec9bf67221247c936792078c41a4

Thank you!

Patch

diff --git a/drivers/nvdimm/bus.c b/drivers/nvdimm/bus.c
index a4fc17db707c..b38d0355b0ac 100644
--- a/drivers/nvdimm/bus.c
+++ b/drivers/nvdimm/bus.c
@@ -176,8 +176,8 @@  static int nvdimm_clear_badblocks_region(struct device *dev, void *data)
 	ndr_end = nd_region->ndr_start + nd_region->ndr_size - 1;
 
 	/* make sure we are in the region */
-	if (ctx->phys < nd_region->ndr_start
-			|| (ctx->phys + ctx->cleared) > ndr_end)
+	if (ctx->phys < nd_region->ndr_start ||
+	    (ctx->phys + ctx->cleared - 1) > ndr_end)
 		return 0;
 
 	sector = (ctx->phys - nd_region->ndr_start) / 512;