Re: [PATCH 0/13] Time: Generic Timeofday Subsystem (v B10)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Andi Kleen <[email protected]> wrote:

> At least on x86-64 there is currently so much other timer related 
> development going on (per CPU TSC timers, no idle tick, 64bit HPET 
> etc.)  that I don't want any x86-64 bits of that merged for the next 
> time. The other stuff needs to settle first.
> 
> I haven't read the patchset in full detail, but from a quick look it's 
> also not obvious too me in which way it is easier and cleaner than the 
> old setup. While the old code was quirky in parts the new one seems to 
> fall more in the overmodularization/too many indirect callbacks trap.

there are 3 "generic" components needed right now to clean up all time 
related stuff: GTOD, ktimers and clockevents. [you know the first two, 
and clockevents is new code from Thomas Gleixner that generalizes timer 
interrupts and introduces one compact notion for 'clock chips'.]

what is the point? Ontop of these, a previously difficult feature, High 
Resolution Timers became _massively_ simpler. All of these patches exist 
together in the -rt tree, so it's not handwaving. The same will be the 
case for idle ticks / dynamic ticks [we started with HRT because it is 
so much harder than idle ticks]. So i do agree with you that GTOD needs 
more work, but it also makes time related features all that much easier.

right now it's GTOD that needs the most work before it can be merged 
upstream, so you picked the right one to criticise :-)

	Ingo
-
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/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux