* Re: Fw: zoran drivers: absense of locking?
2005-10-29 1:45 ` Fw: zoran drivers: absense of locking? Michael Krufky
@ 2005-10-29 4:02 ` Ronald S. Bultje
2005-10-30 10:03 ` Pauline Middelink
1 sibling, 0 replies; 3+ messages in thread
From: Ronald S. Bultje @ 2005-10-29 4:02 UTC (permalink / raw)
To: Michael Krufky
Cc: Johannes Stezenbach, Andrew Morton, Alexey Dobriyan,
Mauro Carvalho Chehab, Michael Krufky, Linux and Kernel Video,
linux-kernel
Hi guys,
I maintain the drivers. So, let's see:
On Fri, 2005-10-28 at 21:45 -0400, Michael Krufky wrote:
> >>From: Alexey Dobriyan <adobriyan@gmail.com>
> >>I've tried to read random part of a tree and now scratching my head
> >>with a question:
> >>
> >> what protects the number and a list of registered codecs in
> >> zoran drivers?
> >>
> >>Example: drivers/media/video/zr36050.c:
> >>
> >> /* amount of chips attached via this driver */
> >> static int zr36050_codecs = 0;
> >>
> >>Decremented in zr36050_unset().
> >>Checked for maximum value, used and incremented in zr36050_setup().
> >>
> >>[Assigment to 0 in zr36050_init_module is not needed. dprintk() in
> >>zr36050_cleanup_module() should be converted to BUG_ON, so I'll ignore
> >>them.]
> >>
> >> zr36050_codecs
> >> zr36050_unset() = struct videocodec::unset
> >> zr36050_setup() = struct videocodec::setup
> >>
> >>The only place where ->unset and ->setup methods are called is
> >>drivers/media/video/videocodec.c:
> >>
> >> zr36050_codecs
> >> zr36050_unset()
> >> videocodec_detach()
> >> zr36050_setup()
> >> videocodec_attach()
> >>
> >>Both videocodec functions are exported.
> >>
> >>No spinlocks or semaphores in sight.
> >>
> >>Does anybody know what protects the list of registered codecs in zoran
> >>drivers?
So, you're theoretically right, this is indeed a theory problem. Now, in
practice it isn't. Why? Because this isn't an actually exported
documented interface. This means two things: userspace cannot access it
(so no security issue), and other drivers than the ones I maintain won't
use it (so locking issues are limited to my own driver set).
It's an internal interface to the zoran driver (zoran_*.c / zr36067.ko).
The zoran driver uses publically exported interfaces (v4l/v4l2), and
those can be accessed from userspace. The zoran driver uses proper
locking everywhere and ensures that nothing goes wrong (or, well, so I
hope). So since the zoran driver is the only driver accessing zr36050.ko
(through videocodec.ko) and uses proper locking itself already, all's
fine in practice.
If other projects intend to use the videocodec interface, I'll consider
adding locking everywhere. But for now, it's fine, I guess.
Cheers,
Ronald
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Fw: zoran drivers: absense of locking?
2005-10-29 1:45 ` Fw: zoran drivers: absense of locking? Michael Krufky
2005-10-29 4:02 ` Ronald S. Bultje
@ 2005-10-30 10:03 ` Pauline Middelink
1 sibling, 0 replies; 3+ messages in thread
From: Pauline Middelink @ 2005-10-30 10:03 UTC (permalink / raw)
To: Michael Krufky
Cc: Johannes Stezenbach, Andrew Morton, Alexey Dobriyan,
Mauro Carvalho Chehab, Michael Krufky, Linux and Kernel Video,
linux-kernel, rbultje
[-- Attachment #1: Type: text/plain, Size: 881 bytes --]
On Fri, 28 Oct 2005 around 21:45:04 -0400, Michael Krufky wrote:
> ZR36067 VIDEO FOR LINUX DRIVER
> P: Ronald Bultje
> M: rbultje@ronald.bitfreak.net
> L: mjpeg-users@lists.sourceforge.net
> W: http://mjpeg.sourceforge.net/driver-zoran/
> S: Maintained
>
> ZR36120 VIDEO FOR LINUX DRIVER
> P: Pauline Middelink
> M: middelin@polyware.nl
> W: http://www.polyware.nl/~middelin/En/hobbies.html
> W: http://www.polyware.nl/~middelin/hobbies.html
> S: Maintained
> I can only guess that this falls under ZR36067 ?? (cc's added)
Totally different beasts... /me only does zr36120
[locking part snipped]
Met vriendelijke groet,
Pauline Middelink
--
GPG Key fingerprint = 2D5B 87A7 DDA6 0378 5DEA BD3B 9A50 B416 E2D0 C3C2
For more details look at my website http://www.polyware.nl/~middelink
[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 2276 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread