diff options
| author | Devin Heitmueller <dheitmueller@kernellabs.com> | 2009-04-28 14:35:27 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-06-16 19:14:13 -0300 |
| commit | 06f837cadbcdedb45f0702cb57c99c404ae921e6 (patch) | |
| tree | 2c6496c058143c8379190e0cebde8d8cf345b86d /drivers/media/video/cx88/cx88-video.c | |
| parent | b02064caebd9b1d73dd29ebb6e75f487c5f0dbc5 (diff) | |
| download | linux-06f837cadbcdedb45f0702cb57c99c404ae921e6.tar.gz linux-06f837cadbcdedb45f0702cb57c99c404ae921e6.tar.bz2 linux-06f837cadbcdedb45f0702cb57c99c404ae921e6.zip | |
V4L/DVB (11784): cx88: Fix race condition between cx8800 startup and hald
A power management fix to properly put the xc5000 into low power mode
revealed a race condition where hald could detect the creation of the device
file and connect to the device while the initial device configuration is
still in progress.
Lock the core structure so that video_release cannot be called and put the
tuner to sleep in the middle of the initial call to cx88_set_tvnorm() in
cx8800_initdev()
Thanks to Michael Krufky for discovering the issue and providing an
environment to test in.
Cc: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: Devin Heitmueller <dheitmueller@kernellabs.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/cx88/cx88-video.c')
| -rw-r--r-- | drivers/media/video/cx88/cx88-video.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/media/video/cx88/cx88-video.c b/drivers/media/video/cx88/cx88-video.c index b993d42fe73c..d6d6d13a6a6b 100644 --- a/drivers/media/video/cx88/cx88-video.c +++ b/drivers/media/video/cx88/cx88-video.c @@ -926,8 +926,10 @@ static int video_release(struct file *file) file->private_data = NULL; kfree(fh); + mutex_lock(&dev->core->lock); if(atomic_dec_and_test(&dev->core->users)) call_all(dev->core, tuner, s_standby); + mutex_unlock(&dev->core->lock); return 0; } |