diff options
author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2008-10-26 07:02:03 -0400 |
---|---|---|
committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2008-10-31 03:48:26 -0400 |
commit | 8449fc3ae58bf8ee5acbd2280754cde67b5db128 (patch) | |
tree | 01c632f8b1adf31f937a67f1e2d000053eea1f1a /drivers/ieee1394/raw1394.c | |
parent | 638570b54346f140bc09b986d93e76025d35180f (diff) |
ieee1394: dv1394: fix possible deadlock in multithreaded clients
Fix a possible though highly unlikely deadlock:
Thread A: Thread B:
- acquire mmap_sem - dv1394_ioctl/read/write()
- dv1394_mmap() - acquire video->mtx
- acquire video->mtx - copy_to/from_user(), possible page fault:
acquire mmap_sem
The simplest fix is to use mutex_trylock() instead of mutex_lock() in
dv1394_mmap(). This changes the behavior under contention in a way
which is visible to userspace clients. However, my guess is that no
clients exist which use mmap vs. ioctl/read/write on the dv1394
character device file interface in concurrent threads.
Reported-by: Johannes Weiner <hannes@saeurebad.de>
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/ieee1394/raw1394.c')
0 files changed, 0 insertions, 0 deletions