]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
bttv: Fix a deadlock in the bttv driver
authorArjan van de Ven <arjan@linux.intel.com>
Tue, 20 May 2008 16:53:52 +0000 (09:53 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Mon, 16 Jun 2008 20:20:06 +0000 (13:20 -0700)
commit69f24455cf25d776f8b19d06c7a43a8185fc633d
tree6bc8f3832dfe5274bbc1c7ff51aed893f119e255
parentc7559a1531958785bdc25363f8cde154011c0f9d
bttv: Fix a deadlock in the bttv driver

commit 81b2dbcad86732ffc02bad87aa25c4651199fc77 in mainline.

vidiocgmbuf() does this:
        mutex_lock(&fh->cap.vb_lock);
        retval = videobuf_mmap_setup(&fh->cap, gbuffers, gbufsize,
                                     V4L2_MEMORY_MMAP);

and videobuf_mmap_setup() then just does
        mutex_lock(&q->vb_lock);
        ret = __videobuf_mmap_setup(q, bcount, bsize, memory);
        mutex_unlock(&q->vb_lock);

which is an obvious double-take deadlock.

This patch fixes this by having vidiocgmbuf() just call the
__videobuf_mmap_setup function instead.

Acked-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Reported-by: Koos Vriezen <koos.vriezen@gmail.com>
Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/media/video/bt8xx/bttv-driver.c
drivers/media/video/videobuf-core.c
include/media/videobuf-core.h