diff options
| author | Axel Lin <axel.lin@ingics.com> | 2015-09-16 16:25:38 +0300 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2015-09-17 18:47:50 +0300 |
| commit | 4ca8b57a0af145f4e791f21dbca6ad789da9ee8b (patch) | |
| tree | 22508d5613e87afd825dddfa0882214f20f879a4 /drivers/nvdimm/pfn_devs.c | |
| parent | 4be9c1fc3df9c3b03c9bde8aec5e44fc73996a3f (diff) | |
| download | linux-4ca8b57a0af145f4e791f21dbca6ad789da9ee8b.tar.xz | |
libnvdimm: pfn_devs: Fix locking in namespace_store
Always take device_lock() before nvdimm_bus_lock() to prevent deadlock.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/nvdimm/pfn_devs.c')
| -rw-r--r-- | drivers/nvdimm/pfn_devs.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/nvdimm/pfn_devs.c b/drivers/nvdimm/pfn_devs.c index 3fd7d0d81a47..71805a1aa0f3 100644 --- a/drivers/nvdimm/pfn_devs.c +++ b/drivers/nvdimm/pfn_devs.c @@ -148,13 +148,13 @@ static ssize_t namespace_store(struct device *dev, struct nd_pfn *nd_pfn = to_nd_pfn(dev); ssize_t rc; - nvdimm_bus_lock(dev); device_lock(dev); + nvdimm_bus_lock(dev); rc = nd_namespace_store(dev, &nd_pfn->ndns, buf, len); dev_dbg(dev, "%s: result: %zd wrote: %s%s", __func__, rc, buf, buf[len - 1] == '\n' ? "" : "\n"); - device_unlock(dev); nvdimm_bus_unlock(dev); + device_unlock(dev); return rc; } |