On 02/05/04 22:24 -0300, Gustavo Franco wrote:
> Hi,
>
> It's the last REDIFF, but i'm not replacing yield() with
> schedule_timeout() because there's a nicedelay() (maybe the
> usecs need to be tuned below) there:
>
>
> static inline void nicedelay(unsigned long usecs)
> {
> current->state = TASK_INTERRUPTIBLE;
> schedule_timeout(HZ);
> return;
> }
And usecs stands for??
Funny thing is that there are actualy uses of this function.
Also, please make -p1 applyable patches.
>
> Hope that helps,
> Gustavo Franco
>
> --- drivers/net/sb1000.c.orig 2004-04-28 20:50:17.000000000 -0300
> +++ drivers/net/sb1000.c 2004-04-28 20:49:10.000000000 -0300
> @@ -271,7 +271,7 @@
> timeout = jiffies + TimeOutJiffies;
> while (a & 0x80 || a & 0x40) {
> /* a little sleep */
> - yield();
> + nicedelay(10000);
>
>
> a = inb(ioaddr[0] + 7);
> if (time_after_eq(jiffies, timeout)) {
> @@ -295,7 +295,7 @@
> timeout = jiffies + TimeOutJiffies;
> while (a & 0x80 || !(a & 0x40)) {
> /* a little sleep */
> - yield();
> + nicedelay(10000);
>
>
> a = inb(ioaddr[1] + 6);
> if (time_after_eq(jiffies, timeout)) {
>
>
>
> _______________________________________________
> Kernel-janitors mailing list
> Kernel-janitors@xxxxxxxxxxxxxx
> http://lists.osdl.org/mailman/listinfo/kernel-janitors
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@xxxxxxxxxxxxxx
http://lists.osdl.org/mailman/listinfo/kernel-janitors
|
