diff options
| author | Alexander Shishkin <alexander.shishkin@linux.intel.com> | 2016-04-08 17:35:04 +0300 |
|---|---|---|
| committer | Alexander Shishkin <alexander.shishkin@linux.intel.com> | 2016-04-08 17:41:25 +0300 |
| commit | e2ea295baf87d78f2ed86ce595b30c691b18b210 (patch) | |
| tree | 329293f71d023e86600582c372a54b7e583b0156 | |
| parent | a45ff6ed742cdfdb3cdebee83d19ab1c00d91fcc (diff) | |
| download | op-kernel-dev-e2ea295baf87d78f2ed86ce595b30c691b18b210.zip op-kernel-dev-e2ea295baf87d78f2ed86ce595b30c691b18b210.tar.gz | |
intel_th: Hold output driver module reference while capture is active
Right now it's possible to unload the output subdevice's driver while
the capture to this output is active. Prevent this by holding the
output driver's module reference.
Signed-off-by: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Reviewed-by: Laurent Fert <laurent.fert@intel.com>
| -rw-r--r-- | drivers/hwtracing/intel_th/core.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/hwtracing/intel_th/core.c b/drivers/hwtracing/intel_th/core.c index 2033947..1be543e 100644 --- a/drivers/hwtracing/intel_th/core.c +++ b/drivers/hwtracing/intel_th/core.c @@ -189,6 +189,9 @@ static int intel_th_output_activate(struct intel_th_device *thdev) if (!thdrv) return -ENODEV; + if (!try_module_get(thdrv->driver.owner)) + return -ENODEV; + if (thdrv->activate) return thdrv->activate(thdev); @@ -209,6 +212,8 @@ static void intel_th_output_deactivate(struct intel_th_device *thdev) thdrv->deactivate(thdev); else intel_th_trace_disable(thdev); + + module_put(thdrv->driver.owner); } static ssize_t active_show(struct device *dev, struct device_attribute *attr, |
