lkml.org 
[lkml]   [2006]   [Jun]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: clocksource
From
Date
On Wed, 2006-06-07 at 02:41 +0200, Roman Zippel wrote:
> On Tue, 6 Jun 2006, john stultz wrote:
> > > With large clock offsets the lookahead doesn't work correctly, basically
> > > because it's already to late and it can cause overadjustment. Because of
> > > this I do an extra lookahead in clocksource_bigadjust().
> >
> > Do you have a hard example for this with numbers? I don't mean to be a
> > pain, but I don't see this right off.
> >
> > With the current code in -mm I can run a test app that disables
> > interrupts for 2 seconds at a time over and over and I'm still keeping
> > synched w/ an NTP server within 30 microseconds.
>
> You need a clock source which doesn't generate it's own interrupts, so
> interrupts and clock updates can run asynchron. The key part above is
> "large clock offsets". In my test program disable the extra lookahead and
> run it with large offsets.

I'm not sure I'm following you here. Almost all clocksources on i386
(specifically, in the case above, I was using the apci_pm) don't
generate interrupts and run asynchronous from the timer interrupt
source.

I did re-review your documentation, and while it does go over the mult
adjustment code in nice understandable terms, the "why" of this
additional look-ahead isn't quite obvious.

> This code gets only limited testing in -mm, it needs to run for weeks
> or months, which I don't expect from the average -mm kernel. This makes
> userspace simulations so damn important and if you don't do this, you're
> playing a very risky game with a kernel which is supposed to be stable.

Agreed, simulation is nice. Thus, I've revived the old simulator which
builds using the existing code in -mm. Its a bit fast/dirty and isn't
exactly like your sim, but maybe you can take a look at it and send
patches to improve it?

You can find it at:
http://sr71.net/~jstultz/tod/simulator_C2.tar.bz2


I'm currently using it in testing my attempts to get your bigadjust code
working, so hopefully it will help there.

> > > For this I also I posted a userspace test program, so that I know how it
> > > behaves, do you have something similiar for yours?
> >
> > At your prodding awhile back I wrote a userspace simulator, but you
> > never commented on it.
>
> It was very hard to get running at all and in the meantime you had updated
> patches and the whole thing didn't work anymore. Sorry, that I didn't
> comment on it more.

Please let me know if you still have difficulty getting this new one
running.

thanks
-john

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

\
 
 \ /
  Last update: 2006-06-08 10:08    [W:0.211 / U:1.784 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site