diff options
| author | Toshi Kani <toshi.kani@hpe.com> | 2017-04-27 16:57:05 -0600 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2017-04-28 15:56:26 -0700 |
| commit | 8d13c0290655b883df9083a2a0af0d782bc38aef (patch) | |
| tree | 21cff984f2c5be871fdcceb749b077b78562d2cd /drivers/nvdimm/bus.c | |
| parent | b2518c78ce76896f0f8f7940bf02104b227e1709 (diff) | |
| download | talos-op-linux-8d13c0290655b883df9083a2a0af0d782bc38aef.tar.gz talos-op-linux-8d13c0290655b883df9083a2a0af0d782bc38aef.zip | |
libnvdimm: fix clear length of nvdimm_forget_poison()
ND_CMD_CLEAR_ERROR command returns 'clear_err.cleared', the length
of error actually cleared, which may be smaller than its requested
'len'.
Change nvdimm_clear_poison() to call nvdimm_forget_poison() with
'clear_err.cleared' when this value is valid.
Cc: <stable@vger.kernel.org>
Fixes: e046114af5fc ("libnvdimm: clear the internal poison_list when clearing badblocks")
Cc: Dave Jiang <dave.jiang@intel.com>
Cc: Vishal Verma <vishal.l.verma@intel.com>
Signed-off-by: Toshi Kani <toshi.kani@hpe.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/nvdimm/bus.c')
| -rw-r--r-- | drivers/nvdimm/bus.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/nvdimm/bus.c b/drivers/nvdimm/bus.c index d214ac44d111..43ddfd487c85 100644 --- a/drivers/nvdimm/bus.c +++ b/drivers/nvdimm/bus.c @@ -219,7 +219,9 @@ long nvdimm_clear_poison(struct device *dev, phys_addr_t phys, if (cmd_rc < 0) return cmd_rc; - nvdimm_forget_poison(nvdimm_bus, phys, len); + if (clear_err.cleared > 0) + nvdimm_forget_poison(nvdimm_bus, phys, clear_err.cleared); + return clear_err.cleared; } EXPORT_SYMBOL_GPL(nvdimm_clear_poison); |
