summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Williams <dan.j.williams@intel.com>2016-01-09 07:48:43 -0800
committerDan Williams <dan.j.williams@intel.com>2016-01-09 08:39:04 -0800
commit87ba05dff3510f9e058b35d3c3fa222b6f406ecc (patch)
tree966538c52bd0b248fad9f9656e3464e9a728caa4
parent16263ff6c72eb4cc00aa287230144dda12ccad12 (diff)
libnvdimm: don't fail init for full badblocks list
If the badblocks list runs out of space it simply means that software is unable to intercept all errors. This is no different than the latent discovery of new badblocks case and should not be an initialization failure condition. Signed-off-by: Dan Williams <dan.j.williams@intel.com>
-rw-r--r--drivers/nvdimm/core.c27
1 files changed, 16 insertions, 11 deletions
diff --git a/drivers/nvdimm/core.c b/drivers/nvdimm/core.c
index 21003b7f0b38..e419d661e294 100644
--- a/drivers/nvdimm/core.c
+++ b/drivers/nvdimm/core.c
@@ -360,6 +360,18 @@ struct nvdimm_bus *__nvdimm_bus_register(struct device *parent,
}
EXPORT_SYMBOL_GPL(__nvdimm_bus_register);
+static void set_badblock(struct gendisk *disk, sector_t s, int num)
+{
+ struct device *dev = disk->driverfs_dev;
+
+ dev_dbg(dev, "Found a poison range (0x%llx, 0x%llx)\n",
+ (u64) s * 512, (u64) num * 512);
+ /* this isn't an error as the hardware will still throw an exception */
+ if (disk_set_badblocks(disk, s, num))
+ dev_info_once(dev, "%s: failed for sector %llx\n",
+ __func__, (u64) s);
+}
+
/**
* __add_badblock_range() - Convert a physical address range to bad sectors
* @disk: the disk associated with the namespace
@@ -396,15 +408,14 @@ static int __add_badblock_range(struct gendisk *disk, u64 ns_offset, u64 len)
while (remaining) {
int done = min_t(u64, remaining, INT_MAX);
- rc = disk_set_badblocks(disk, s, done);
- if (rc)
- return rc;
+ set_badblock(disk, s, done);
remaining -= done;
s += done;
}
- return 0;
} else
- return disk_set_badblocks(disk, start_sector, num_sectors);
+ set_badblock(disk, start_sector, num_sectors);
+
+ return 0;
}
/**
@@ -463,9 +474,6 @@ int nvdimm_namespace_add_poison(struct gendisk *disk, resource_size_t offset,
rc = __add_badblock_range(disk, start - ns_start, len);
if (rc)
return rc;
- dev_info(&nvdimm_bus->dev,
- "Found a poison range (0x%llx, 0x%llx)\n",
- start, len);
continue;
}
/* Deal with overlap for poison starting before the namespace */
@@ -480,9 +488,6 @@ int nvdimm_namespace_add_poison(struct gendisk *disk, resource_size_t offset,
rc = __add_badblock_range(disk, 0, len);
if (rc)
return rc;
- dev_info(&nvdimm_bus->dev,
- "Found a poison range (0x%llx, 0x%llx)\n",
- pl->start, len);
}
}