Re: [patch 1/2] x2apic: fix reserved APIC register accesses in print_local_APIC()

[Suresh Siddha <suresh.b.siddha@intel.com>]

On Tue, Sep 02, 2008 at 10:28:54PM -0700, Maciej W. Rozycki wrote:
> On Tue, 2 Sep 2008, Suresh Siddha wrote:
> 
> > APIC_RRR is a reserved register for integrated APIC's.
> 
>  NAK this change -- the Remote Read register is officially supported up to
> the Pentium integrated APIC and to the best of my knowledge it works with
> the P6 APIC as well.  Any APIC using the serial inter-APIC bus should
> support it.

from SDV Vol 3a Section 17.27.1

  The remote read delivery mode provided in the 82489DX and local APIC for
  Pentium processors is not supported in the local APIC in the Pentium 4, Intel
  Xeon, and P6 family processors.

It looks like it is not supported (officially atleast) even in APIC bus
architectures. Only the Pentium integrated APIC seems to be the exception.

I have to probably use maxlvt check, to selectively enable this debug output
for Pentium and !APIC_INTEGRATED check for 82489DX.

thanks,
suresh