[Fuego] [PATCH 1/2] migratetest: parser.py: Correct getting value of, measurements

Tim.Bird at sony.com Tim.Bird at sony.com
Thu Feb 8 05:48:55 UTC 2018



> -----Original Message-----
> From: Hoang Van Tuyen
> 
> We want to get the whole number like 20499 but the issue is that
> it just got the first digit like 2. This commit fixs this problem.
> 
> Signed-off-by: Hoang Van Tuyen <tuyen.hoangvan at toshiba-tsdv.com>
> ---
>   engine/tests/Benchmark.migratetest/parser.py | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/engine/tests/Benchmark.migratetest/parser.py
> b/engine/tests/Benchmark.migratetest/parser.py
> index ae88f04..968163a 100755
> --- a/engine/tests/Benchmark.migratetest/parser.py
> +++ b/engine/tests/Benchmark.migratetest/parser.py
> @@ -16,11 +16,11 @@ if matches_max and matches_min and matches_avg:
>          avg_intervals = []
>          max_intervals = []
>          for thread_max in matches_max:
> -               max_intervals.append(float(thread_max[0]))
> +               max_intervals.append(float(thread_max))
>          for thread_min in matches_min:
> -               min_intervals.append(float(thread_min[0]))
> +               min_intervals.append(float(thread_min))
>          for thread_avg in matches_avg:
> -               avg_intervals.append(float(thread_avg[0]))
> +               avg_intervals.append(float(thread_avg))
>          measurements['default.intervals'] = [
>                  {"name": "max_interval", "measure" : max(max_intervals)},
>                  {"name": "min_interval", "measure" : min(min_intervals)},
> --
> 2.1.4
That appears to have fixed it.  Thanks.
Applied and pushed.

Any idea why the task 0 numbers are so bad?

I'm not running an RT kernel - is that the reason?

Task 0 (prio 2) (pid 26981):
   Max: 20036 us
   Min: 20013 us
   Tot: 200187 us
   Avg: 20018 us

 Task 1 (prio 3) (pid 26982):
   Max: 102 us
   Min: 14 us
   Tot: 473 us
   Avg: 47 us

 Task 2 (prio 4) (pid 26983):
   Max: 26 us
   Min: 9 us
   Tot: 135 us
   Avg: 13 us


 -- Tim


More information about the Fuego mailing list