From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp-vbr5.xs4all.nl ([194.109.24.25]:1298 "EHLO smtp-vbr5.xs4all.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751051AbaAQKEU (ORCPT ); Fri, 17 Jan 2014 05:04:20 -0500 Message-ID: <52D90012.6080608@xs4all.nl> Date: Fri, 17 Jan 2014 11:04:02 +0100 From: Hans Verkuil MIME-Version: 1.0 To: Antti Palosaari CC: linux-media@vger.kernel.org Subject: Re: [PATCH 6/6] v4l: disable lockdep on vb2_fop_mmap() References: <1388292700-18369-1-git-send-email-crope@iki.fi> <1388292700-18369-7-git-send-email-crope@iki.fi> In-Reply-To: <1388292700-18369-7-git-send-email-crope@iki.fi> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Hi Antti, Is this still needed after this commit was merged? http://git.linuxtv.org/media_tree.git/commit/b18a8ff29d80b132018d33479e86ab8ecaee6b46 Regards, Hans On 12/29/2013 05:51 AM, Antti Palosaari wrote: > Avoid that lockdep warning: > > [ INFO: possible circular locking dependency detected ] > 3.13.0-rc1+ #77 Tainted: G C O > ------------------------------------------------------- > video_source:sr/32072 is trying to acquire lock: > (&dev->mutex#2){+.+.+.}, at: [] vb2_fop_mmap+0x33/0x90 [videobuf2_core] > > but task is already holding lock: > (&mm->mmap_sem){++++++}, at: [] vm_mmap_pgoff+0x6f/0xc0 > > Possible unsafe locking scenario: > CPU0 CPU1 > ---- ---- > lock(&mm->mmap_sem); > lock(&dev->mutex#2); > lock(&mm->mmap_sem); > lock(&dev->mutex#2); > *** DEADLOCK *** > > Signed-off-by: Antti Palosaari > --- > drivers/media/v4l2-core/videobuf2-core.c | 14 +++++++++++++- > 1 file changed, 13 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c > index 12df9fd..2a74295 100644 > --- a/drivers/media/v4l2-core/videobuf2-core.c > +++ b/drivers/media/v4l2-core/videobuf2-core.c > @@ -2641,12 +2641,24 @@ int vb2_fop_mmap(struct file *file, struct vm_area_struct *vma) > struct video_device *vdev = video_devdata(file); > struct mutex *lock = vdev->queue->lock ? vdev->queue->lock : vdev->lock; > int err; > + /* > + * FIXME: Ugly hack. Disable possible lockdep as it detects possible > + * deadlock. "INFO: possible circular locking dependency detected" > + */ > + lockdep_off(); > > - if (lock && mutex_lock_interruptible(lock)) > + if (lock && mutex_lock_interruptible(lock)) { > + lockdep_on(); > return -ERESTARTSYS; > + } > + > err = vb2_mmap(vdev->queue, vma); > + > if (lock) > mutex_unlock(lock); > + > + lockdep_on(); > + > return err; > } > EXPORT_SYMBOL_GPL(vb2_fop_mmap); >