block/pf: replace pf_sleep() with
Subsystems (2)
PATCH 3/8 block/pf: replace pf_sleep() with
Any comments would be appreciated. This is a re-push of a patch I submitted 20 July which hasn't been merged as of 2.6.9-rc1-mm5/2.6.9-rc2.
Description: msleep_interruptible() is used instead of pf_sleep() to guarantee the task delays as expected. The defintion of pf_sleep() is also removed.
Signed-off-by Nishanth Aravamudan <[email protected]>
drivers/block/paride/pf.c +2 -8
@@ -526,12 +526,6 @@ static void pf_eject(struct pf_unit *pf)#define PF_RESET_TMO 30/* in tenths of a second */ -static void pf_sleep (int cs) -{ - current->state =TASK_INTERRUPTIBLE ; -schedule_timeout (cs); -} -/* the ATAPI standard actually specifies the contents of all 7 registers after a reset, but the specification is ambiguous concerning the last two bytes, and different drives interpret the standard differently.
@@ -546,11 +540,11 @@ static int pf_reset(struct pf_unit *pf)write_reg (pf,6 ,0xa0 +0x10 *pf->drive);write_reg (pf,7 ,8 ); -pf_sleep (20 *HZ /1000 ); +msleep_interruptible (20 ); k =0 ;while ((k++ <PF_RESET_TMO ) && (status_reg (pf) &STAT_BUSY )) -pf_sleep (HZ /10 ); +msleep_interruptible (100 ); flg =1 ;for (i =0 ; i <5 ; i++)
_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors