http://bugzilla.kernel.org/show_bug.cgi?id=11000





------- Comment #11 from [EMAIL PROTECTED]  2008-07-01 23:34 -------
When I compiled and tested the 2.6.26-rc8 kernel from kernel.org, the
kacpi_notify spinning symptom went away, but it's because the thermal_zone THRM
device (/proc/acpi/thermal_zone/THRM) did not load (it did in earlier kernels).
 The problem is that my BIOS reports temperatures in degrees Celsius (i.e. 40
degrees Celsius would be '40'), but the ACPI spec clearly states that it should
report temperatures in kelvins times 10 (i.e. 300 kelvins would be '3000'). 
The ACPI driver doesn't know this, so it acts like the reported values are in
kelvins and does the conversion, resulting in temperature values like -269C and
-273C.  Because of this additional code:

/*
                 * Treat freezing temperatures as invalid as well; some
                 * BIOSes return really low values and cause reboots at
startup.
                 * Below zero (Celcius) values clearly aren't right for sure..
                 * ... so lets discard those as invalid.
                 */
                if (ACPI_FAILURE(status) ||
                                tz->trips.critical.temperature <= 2732) {
                        tz->trips.critical.flags.valid = 0;
                        ACPI_EXCEPTION((AE_INFO, status,
                                        "No or invalid critical threshold"));
                        return -ENODEV;

The THRM device fails to load because the critical temperature looks like it's
below freezing.

I added a simple sanity check to thermal.c for all the temperature reads I
could find.  It will work on correct BIOSes as well, provided that the CPU
really isn't -216.7 degrees Celsius or colder.  The patch is posted below.

When I made this change, THRM came back, and the kacpi_notify spinning problem
returned as before.  The requested information is posted below.  I triggered
the kacpi_notify spinning by suspending and resuming.

By the way, my computer is from around December 2001, if you're interested.

I'm finishing up the event queue flood protector.  It's sort of large because
it now has a procfs folder for configuration.  I'll post the patch for that
when I'm done and have tested it on my machine.


-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
acpi-bugzilla mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to