Commit 8b1b41ee authored by Linus Torvalds's avatar Linus Torvalds

Merge branch 'libnvdimm-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm

Pull libnvdimm fixes from Dan Williams:
 "None of these are showstoppers for 4.10 and could wait for 4.11 merge
  window, but they are low enough risk for this late in the cycle and
  the fixes have waiting users . They have received a build success
  notification from the 0day robot, pass the latest ndctl unit tests,
  and appeared in next:

   - Fix a crash that can result when SIGINT is sent to a process that
     is awaiting completion of an address range scrub command. We were
     not properly cleaning up the workqueue after
     wait_event_interruptible().

   - Fix a memory hotplug failure condition that results from not
     reserving enough space out of persistent memory for the memmap. By
     default we align to 2M allocations that the memory hotplug code
     assumes, but if the administrator specifies a non-default
     4K-alignment then we can fail to correctly size the reservation.

   - A one line fix to improve the predictability of libnvdimm block
     device names. A common operation is to reconfigure /dev/pmem0 into
     a different mode. For example, a reconfiguration might set a new
     mode that reserves some of the capacity for a struct page memmap
     array. It surprises users if the device name changes to
     "/dev/pmem0.1" after the mode change and then back to /dev/pmem0
     after a reboot.

   - Add 'const' to some function pointer tables"

* 'libnvdimm-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm:
  libnvdimm, pfn: fix memmap reservation size versus 4K alignment
  acpi, nfit: fix acpi_nfit_flush_probe() crash
  libnvdimm, namespace: do not delete namespace-id 0
  nvdimm: constify device_type structures
