Re: [PATCH/RFC] msleep() with hrtimers

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

 



Hi,

On Mon, 16 Jul 2007, Ingo Molnar wrote:

> I explained it numerous times (remember the 'timeout' vs. 'timer event' 
> discussion?) that i consider timer granularity important to scalability. 
> Basically, in every case where we know with great certainty that a 
> time-out will _not_ occur (where the time-out is in essence just an 
> exception handling mechanism), using struct timer_list is the best 
> solution.

Whether the timer expires or not is in many cases completely irrelevant.
You need special cases, where the timer wheel behaviour becomes an issue 
and whether hrtimer would behave any better in such situations is 
questionable.
Again, for the average user such details are pretty much irrelevant.

> what i consider harmful on the other hand are all the HZ assumptions 
> embedded into various pieces of code. The most harmful ones are design 
> details that depend on HZ and kernel-internal API details that depends 
> on HZ. Yes, NTP was such an example, and it was hard to fix, and you 
> didnt help much with that.

Stop spreading lies! :-(
One only has to look at the history of kernel/time/ntp.c
John's rather simple "HZ free ntp" patch wouldn't have been that simple 
without all the cleanup patches before that done by me, which were 
precisely intended to make this possible.

> (perhaps that is one source of this 
> increasingly testy exchange ;-)

No, it's your prejudice against me based on wrong facts.
Get your facts straight and stop being an ass.

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/

[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