RE: [PATCH] Typo in x86 apic.c with DIVISOR setup

["Maciej W. Rozycki" <macro@linux-mips.org>]

On Fri, 10 Oct 2008, Pallipadi, Venkatesh wrote:

> I understand that using 1 instead of 16 makes APIC timer count 16 times faster.
> I did that change as I thought that was the intention of making APIC_DIVISOR as 1.
> Otherwise, there is no real impact of APIC_DIVISOR #define from being there. It
> Is just changing the value of calibration_result across 32 bit and 64 bit kernels.

 To use the value of 1 for the predivisor, special arrangements would have 
to be made for the discrete 82489DX APIC.  And we want to keep code 
variations as little as possible, especially when uncommon configurations 
are considered.  The value of 16 is supported universally with no special 
cases.

> For example, "host bus clock speed" message will print different values across
> 32 bit and 64 bit kernels on the same system, if we just make APIC_DIVISOR
> 16 for 32 bit and 1 for 64 bit kernel.

 All places have to agree on the value of the predivisor of course -- if 
you can see a problem somewhere, then please do send a patch.

  Maciej