2004-09-23 23:53:56

by maximilian attems

[permalink] [raw]
Subject: [patch 09/21] media/c-qcam: 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/c-qcam.c | 6 ++----
1 files changed, 2 insertions(+), 4 deletions(-)

diff -puN drivers/media/video/c-qcam.c~msleep_interruptible-drivers_media_video_c-qcam drivers/media/video/c-qcam.c
--- linux-2.6.9-rc2-bk7/drivers/media/video/c-qcam.c~msleep_interruptible-drivers_media_video_c-qcam 2004-09-21 21:16:54.000000000 +0200
+++ linux-2.6.9-rc2-bk7-max/drivers/media/video/c-qcam.c 2004-09-21 21:16:54.000000000 +0200
@@ -103,8 +103,7 @@ static unsigned int qcam_await_ready1(st
{
if (qcam_ready1(qcam) == value)
return 0;
- current->state=TASK_INTERRUPTIBLE;
- schedule_timeout(HZ/10);
+ msleep_interruptible(100);
}

/* Probably somebody pulled the plug out. Not much we can do. */
@@ -129,8 +128,7 @@ static unsigned int qcam_await_ready2(st
{
if (qcam_ready2(qcam) == value)
return 0;
- current->state=TASK_INTERRUPTIBLE;
- schedule_timeout(HZ/10);
+ msleep_interruptible(100);
}

/* Probably somebody pulled the plug out. Not much we can do. */
_