[lkml]   [2011]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 6/6] clocksource: tile: convert to use clocksource_register_hz
On Wed, 2011-06-01 at 12:18 -0400, Chris Metcalf wrote:
> On 6/1/2011 3:32 AM, John Stultz wrote:
> > From: John Stultz <>
> >
> > Convert tile to use clocksource_register_hz.
> >
> > Untested. Help from maintainers would be appreciated.
> >
> > CC: Chris Metcalf <>
> This patch boots up fine on tile. It provides a shift count of 31, with a
> mult value in the billions. This does mean that clocksource_cyc2ns() will
> only be able to give correct timer conversions for up to about 9 seconds.
> Is this believed to be adequate for all the uses the cycle counter is put
> to? I can run our normal regression tests, but I'd like to get a
> high-level sanity check from the clocksource folks.

I believe 9 seconds should be plenty for timekeeping, so I suspect it
should be fine. Although I've not closely checked in this case if the
tile clocksource is used for other purposes, so do let me know if you
see any issues in your testing.

Additionally Thomas has a patch (which I thought already landed in
3.0-rc, but maybe not) which should pick a shift value that gets us
close to 600 second intervals (hardware allowing), allowing for much
longer NOHZ periods.

The important part is that we're centralizing this logic so it can be
properly adjusted without having to tune 30-some clocksources

Thanks for the testing and review!

 \ /
  Last update: 2011-06-01 19:57    [W:0.056 / U:1.448 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site