RE: [PATCH] cpuidle: avoid singing capacitors

["Pallipadi, Venkatesh" <venkatesh.pallipadi@intel.com>]

 

>-----Original Message-----
>From: Pavel Machek [mailto:pavel@ucw.cz] 
>Sent: Monday, March 03, 2008 1:09 PM
>To: Pierre Ossman
>Cc: Pallipadi, Venkatesh; Adam Belay; 
>linux-pm@lists.linux-foundation.org; LKML; Andi Kleen; Lee Revell
>Subject: Re: [PATCH] cpuidle: avoid singing capacitors
>
>On Mon 2008-03-03 22:03:10, Pierre Ossman wrote:
>> On Mon, 3 Mar 2008 21:46:03 +0100
>> Pavel Machek <pavel@ucw.cz> wrote:
>> 
>> > >  
>> > > +static unsigned int min_deep_sleep = 2000;
>> > > +
>> > 
>> > Well, why not, but I believe we should default to old 
>behaviour... not
>> > all machines are cheaply-build.
>> 
>> One would hope. ;)
>> 
>> But the problem is that most people will not be able to find this
>> option (or even know such an option exists). I'd guess the 
>distros will
>> just end up having this on by default anyway. And since I could not
>> measure any extra power drain, I believe it's hard to 
>justify having it
>> off by default (more than by pure principle).
>
>So just leave it off by default, and let distros break their own
>kernels ;-).
>								

I prefer leaving it off my default and enabling it on faulty hardware by
some blacklist.

Thanks,
Venki