Re: [PATCH] Add option to generate shell script from Tunables

2011-08-23 Thread Jan Kaluža
On 08/14/2011 05:48 AM, Joshua Megerman wrote: I have read your response from 6/28/11 to this patch, and I am very dissapointed in the hypocritical nature of your decisions. While I understand your reasoning in wanting to have a split between Powertop and Powertune, the latter currently

Re: [PATCH] Add option to generate shell script from Tunables

2011-08-22 Thread Joshua Megerman
I have read your response from 6/28/11 to this patch, and I am very dissapointed in the hypocritical nature of your decisions. While I understand your reasoning in wanting to have a split between Powertop and Powertune, the latter currently doesn't exist, and as such we only have Powertop to

Re: [PATCH] Add option to generate shell script from Tunables

2011-07-19 Thread Valentijn Scholten
Arjan van de Ven arjan@... writes: Hi, I would like some more opinions on this from people on the list. So initially my plan was to, instead of powertop show the shell tunables, document these in a website/document instead. That way there could be a better explanation of gotcha's, kernel

Re: [PATCH] Add option to generate shell script from Tunables

2011-06-28 Thread Jan Kaluža
On 06/07/2011 03:48 PM, Arjan van de Ven wrote: On 6/7/2011 4:45 AM, Jan Kaluža wrote: Hi, attached patch adds new -s (--script) option to generate shell script with commands to turn every Bad item from Tunables tab into Good state. It adds new tunable::toggle_script() method which returns

Re: [PATCH] Add option to generate shell script from Tunables

2011-06-28 Thread Kok, Auke-jan H
On Tue, Jun 28, 2011 at 12:25 AM, Jan Kaluža jkal...@redhat.com wrote: On 06/07/2011 03:48 PM, Arjan van de Ven wrote: On 6/7/2011 4:45 AM, Jan Kaluža wrote: Hi, attached patch adds new -s (--script) option to generate shell script with commands to turn every Bad item from Tunables tab into

Re: [PATCH] Add option to generate shell script from Tunables

2011-06-09 Thread Jan Kaluža
On 06/07/2011 07:04 PM, Kok, Auke-jan H wrote: On Tue, Jun 7, 2011 at 8:45 AM, Peter Hüwepeterhu...@gmx.de wrote: Am Dienstag 07 Juni 2011, 15:48:11 schrieb Arjan van de Ven: I'm open to suggestions. Hi, I would like some more opinions on this from people on the list. So initially my plan

[PATCH] Add option to generate shell script from Tunables

2011-06-07 Thread Jan Kaluža
Hi, attached patch adds new -s (--script) option to generate shell script with commands to turn every Bad item from Tunables tab into Good state. It adds new tunable::toggle_script() method which returns command to switch state of particular tunable item and implements that method for all

Re: [PATCH] Add option to generate shell script from Tunables

2011-06-07 Thread Arjan van de Ven
On 6/7/2011 4:45 AM, Jan Kaluža wrote: Hi, attached patch adds new -s (--script) option to generate shell script with commands to turn every Bad item from Tunables tab into Good state. It adds new tunable::toggle_script() method which returns command to switch state of particular

Re: [PATCH] Add option to generate shell script from Tunables

2011-06-07 Thread Peter Hüwe
Am Dienstag 07 Juni 2011, 15:48:11 schrieb Arjan van de Ven: I'm open to suggestions. Hi, I would like some more opinions on this from people on the list. So initially my plan was to, instead of powertop show the shell tunables, document these in a website/document instead. That way

Re: [PATCH] Add option to generate shell script from Tunables

2011-06-07 Thread Jaroslav Skarvada
- Original Message - On 6/7/2011 4:45 AM, Jan Kaluža wrote: Hi, attached patch adds new -s (--script) option to generate shell script with commands to turn every Bad item from Tunables tab into Good state. It adds new tunable::toggle_script() method which returns command