lkml.org 
[lkml]   [2003]   [Jul]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: Fw: missing #if for 1000 HZ
Andrew Morton wrote:
> Guys?

If I understand what is being done, he is right. By this measure the
current error in the div (by shifting) is 2.4%.

I wonder how this plays with(line 639):

time_phase += time_adj;
if (time_phase <= -FINENSEC) {
long ltemp = -time_phase >> (SHIFT_SCALE - 10);
time_phase += ltemp << (SHIFT_SCALE - 10);
delta_nsec -= ltemp;
}

where, we are also using >>10 ~= /1000. I _think_ this ladder
approximation only delays part of the correction but does not loose
it, but the more eyes the better.

-g
>
> Begin forwarded message:
>
> Date: 22 Jul 2003 21:01:16 -0400
> From: Albert Cahalan <albert@users.sourceforge.net>
> To: linux-kernel@vger.kernel.org, Ulrich.Windl@rz.uni-regensburg.de
> Cc: Andrew Morton <akpm@digeo.com>, alan@redhat.com
> Subject: missing #if for 1000 HZ
>
>
> This should improve timekeeping a bit @ 1000 HZ.
>
>
> diff -Naurd old/kernel/timer.c new/kernel/timer.c
> --- old/kernel/timer.c 2003-07-18 17:27:01.000000000 -0400
> +++ new/kernel/timer.c 2003-07-18 17:32:19.000000000 -0400
> @@ -606,6 +606,15 @@
> else
> time_adj += (time_adj >> 2) + (time_adj >> 5);
> #endif
> +#if HZ == 1000
> + /* Compensate for (HZ==1000) != (1 << SHIFT_HZ).
> + * Add 1.5625% and 0.78125% to get 1023.4375; => only 0.05% error (p. 14)
> + */
> + if (time_adj < 0)
> + time_adj -= (-time_adj >> 6) + (-time_adj >> 7);
> + else
> + time_adj += (time_adj >> 6) + (time_adj >> 7);
> +#endif
> }
>
> /* in the NTP reference this is called "hardclock()" */
>
>
>
>

--
George Anzinger george@mvista.com
High-res-timers: http://sourceforge.net/projects/high-res-timers/
Preemption patch: http://www.kernel.org/pub/linux/kernel/people/rml

-
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: 2005-03-22 13:47    [W:0.021 / U:1.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site