parents f7d6040a bfb34527
...@@ -2704,6 +2704,7 @@ static int acpi_nfit_flush_probe(struct nvdimm_bus_descriptor *nd_desc) ...@@ -2704,6 +2704,7 @@ static int acpi_nfit_flush_probe(struct nvdimm_bus_descriptor *nd_desc)
struct acpi_nfit_desc *acpi_desc = to_acpi_nfit_desc(nd_desc); struct acpi_nfit_desc *acpi_desc = to_acpi_nfit_desc(nd_desc);
struct device *dev = acpi_desc->dev; struct device *dev = acpi_desc->dev;
struct acpi_nfit_flush_work flush; struct acpi_nfit_flush_work flush;
int rc;
/* bounce the device lock to flush acpi_nfit_add / acpi_nfit_notify */ /* bounce the device lock to flush acpi_nfit_add / acpi_nfit_notify */
device_lock(dev); device_lock(dev);
...@@ -2716,7 +2717,10 @@ static int acpi_nfit_flush_probe(struct nvdimm_bus_descriptor *nd_desc) ...@@ -2716,7 +2717,10 @@ static int acpi_nfit_flush_probe(struct nvdimm_bus_descriptor *nd_desc)
INIT_WORK_ONSTACK(&flush.work, flush_probe); INIT_WORK_ONSTACK(&flush.work, flush_probe);
COMPLETION_INITIALIZER_ONSTACK(flush.cmp); COMPLETION_INITIALIZER_ONSTACK(flush.cmp);
queue_work(nfit_wq, &flush.work); queue_work(nfit_wq, &flush.work);
return wait_for_completion_interruptible(&flush.cmp);
rc = wait_for_completion_interruptible(&flush.cmp);
cancel_work_sync(&flush.work);
return rc;
} }
static int acpi_nfit_clear_to_send(struct nvdimm_bus_descriptor *nd_desc, static int acpi_nfit_clear_to_send(struct nvdimm_bus_descriptor *nd_desc,
......
...@@ -52,17 +52,17 @@ static void namespace_blk_release(struct device *dev) ...@@ -52,17 +52,17 @@ static void namespace_blk_release(struct device *dev)
kfree(nsblk); kfree(nsblk);
} }
static struct device_type namespace_io_device_type = { static const struct device_type namespace_io_device_type = {
.name = "nd_namespace_io", .name = "nd_namespace_io",
.release = namespace_io_release, .release = namespace_io_release,
}; };
static struct device_type namespace_pmem_device_type = { static const struct device_type namespace_pmem_device_type = {
.name = "nd_namespace_pmem", .name = "nd_namespace_pmem",
.release = namespace_pmem_release, .release = namespace_pmem_release,
}; };
static struct device_type namespace_blk_device_type = { static const struct device_type namespace_blk_device_type = {
.name = "nd_namespace_blk", .name = "nd_namespace_blk",
.release = namespace_blk_release, .release = namespace_blk_release,
}; };
...@@ -962,8 +962,8 @@ static ssize_t __size_store(struct device *dev, unsigned long long val) ...@@ -962,8 +962,8 @@ static ssize_t __size_store(struct device *dev, unsigned long long val)
struct nvdimm_drvdata *ndd; struct nvdimm_drvdata *ndd;
struct nd_label_id label_id; struct nd_label_id label_id;
u32 flags = 0, remainder; u32 flags = 0, remainder;
int rc, i, id = -1;
u8 *uuid = NULL; u8 *uuid = NULL;
int rc, i;
if (dev->driver || ndns->claim) if (dev->driver || ndns->claim)
return -EBUSY; return -EBUSY;
...@@ -972,11 +972,13 @@ static ssize_t __size_store(struct device *dev, unsigned long long val) ...@@ -972,11 +972,13 @@ static ssize_t __size_store(struct device *dev, unsigned long long val)
struct nd_namespace_pmem *nspm = to_nd_namespace_pmem(dev); struct nd_namespace_pmem *nspm = to_nd_namespace_pmem(dev);
uuid = nspm->uuid; uuid = nspm->uuid;
id = nspm->id;
} else if (is_namespace_blk(dev)) { } else if (is_namespace_blk(dev)) {
struct nd_namespace_blk *nsblk = to_nd_namespace_blk(dev); struct nd_namespace_blk *nsblk = to_nd_namespace_blk(dev);
uuid = nsblk->uuid; uuid = nsblk->uuid;
flags = NSLABEL_FLAG_LOCAL; flags = NSLABEL_FLAG_LOCAL;
id = nsblk->id;
} }
/* /*
...@@ -1039,10 +1041,11 @@ static ssize_t __size_store(struct device *dev, unsigned long long val) ...@@ -1039,10 +1041,11 @@ static ssize_t __size_store(struct device *dev, unsigned long long val)
/* /*
* Try to delete the namespace if we deleted all of its * Try to delete the namespace if we deleted all of its
* allocation, this is not the seed device for the region, and * allocation, this is not the seed or 0th device for the
* it is not actively claimed by a btt instance. * region, and it is not actively claimed by a btt, pfn, or dax
* instance.
*/ */
if (val == 0 && nd_region->ns_seed != dev && !ndns->claim) if (val == 0 && id != 0 && nd_region->ns_seed != dev && !ndns->claim)
nd_device_unregister(dev, ND_ASYNC); nd_device_unregister(dev, ND_ASYNC);
return rc; return rc;
......
...@@ -627,15 +627,12 @@ static int nd_pfn_init(struct nd_pfn *nd_pfn) ...@@ -627,15 +627,12 @@ static int nd_pfn_init(struct nd_pfn *nd_pfn)
size = resource_size(&nsio->res); size = resource_size(&nsio->res);
npfns = (size - start_pad - end_trunc - SZ_8K) / SZ_4K; npfns = (size - start_pad - end_trunc - SZ_8K) / SZ_4K;
if (nd_pfn->mode == PFN_MODE_PMEM) { if (nd_pfn->mode == PFN_MODE_PMEM) {
unsigned long memmap_size;
/* /*
* vmemmap_populate_hugepages() allocates the memmap array in * vmemmap_populate_hugepages() allocates the memmap array in
* HPAGE_SIZE chunks. * HPAGE_SIZE chunks.
*/ */
memmap_size = ALIGN(64 * npfns, HPAGE_SIZE); offset = ALIGN(start + SZ_8K + 64 * npfns + dax_label_reserve,
offset = ALIGN(start + SZ_8K + memmap_size + dax_label_reserve, max(nd_pfn->align, HPAGE_SIZE)) - start;
nd_pfn->align) - start;
} else if (nd_pfn->mode == PFN_MODE_RAM) } else if (nd_pfn->mode == PFN_MODE_RAM)
offset = ALIGN(start + SZ_8K + dax_label_reserve, offset = ALIGN(start + SZ_8K + dax_label_reserve,
nd_pfn->align) - start; nd_pfn->align) - start;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment