[PATCH] nomadik: prevent sched_clock() wraparound
linus.ml.walleij at gmail.com
Tue Nov 16 17:15:02 EST 2010
2010/11/16 john stultz <johnstul at us.ibm.com>:
> On Tue, 2010-11-16 at 10:11 +0100, Linus Walleij wrote:
> The cycle value being passed as v is likely to be large, and the
> clocksource mult and shift are calculated to be as large as possible
> without overflowing when given only a few seconds worth of cycles.
> So its very likely that after a few seconds of running (or even less,
> with the 32_to_63 conversion), the cyc2ns function will overflow, as
> v*mult will be greater then 64bits.
Darn you're right of course.
I'll attempt to use
clocks_calc_mult_shift(&mult, &shift, rate, NSEC_PER_SEC, 3600*24*365);
For getting a somewhat more proper mult+shift for sched_clock().
>> + mod_timer(&cnt32_to_63_keepwarm_timer, round_jiffies(jiffies + data));
>> + (void) sched_clock();
> Also, just FYI: This can be problematic on PREEMPT_RT systems, where
> timers can be delayed by high priority processes hogging the cpu.
Hm is there a proper remedy for this? The watchdog in kernel/watchdog.c
creates a HRtimer based thread, SCHED_FIFO and prio MAX_RT_PRIO-1
is that what you should do for the ultimate sched_clock()
In that case this is ripe for breaking out into a separate helper
somewhere, like a callable
More information about the linux-arm-kernel