[ATALK] aarp: replace schedule_timeout() with msleep()

From: Nishanth Aravamudan <nacc@us.ibm.com>

Use msleep() instead of schedule_timeout() to guarantee the task
delays as expected. The current code is not wrong, but it does not account for
early return due to signals, so I think msleep() should be appropriate.

Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com>
Signed-off-by: Domen Puncer <domen@coderock.org>
Signed-off-by: David S. Miller <davem@davemloft.net>

authored by Nishanth Aravamudan and committed by David S. Miller 285b3afe 7abaa27c

+3 -4
+3 -4
net/appletalk/aarp.c
··· 35 35 #include <net/datalink.h> 36 36 #include <net/psnap.h> 37 37 #include <linux/atalk.h> 38 + #include <linux/delay.h> 38 39 #include <linux/init.h> 39 40 #include <linux/proc_fs.h> 40 41 #include <linux/seq_file.h> ··· 463 462 aarp_send_probe(atif->dev, &atif->address); 464 463 465 464 /* Defer 1/10th */ 466 - current->state = TASK_INTERRUPTIBLE; 467 - schedule_timeout(HZ / 10); 465 + msleep(100); 468 466 469 467 if (atif->status & ATIF_PROBE_FAIL) 470 468 break; ··· 510 510 aarp_send_probe(atif->dev, sa); 511 511 512 512 /* Defer 1/10th */ 513 - current->state = TASK_INTERRUPTIBLE; 514 513 write_unlock_bh(&aarp_lock); 515 - schedule_timeout(HZ / 10); 514 + msleep(100); 516 515 write_lock_bh(&aarp_lock); 517 516 518 517 if (entry->status & ATIF_PROBE_FAIL)