From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: linux-next: Tree for June 13: IO APIC breakage on HP nx6325 Date: Thu, 26 Jun 2008 10:37:19 +0200 Message-ID: <200806261037.20108.rjw@sisk.pl> References: <20080613232214.394fd6fd.sfr@canb.auug.org.au> <200806201422.12659.rjw@sisk.pl> <20080624091516.GA5642@ucw.cz> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20080624091516.GA5642@ucw.cz> Content-Disposition: inline Sender: linux-acpi-owner@vger.kernel.org To: Pavel Machek Cc: Matthew Garrett , "Maciej W. Rozycki" , Ingo Molnar , Stephen Rothwell , linux-next@vger.kernel.org, LKML , Thomas Gleixner , ACPI Devel Maling List , Len Brown List-Id: linux-next.vger.kernel.org On Tuesday, 24 of June 2008, Pavel Machek wrote: > Hi! > > > > What does ACPI claim the trip points are set to in this case? On the > > > 6125, if IRQ 2 is enabled in the APIC then the DSDT sets all the thermal > > > trip points to 16 degrees C. I suspect this means that enabling IRQ 2 is > > > the wrong thing to do on this chipset. > > > > Ah, indeed, thanks for the hint. This is the output of > > > > $ cat /proc/acpi/thermal_zone/TZ*/trip_points > > > > in the failing case: > > > > critical (S5): 105 C > > passive: 16 C: tc1=1 tc2=2 tsp=100 devices=C000 C001 > > active[0]: 16 C: devices=C34F > > active[1]: 16 C: devices=C350 > > active[2]: 16 C: devices=C351 > > active[3]: 16 C: devices=C352 > > critical (S5): 100 C > > passive: 16 C: tc1=1 tc2=2 tsp=300 devices=C000 C001 > > critical (S5): 100 C > > passive: 16 C: tc1=1 tc2=2 tsp=300 devices=C000 C001 > > Can we call the ACPI BIOS to be terminally broken at this point? It is broken, but the configuration worked before the patch. Consequently, the patch introduces a regression. Thanks, Rafael