> > Hi > > I have noted that "scaled_pentium_mhz" variable in i386/lapic.c is > defined and initialized, but never used anywhere (grep -R > scaled_pentium_mhz /usr/src say nothing [after patch applied]). > > Remove it ?
Good cache. Committed. Thanks. > Index: arch/i386/i386/lapic.c > =================================================================== > RCS file: /cvs/src/sys/arch/i386/i386/lapic.c,v > retrieving revision 1.32 > diff -u -p -r1.32 lapic.c > --- arch/i386/i386/lapic.c 2 Jun 2013 18:16:42 -0000 1.32 > +++ arch/i386/i386/lapic.c 26 Oct 2013 14:35:03 -0000 > @@ -234,7 +234,6 @@ u_int32_t lapic_per_second; > u_int32_t lapic_frac_usec_per_cycle; > u_int64_t lapic_frac_cycle_per_usec; > u_int32_t lapic_delaytab[26]; > -u_int64_t scaled_pentium_mhz; > > void > lapic_clockintr(void *arg) > @@ -368,8 +367,6 @@ lapic_calibrate_timer(struct cpu_info *c > tmp = (lapic_per_second * (u_int64_t)1 << 32) / 1000000; > > lapic_frac_cycle_per_usec = tmp; > - > - scaled_pentium_mhz = (1ULL << 32) / cpuspeed; > > /* > * Compute delay in cycles for likely short delays in usec. > >
