Nishanth Aravamudan
2005-Jan-20 19:06 UTC
[Samba] [PATCH 8/40] fs/proc: replace schedule_timeout() with msleep()
Hi, Please consider applying. Description: Use msleep() instead of schedule_timeout() to guarantee the task delays as expected. TASK_INTERRUPTIBLE is used in the current code, but signals are not checked for, so I believe the change to msleep() is appropriate. Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com> --- 2.6.11-rc1-kj-v/fs/smbfs/proc.c 2005-01-15 16:55:41.000000000 -0800 +++ 2.6.11-rc1-kj/fs/smbfs/proc.c 2005-01-18 11:07:41.000000000 -0800 @@ -23,6 +23,7 @@ #include <linux/smb_fs.h> #include <linux/smbno.h> #include <linux/smb_mount.h> +#include <linux/delay.h> #include <net/sock.h> @@ -2397,8 +2398,7 @@ smb_proc_readdir_long(struct file *filp, if (req->rq_rcls == ERRSRV && req->rq_err == ERRerror) { /* a damn Win95 bug - sometimes it clags if you ask it too fast */ - current->state = TASK_INTERRUPTIBLE; - schedule_timeout(HZ/5); + msleep(200); continue; }