2004-09-23 23:54:56

by maximilian attems

[permalink] [raw]
Subject: [patch 11/21] media/cx88-tvaudio: replace schedule_timeout() with msleep_interruptible()




Any comments would be appreciated.

Description: Use msleep_interruptible() instead of schedule_timeout() to
guarantee the task delays as expected.

Signed-off-by: Nishanth Aravamudan <[email protected]>

Signed-off-by: Maximilian Attems <[email protected]>
---

linux-2.6.9-rc2-bk7-max/drivers/media/video/cx88/cx88-tvaudio.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff -puN drivers/media/video/cx88/cx88-tvaudio.c~msleep_interruptible-drivers_media_video_cx88_cx88-tvaudio drivers/media/video/cx88/cx88-tvaudio.c
--- linux-2.6.9-rc2-bk7/drivers/media/video/cx88/cx88-tvaudio.c~msleep_interruptible-drivers_media_video_cx88_cx88-tvaudio 2004-09-21 21:16:57.000000000 +0200
+++ linux-2.6.9-rc2-bk7-max/drivers/media/video/cx88/cx88-tvaudio.c 2004-09-21 21:16:57.000000000 +0200
@@ -49,6 +49,7 @@
#include <linux/vmalloc.h>
#include <linux/init.h>
#include <linux/smp_lock.h>
+#include <linux/delay.h>

#include "cx88.h"

@@ -785,8 +786,7 @@ int cx88_audio_thread(void *data)
dprintk("cx88: tvaudio thread started\n");

for (;;) {
- set_current_state(TASK_INTERRUPTIBLE);
- schedule_timeout(HZ*3);
+ msleep_interruptible(3000);
if (signal_pending(current))
break;
if (dev->shutdown)
_