lkml.org 
[lkml]   [2008]   [Oct]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRE: [PATCH] Typo in x86 apic.c with DIVISOR setup
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


\
 
 \ /
  Last update: 2008-10-10 18:33    [W:0.089 / U:0.416 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site