diff options
author | Hans Verkuil <hverkuil@xs4all.nl> | 2010-10-11 11:36:37 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-10-20 23:18:15 -0400 |
commit | 9c84d89bf13b702ca0e0db5eb3880300c4bd58f4 (patch) | |
tree | c0b11f9fe5e386187881d59247a9e274046d7660 /Documentation/video4linux | |
parent | 1cd72781db93c00fbee0bf4aed9e310a1d589738 (diff) |
[media] radio-mr800: fix locking order
Don't hold the lock before unregistering the device, since when the
device is unregistered the datastruct containing the lock may be freed
(if the refcount went to 0).
Also fixed the framework documentation that erroneously suggested the
wrong locking order as well.
Reported-by: David Ellingsworth <david@identd.dyndns.org>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Acked-by: David Ellingsworth <david@identd.dyndns.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'Documentation/video4linux')
-rw-r--r-- | Documentation/video4linux/v4l2-framework.txt | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Documentation/video4linux/v4l2-framework.txt b/Documentation/video4linux/v4l2-framework.txt index a128e012a45c..f22f35c271f3 100644 --- a/Documentation/video4linux/v4l2-framework.txt +++ b/Documentation/video4linux/v4l2-framework.txt | |||
@@ -488,7 +488,7 @@ also waits in the code, then you should do the same to allow other processes | |||
488 | to access the device node while the first process is waiting for something. | 488 | to access the device node while the first process is waiting for something. |
489 | 489 | ||
490 | The implementation of a hotplug disconnect should also take the lock before | 490 | The implementation of a hotplug disconnect should also take the lock before |
491 | calling v4l2_device_disconnect and video_unregister_device. | 491 | calling v4l2_device_disconnect. |
492 | 492 | ||
493 | video_device registration | 493 | video_device registration |
494 | ------------------------- | 494 | ------------------------- |