Messages in this thread | | | From | Andi Kleen <> | Subject | Re: blink driver power saving | Date | Mon, 2 Jul 2007 01:59:50 +0200 |
| |
On Monday 02 July 2007 00:14, Linus Torvalds wrote: > On Sun, 1 Jul 2007, Andi Kleen wrote: > > What is so bad with it? Note it's a debugging facility and used > > for kcrash kernels where the video output doesn't work. But they > > normally only run a few minutes to dump the previous state to disk > > and then reboot. > > It has been a total disaster from beginning to end. > > It wastes power.
Again: It's not intended for normal kernels. It's a debugging feature. It's not intended for normal kernels. It's a debugging feature. It's not intended for normal kernels. It's a debugging feature.
Got it now? Power wasting or not just doesn't matter for it.
> It hangs machines when it tries to blink.
Yes, there seem to be more buggy keyboard controllers around than I anticipated. Very sad that IBM couldn't even get such a simple thing right.
Well only those that could be already hung from user space with setleds (that was also confirmed). Actually I thought they didn't hang completely, but just stopped reacting to the keyboard (which is actually pretty bad for every user to be able to trigger)
I guess the better way to handle those would be to find out the minimum frequency of blinking that is still ok and rate limit it to that in the keyboard driver.
Anyways, Stephen's patch just doesn't make sense: he clearly didn't understand the code at all. Before you apply it and cripple it better drop the driver completely.
-Andi - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |