From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linux-media@vger.kernel.org
Subject: Re: [GIT PATCHES FOR 2.6.37] Move V4L2 locking into the core framework
Date: Thu, 30 Sep 2010 08:55:21 -0300 [thread overview]
Message-ID: <4CA47AA9.8040403@redhat.com> (raw)
In-Reply-To: <201009261425.00146.hverkuil@xs4all.nl>
Em 26-09-2010 09:25, Hans Verkuil escreveu:
> Hi Mauro,
>
> These are the locking patches. It's based on my previous test tree, but with
> more testing with em28xx and radio-mr800 and some small tweaks relating to
> disconnect handling and video_is_registered().
>
> I also removed the unused get_unmapped_area file op and I am now blocking
> any further (unlocked_)ioctl calls after the device node is unregistered.
> The only things an application can do legally after a disconnect is unmap()
> and close().
>
> This patch series also contains a small em28xx fix that I found while testing
> the em28xx BKL removal patch.
>
> Regards,
>
> Hans
>
> The following changes since commit dace3857de7a16b83ae7d4e13c94de8e4b267d2a:
> Hans Verkuil (1):
> V4L/DVB: tvaudio: remove obsolete tda8425 initialization
>
> are available in the git repository at:
>
> ssh://linuxtv.org/git/hverkuil/v4l-dvb.git bkl
Applied, thanks.
> Hans Verkuil (10):
> v4l2-dev: after a disconnect any ioctl call will be blocked.
> v4l2-dev: remove get_unmapped_area
> v4l2: add core serialization lock.
> videobuf: prepare to make locking optional in videobuf
> videobuf: add ext_lock argument to the queue init functions
> videobuf: add queue argument to videobuf_waiton()
You forgot two to add the extra parameter on two drivers that uses vmalloc,
noticed when I tried to compile it for i386 arch. I've already added them.
Could you please double check if everything is compiling fine on the other archs?
Thanks,
Mauro
next prev parent reply other threads:[~2010-09-30 11:55 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-26 12:25 [GIT PATCHES FOR 2.6.37] Move V4L2 locking into the core framework Hans Verkuil
2010-09-30 11:55 ` Mauro Carvalho Chehab [this message]
2010-10-10 17:33 ` David Ellingsworth
2010-10-11 14:45 ` David Ellingsworth
2010-10-11 15:40 ` Hans Verkuil
2010-10-11 15:48 ` Mauro Carvalho Chehab
2010-10-11 15:54 ` Hans Verkuil
2010-10-11 16:23 ` Laurent Pinchart
2010-10-11 16:33 ` Mauro Carvalho Chehab
2010-10-11 18:05 ` David Ellingsworth
2010-10-11 19:10 ` Mauro Carvalho Chehab
2010-10-15 16:36 ` David Ellingsworth
2010-10-15 16:49 ` David Ellingsworth
2010-10-15 17:58 ` Hans Verkuil
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4CA47AA9.8040403@redhat.com \
--to=mchehab@redhat.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.