Re: [PATCH PM-QA v3] Refactor pass/fail/skip code in PM-QA

On 29 July 2014 19:36, Lisa Nguyen wrote: > Focus on returning the results of each test script rather > than the results of each subtest. This will help to keep the > number of PM-QA test results consistent across multiple boards > regardless of number of frequencies, cores, etc. > > Examples befo

[PATCH idlestat 05/12] Add incremental verbose option

This patch adds the '-v' option in the command line. The level of verbosity is choosen by the number of time the -v occurs. no -v option : verbosity 0 -v : verbosity 1 -vv : verbosity 2 -vvv : verbosity 3 etc ... Note this option is not yet used in the code. Signed-of

[PATCH idlestat 07/12] Split display function

Make some code separations to prepare the code for cleanup. Signed-off-by: Daniel Lezcano --- idlestat.c | 84 ++-- 1 file changed, 54 insertions(+), 30 deletions(-) diff --git a/idlestat.c b/idlestat.c index e4dbb89..b084b5d 100644 ---

[PATCH idlestat 10/12] Add option to choose the data to be displayed

Add the options to select c-states, p-states and wakeup output. --idle, -c : show c-states --frequency, -p : show p-states --wakeup, -w: show wakeup sources Signed-off-by: Daniel Lezcano --- idlestat.c | 33 ++--- idlestat.h |4 2 files changed, 3

[PATCH idlestat 09/12] Improve the output

The output is now in the form of: | P-state | min| max| avg| total | hits | | cpu0 |

[PATCH idlestat 01/12] Fix CPU affinity mask

From: Patrick Bellasi To properly identify the initial PState of each CPU, the idlestat_wake_all function is used to force an idle state exit on all CPUs by pinning idlestat on each CPUs. However, this routine forgets to recover the original mask at the end of the pinning process. This patch sol

[PATCH idlestat 12/12] Report even if the application returned an error code

In case the application is killed by the timeout and it handles the SIGTERM, it may return an error, thus idlestat won't display the result. Ignore the return code of the application. Signed-off-by: Daniel Lezcano --- idlestat.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

[PATCH idlestat 08/12] Don't double check with the estimate function

Signed-off-by: Daniel Lezcano --- idlestat.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/idlestat.c b/idlestat.c index b084b5d..e5dde8c 100644 --- a/idlestat.c +++ b/idlestat.c @@ -72,15 +72,7 @@ static int display_cstates(struct cpuidle_cstates *cstates, char

[PATCH idlestat 02/12] Fix some lines to comply with 80 chars length

Signed-off-by: Daniel Lezcano --- idlestat.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/idlestat.c b/idlestat.c index 2a0c549..30860db 100644 --- a/idlestat.c +++ b/idlestat.c @@ -1162,7 +1162,8 @@ static int idlestat_file_for_each_line(const char *path, v

[PATCH idlestat 11/12] Split the display to show the wakeup separetely

Signed-off-by: Daniel Lezcano --- idlestat.c | 102 ++-- topology.c | 10 +++--- 2 files changed, 70 insertions(+), 42 deletions(-) diff --git a/idlestat.c b/idlestat.c index da1e14b..24248b2 100644 --- a/idlestat.c +++ b/idlestat.c @@ -

[PATCH idlestat 06/12] Remove dump and iterations options

Signed-off-by: Daniel Lezcano --- idlestat.c | 72 ++-- idlestat.h |2 -- topology.c | 12 +- topology.h |5 ++--- 4 files changed, 19 insertions(+), 72 deletions(-) diff --git a/idlestat.c b/idlestat.c index 8138e35..e4

[PATCH idlestat 04/12] Add 'patches' directory to the ignored list of files in .gitignore

Signed-off-by: Daniel Lezcano --- .gitignore |1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 968abde..4ee98b8 100644 --- a/.gitignore +++ b/.gitignore @@ -30,6 +30,7 @@ *.orig *~ \#*# +patches # # Top-level files -- 1.7.9.5 __

[PATCH idlestat 03/12] Remove the debug and iteration options

This function are pointless, remove them. Signed-off-by: Daniel Lezcano --- idlestat.c | 16 ++-- idlestat.h |1 - 2 files changed, 2 insertions(+), 15 deletions(-) diff --git a/idlestat.c b/idlestat.c index 30860db..abbb787 100644 --- a/idlestat.c +++ b/idlestat.c @@ -1042,11