I'm planning to do a new release, and I'll check this this week. Hope to see a 
new release before the end of next week.

Auke


-----Original Message-----
From: [email protected] [mailto:[email protected]] On 
Behalf Of Arjan van de Ven
Sent: Friday, July 02, 2010 12:33 PM
To: Guy Lunardi
Cc: [email protected]
Subject: Re: [Discuss] Recent kernel configs with more than 5,000 lines

On 7/2/2010 12:31 PM, Guy Lunardi wrote:
> Hello,
>
> Ran into an issue tonight trying to run powertop. Discovered by the
> latest SUSE kernels run with configs in excess of  5,000 lines.
>
> Powertop crashes failing to load all the kernel configuration lines. I
> got 5,040 lines while config.c is hard-coded to only take in 5,000.
>
> Increasing the number of configlines takes care of the issue.
>
> --- config.c  2010-07-02 23:16:30.000000000 +0400
> +++ config.c-morelines        2010-07-02 23:17:48.000000000 +0400
> @@ -33,7 +33,7 @@
>   #include "powertop.h"
>
>   /* static arrays are not nice programming.. but they're easy */
> -static char configlines[5000][100];
> +static char configlines[5400][100];
>   static int configcount;
>
>
> Could you please add this to the 1.12 svn version?
>    

we already fixed this in the git tree for powertop.... quite some time ago

_______________________________________________
Discuss mailing list
[email protected]
http://lists.lesswatts.org/listinfo/discuss
_______________________________________________
Discuss mailing list
[email protected]
http://lists.lesswatts.org/listinfo/discuss

Reply via email to