diff options
| author | Alexander Duyck <alexander.h.duyck@linux.intel.com> | 2018-09-25 22:53:02 +0200 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2018-09-26 21:02:31 +0200 |
| commit | b6eae0f61db27748606cc00dafcfd1e2c032f0a5 (patch) | |
| tree | 98569b815acec26be1d8a78ddfcecd4ac8cd60c7 /drivers/nvdimm/bus.c | |
| parent | libnvdimm: remove duplicate include (diff) | |
| download | linux-b6eae0f61db27748606cc00dafcfd1e2c032f0a5.tar.xz linux-b6eae0f61db27748606cc00dafcfd1e2c032f0a5.zip | |
libnvdimm: Hold reference on parent while scheduling async init
Unlike asynchronous initialization in the core we have not yet associated
the device with the parent, and as such the device doesn't hold a reference
to the parent.
In order to resolve that we should be holding a reference on the parent
until the asynchronous initialization has completed.
Cc: <stable@vger.kernel.org>
Fixes: 4d88a97aa9e8 ("libnvdimm: ...base ... infrastructure")
Signed-off-by: Alexander Duyck <alexander.h.duyck@linux.intel.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, 4 insertions, 0 deletions
diff --git a/drivers/nvdimm/bus.c b/drivers/nvdimm/bus.c index 8aae6dcc839f..9148015ed803 100644 --- a/drivers/nvdimm/bus.c +++ b/drivers/nvdimm/bus.c @@ -488,6 +488,8 @@ static void nd_async_device_register(void *d, async_cookie_t cookie) put_device(dev); } put_device(dev); + if (dev->parent) + put_device(dev->parent); } static void nd_async_device_unregister(void *d, async_cookie_t cookie) @@ -507,6 +509,8 @@ void __nd_device_register(struct device *dev) if (!dev) return; dev->bus = &nvdimm_bus_type; + if (dev->parent) + get_device(dev->parent); get_device(dev); async_schedule_domain(nd_async_device_register, dev, &nd_async_domain); |