Messages in this thread | | | Date | Tue, 27 May 2008 15:26:27 -0400 | From | "Devin Heitmueller" <> | Subject | Re: [PATCH] video4linux: Push down the BKL |
| |
Hello Mauro,
On Tue, May 27, 2008 at 2:59 PM, Mauro Carvalho Chehab <mchehab@infradead.org> wrote: > For example, em28xx has already a lock at the operations that change values at > "dev" struct, including open() method. However, since the lock is not called at > get operations, it needs to be fixed. I would also change it from mutex to a > read/write semaphore, since two (or more) get operations can safely happen in > parallel.
Please bear in mind that we have not worked out the locking semantics for hybrid tuner devices, and it's entirely possible that the get() routines will need to switch the tuner mode, which would eliminate any benefits of converting to a read/write semaphore.
I'm not sure yet exactly how that's going to work, but it's something that might prompt you to defer converting it from a mutex until we have that worked out.
-- Devin J. Heitmueller http://www.devinheitmueller.com AIM: devinheitmueller
| |