Messages in this thread | | | Date | Sun, 30 Mar 2008 21:23:23 -0500 | From | Jack Steiner <> | Subject | Re: [PATCH 8/8] x86_64: Support for new UV apic |
| |
On Sun, Mar 30, 2008 at 07:13:42PM -0700, Yinghai Lu wrote: > On Sun, Mar 30, 2008 at 7:06 PM, Jack Steiner <steiner@sgi.com> wrote: > > > so this is "the new one of Friday"? > > > > Yes, and it has the same bug although it is located > > in a slightly different place. > > > > A few minutes ago, I posted a patch to delete the extra lines. > > > > > > > > > Did you test it on non UV_X2APIC box? > > > > The code is clearly wrong. I booted on an 8p AMD box and > > had no problems. Apparently the kernel (at least basic booting) is > > not too sensitive to incorrect apicids being returned. Most > > critical-to-boot code must use apicids from the ACPI tables. > > However, the bug does affect numa node assignment. And probably > > other places, too. > > please consider one global get_apic_id() and bad_apicid to replace > GET_APIC_ID and BAD_APICID at this point.
I think that makes sense.
The x2apic patch that should be posted in the near future also makes significant changes in this area. Once that patch is posted, I'll make the simplifications.
Ok???
--- jack
| |