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

From: Roman Zippel
Date: Mon Jul 16 2007 - 12:55:09 EST


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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/