diff options
| author | Dan Williams <dan.j.williams@intel.com> | 2008-12-03 19:17:07 -0500 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2008-12-03 19:25:36 -0500 |
| commit | b0b42b16ff2b90f17bc1a4308366c9beba4b276e (patch) | |
| tree | d9025b46fe5360300ae4e430df3246c67602adad | |
| parent | 532d3b1f86f41834a25373e3ded981d68e4ce17f (diff) | |
dmaengine: protect 'id' from concurrent registrations
There is a possibility to have two devices registered with the same id.
Cc: <stable@kernel.org>
Acked-by: Maciej Sosnowski <maciej.sosnowski@intel.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
| -rw-r--r-- | drivers/dma/dmaengine.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/dma/dmaengine.c b/drivers/dma/dmaengine.c index 5317e08221ec..657996517374 100644 --- a/drivers/dma/dmaengine.c +++ b/drivers/dma/dmaengine.c | |||
| @@ -388,7 +388,10 @@ int dma_async_device_register(struct dma_device *device) | |||
| 388 | 388 | ||
| 389 | init_completion(&device->done); | 389 | init_completion(&device->done); |
| 390 | kref_init(&device->refcount); | 390 | kref_init(&device->refcount); |
| 391 | |||
| 392 | mutex_lock(&dma_list_mutex); | ||
| 391 | device->dev_id = id++; | 393 | device->dev_id = id++; |
| 394 | mutex_unlock(&dma_list_mutex); | ||
| 392 | 395 | ||
| 393 | /* represent channels in sysfs. Probably want devs too */ | 396 | /* represent channels in sysfs. Probably want devs too */ |
| 394 | list_for_each_entry(chan, &device->channels, device_node) { | 397 | list_for_each_entry(chan, &device->channels, device_node) { |
