| Message ID | 20150411122056.GN12732@n2100.arm.linux.org.uk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Sat, 11 Apr 2015, Russell King - ARM Linux wrote: > We have recently had an example of someone wanting to use a 90kHz timer > for the software delay loop. > > udelay() needs to have at least microsecond resolution to allow drivers > access to a delay mechanism with a reasonable chance of delaying the > period they requested within at least a 50% marging of error, especially > for small delays. > > Discussion about the udelay() accuracy can be found at: > https://lkml.org/lkml/2011/1/9/37 > > Reject timers which are unable to supply this level of resolution. Is there a platform where that would be the only available timer? Well, the code would use the loop based delay in that case. Acked-by: Nicolas Pitre <nico@linaro.org> > Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk> > --- > arch/arm/lib/delay.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm/lib/delay.c b/arch/arm/lib/delay.c > index 312d43eb686a..8044591dca72 100644 > --- a/arch/arm/lib/delay.c > +++ b/arch/arm/lib/delay.c > @@ -83,6 +83,12 @@ void __init register_current_timer_delay(const struct delay_timer *timer) > NSEC_PER_SEC, 3600); > res = cyc_to_ns(1ULL, new_mult, new_shift); > > + if (res > 1000) { > + pr_err("Ignoring delay timer %ps, which has insufficient resolution of %lluns\n", > + timer, res); > + return; > + } > + > if (!delay_calibrated && (!delay_res || (res < delay_res))) { > pr_info("Switching to timer-based delay loop, resolution %lluns\n", res); > delay_timer = timer; > > -- > FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up > according to speedtest.net. > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > >
On Sat, Apr 11, 2015 at 01:42:35PM -0400, Nicolas Pitre wrote: > On Sat, 11 Apr 2015, Russell King - ARM Linux wrote: > > > We have recently had an example of someone wanting to use a 90kHz timer > > for the software delay loop. > > > > udelay() needs to have at least microsecond resolution to allow drivers > > access to a delay mechanism with a reasonable chance of delaying the > > period they requested within at least a 50% marging of error, especially > > for small delays. > > > > Discussion about the udelay() accuracy can be found at: > > https://lkml.org/lkml/2011/1/9/37 > > > > Reject timers which are unable to supply this level of resolution. > > Is there a platform where that would be the only available timer? > Well, the code would use the loop based delay in that case. > > Acked-by: Nicolas Pitre <nico@linaro.org> See the thread "Guarantee udelay(N) spins at least N microseconds" where we have someone trying to use a 90kHz counter for the delay loop, where there's complaints that delays are (a) not accurate and (b) can equate to no delay at all due to the low resolution of the counter, with all sorts of rediculous arguments put forward.
diff --git a/arch/arm/lib/delay.c b/arch/arm/lib/delay.c index 312d43eb686a..8044591dca72 100644 --- a/arch/arm/lib/delay.c +++ b/arch/arm/lib/delay.c @@ -83,6 +83,12 @@ void __init register_current_timer_delay(const struct delay_timer *timer) NSEC_PER_SEC, 3600); res = cyc_to_ns(1ULL, new_mult, new_shift); + if (res > 1000) { + pr_err("Ignoring delay timer %ps, which has insufficient resolution of %lluns\n", + timer, res); + return; + } + if (!delay_calibrated && (!delay_res || (res < delay_res))) { pr_info("Switching to timer-based delay loop, resolution %lluns\n", res); delay_timer = timer;
We have recently had an example of someone wanting to use a 90kHz timer for the software delay loop. udelay() needs to have at least microsecond resolution to allow drivers access to a delay mechanism with a reasonable chance of delaying the period they requested within at least a 50% marging of error, especially for small delays. Discussion about the udelay() accuracy can be found at: https://lkml.org/lkml/2011/1/9/37 Reject timers which are unable to supply this level of resolution. Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk> --- arch/arm/lib/delay.c | 6 ++++++ 1 file changed, 6 insertions(+)