On Thu, May 15, 2008 at 10:15:39AM +0800, Zhang Rui wrote: > On Thu, 2008-05-15 at 02:58 +0100, Matthew Garrett wrote: > > I'll modify my patch so it uses the devices in a _TZD package if they're > > present. > > > Hmm, ACPI thermal zone doesn't know how to throttle the devices in a > _TZD package, so I don't think we can use the _TZD devices. If they have a _TZD, it's presumably because they need cooling. If the kernel isn't going to be able to do that, what happens if the userspace daemon crashes or fails to start for some reason? The alternative is to leave it as it is currently in my patch. If the userspace daemon starts, it disables the ACPI control and will handle it itself. If it doesn't start, the kernel does the only thing it can - reduce the speed of the CPU. If there are no plans to add hooks for the drivers to be throttled by the kernel, then I'd argue that this is the right thing to do. -- Matthew Garrett | mjg59@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
