[RESEND][PATCH] linux-2.5.60_timer-tsc-cleanups_A1

From: john stultz (johnstul@us.ibm.com)
Date: Tue Feb 11 2003 - 19:19:33 EST


Linus, All

        This cleanup patch makes fast_gettimeoffset_quotient (a timer_tsc
specific variable) static, and replaces its usage with cpu_khz, making
it timer_opt independent.

Please apply.

thanks
-john

diff -Nru a/arch/i386/kernel/smpboot.c b/arch/i386/kernel/smpboot.c
--- a/arch/i386/kernel/smpboot.c Tue Feb 11 16:12:33 2003
+++ b/arch/i386/kernel/smpboot.c Tue Feb 11 16:12:33 2003
@@ -182,8 +182,6 @@
 
 #define NR_LOOPS 5
 
-extern unsigned long fast_gettimeoffset_quotient;
-
 /*
  * accurate 64-bit/32-bit division, expanded to 32-bit divisions and 64-bit
  * multiplication. Not terribly optimized but we need it at boot time only
@@ -223,7 +221,8 @@
 
         printk("checking TSC synchronization across %u CPUs: ", num_booting_cpus());
 
- one_usec = ((1<<30)/fast_gettimeoffset_quotient)*(1<<2);
+ /* convert from kcyc/sec to cyc/usec */
+ one_usec = cpu_khz / 1000;
 
         atomic_set(&tsc_start_flag, 1);
         wmb();
diff -Nru a/arch/i386/kernel/timers/timer_tsc.c b/arch/i386/kernel/timers/timer_tsc.c
--- a/arch/i386/kernel/timers/timer_tsc.c Tue Feb 11 16:12:33 2003
+++ b/arch/i386/kernel/timers/timer_tsc.c Tue Feb 11 16:12:33 2003
@@ -29,7 +29,7 @@
  * Equal to 2^32 * (1 / (clocks per usec) ).
  * Initialized in time_init.
  */
-unsigned long fast_gettimeoffset_quotient;
+static unsigned long fast_gettimeoffset_quotient;
 
 static unsigned long get_offset_tsc(void)
 {

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



This archive was generated by hypermail 2b29 : Sat Feb 15 2003 - 22:00:36 EST