https://bugzilla.novell.com/show_bug.cgi?id=350017
User trenn@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=350017#c11
Thomas Renninger changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |behlert@novell.com
Status|ASSIGNED |NEEDINFO
Info Provider| |Robert.Moore@intel.com
--- Comment #11 from Thomas Renninger 2008-01-14 11:13:51 MST ---
The problem is this:
+ External (\_PR_.CPU0)
+
Scope (\_PR)
{
- Processor (\_PR.CPU0, 0x00, 0x00000000, 0x00) {}
- Processor (\_PR.CPU1, 0x01, 0x00000000, 0x00) {}
+ Processor (\_PR.C000, 0x00, 0x00000000, 0x00) {}
+ Processor (\_PR.C001, 0x01, 0x00000000, 0x00) {}
+ Processor (\_PR.C002, 0x02, 0x00000000, 0x00) {}
+ Processor (\_PR.C003, 0x03, 0x00000000, 0x00) {}
}
But the passive trip point device is still:
Name (_PSL, Package (0x01)
{
\_PR.CPU0
})
So they changed the name of CPU{0,1} cpu devices to C00{0,1,2,3}
but forgot to change the reference to the passive trip point device.
Someone should tell this ASUS, but I don't have any contacts to them, Stefan
any idea whom we could contact there?
I also wonder why the call to _PSL succeeds when the thermal module is
initialized, tz->trips.passive.flags.valid should be 0?:
status = acpi_evaluate_reference(tz->device->handle, "_PSL", NULL,
&tz->trips.passive.devices);
if (ACPI_FAILURE(status))
tz->trips.passive.flags.valid = 0;
Robert/Len, could you help here, pls.
IMO this should get fixed in drivers/acpi/utils.c:acpi_evaluate_reference(..)
The package which includes an invalid reference should get checked there at the
end.
Then tz->trips.passive.flags.valid == 0
and the user gets the correct hint for the BIOS bug:
if (!tz->trips.passive.flags.valid)
printk(KERN_WARNING PREFIX "Invalid passive threshold\n");
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.