Re: [PATCH] deinline sleep/delay functions

From: Arjan van de Ven
Date: Thu Jun 30 2005 - 06:25:14 EST



> An if(usec > 2000) { printk(..); dump_stack(); } will do.

that's runtime not compile time.
The old situation was a compile time check which is far more powerful.

>
> But do you really want to do this? There might be legitimate reasons
> to compute udelay's parameter with results which are sometimes large.

however that's not valid, because a really large udelay parameter will
cause the delay loop math to overflow. That was the main reason for the
"no more than 2ms or so" restriction.
>
> If you really want to, let's decide on this limit now,

the existing code already has a limit so why not just use that?


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/