Messages in this thread | | | Subject | Re: IO-APIC on nforce2 [PATCH] + [PATCH] for nmi_debug=1 + [PATCH] for idle=C1halt, 2.6.5 | From | Craig Bradney <> | Date | Thu, 22 Apr 2004 22:50:33 +0200 |
| |
On Thu, 2004-04-22 at 17:03, Len Brown wrote: > On Thu, 2004-04-22 at 04:45, Craig Bradney wrote: > > > > Yes, you need to enable ACPI and IOAPIC. The goal of this patch > > > is to address the XT-PIC timer issue in IOAPIC mode. > > > > > > Here's the latest (vs 2.6.5). > > > > > > Do we need any other patch? eg the idlec1halt patch? My Athlon still has > > 2.6.3 on it. > > If you needed idlec1halt before, you still need it. > This patch just addresses the XT-PIC timer issue. > > > > > > I've got 1 Abit, 2 Asus, and 1 Shuttle DMI entry. Let me know if the > > > product names (1st line of dmidecode entry) are correct, > > > these are not from DMI, but are supposed to be human-readable titles. > > > > + { ignore_timer_override, "Asus A7N8X v2", { > > > + MATCH(DMI_BOARD_VENDOR, "ASUSTeK Computer INC."), > > > + MATCH(DMI_BOARD_NAME, "A7N8X2.0"), > > > + MATCH(DMI_BIOS_VERSION, "ASUS A7N8X2.0 Deluxe ACPI BIOS Rev 1007"), > > > + MATCH(DMI_BIOS_DATE, "10/06/2003") }}, > > > > my dmidecode output also shows (in the first BIOS information section): > > Vendor: Phoenix Technologies, LTD > > although the Manufacturer is ASUSTek Computer INC. form the Base Board > > and System sections. > > Right, DMI has separate sections for System, Board, BIOS, and we're > using two pieces from the BOARD and two pieces from the BIOS sections. > > > Not really sure about the code. If it matches on all of above then it > > might not work. Ill try a new kernel later today and see the result. > > The workaround is triggered only if all the MATCH()'s above match. > If it doesn't trigger, then either I munged it on copy out of dmidecode > or you've got a different BIOS and we need a new dmidecode... > > > > I'm interested only in the latest BIOS -- if it is still broken. > > > The assumption is that if a fixed BIOS is available, the users > > > should upgrade. > > > > > > > Yes, I just checked yesterday and there was nothing new.
[Have sent this email with attachments directly to Len, attachments are just /proc/interrupts and dmegs output. If someone is interested, please ask for them]
Hi Len
Please find attached /proc/interrupts and dmesg from 3 boots, 2 with new kernel.
263 : gentoo-dev-sources-r1 2.6.3 kernel with Ross Dickson's idleC1halt and IOAPIC patches.
265: gentoo-dev-sources-r1 2.6.5 kernel with Ross Dickson's idleC1halt for 2.6.5 kernel only. Note in 265pi (/proc/interrupts): 0: 54821 XT-PIC timer
265-lb: gentoo-dev-sources-r1 2.6.5 kernel with Ross Dickson's idleC1halt for 2.6.5 kernel and your patch for the interrupt. Note in 265pi-lb (/proc/interrupts): 0: 51144 IO-APIC-edge timer
so.. looks good here. :) I was surprised to see this effect with no boot kernel option though. Having read the code I see you set the value to 1 and therefore on. Seems fine to me.
regards Craig [unhandled content-type:application/pgp-signature] | |