Re: [PATCH PMQA] cpuidle_killer: allocate per cpu process array dynamically

2014-04-29 Thread Mohammad Merajul Islam Molla
Hello, I would like to share two observations - 1. Is it necessary to initialize nrcpus = 2 anymore? 2. Another problem may happen in the code below where waitpid is called - for (i = 0; i < nrcpus; i++) { int status; waitpid(pids[i], &status, 0);

[PATCH PMQA] cpuidle_killer: allocate per cpu process array dynamically

2014-04-29 Thread Sanjay Singh Rawat
currently percpu process array is set to 2, which results in segfault Signed-off-by: Sanjay Singh Rawat --- cpuidle/cpuidle_killer.c |7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/cpuidle/cpuidle_killer.c b/cpuidle/cpuidle_killer.c index 5e7320f..09009ef 100644 ---

[PATCH]: PowerTOP README file needs to be updated

2014-04-29 Thread Mohammad Merajul Islam Molla
Hello, It looks like README file in powertop branch is out of sync with how powertop should be actually built. It should be updated. diff --git a/README b/README index 2961eba..d83b315 100644 --- a/README +++ b/README @@ -2,9 +2,10 @@ -- To build and install PowerT