Hi, On Fri, 15 Jul 2005, john stultz wrote:
> In my attempts to rework the timeofday subsystem, it was suggested I > try to avoid mixing cleanups with functional changes. In response to the > suggestion I've tried to break out the majority of the NTP cleanups I've > been working out of my larger patch and try to feed it in piece meal. > > The goal of this patch set is to isolate the in kernel NTP state machine > in the hope of simplifying the current timeofday code. I don't really like, where you taken it with ntp_advance(). With these patches you put half the ntp state machine in there and execute it at every single tick. >From the previous patches I can guess where you want to go with this, but I think it's the wrong direction. The code is currently as is for a reason, it's optimized for tick based system and I don't see a reason to change this for tick based system. If you want to change this for cycle based system, you have to give more control to the arch/timer source, which simply call a different set of functions and the ntp core system basically just acts as a library to the timer source. Tick based timer sources continue to update xtime and cycle based system will modify the cycle multiplier (e.g. what ppc64 does). Don't force everything to use the same set of functions, you'll make it only more complex. Larger ntp state updates don't have to be done more than once a second and leave the details of how the clock is updated to the clock source (just provide some library functions it can use). bye, Roman - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
