[Fuego] [PATCH 1/3] svsematest: Add a new test of the rt-tests

Tim.Bird at sony.com Tim.Bird at sony.com
Sat Jan 27 00:29:43 UTC 2018


> -----Original Message-----
> From: Hoang Van Tuyen 
> The svsematest starts two threads or fork two processes and
> measure the latency of SYSV semaphores.
> 
> Signed-off-by: Hoang Van Tuyen <tuyen.hoangvan at toshiba-tsdv.com>
> ---
>   .../tests/Benchmark.svsematest/chart_config.json   |  5 +++++
>   engine/tests/Benchmark.svsematest/criteria.json    | 26
> ++++++++++++++++++++++
>   engine/tests/Benchmark.svsematest/fuego_test.sh    | 25
> +++++++++++++++++++++
>   engine/tests/Benchmark.svsematest/parser.py        | 23
> +++++++++++++++++++
>   engine/tests/Benchmark.svsematest/reference.json   | 26
> ++++++++++++++++++++++
>   engine/tests/Benchmark.svsematest/spec.json        | 14 ++++++++++++
>   6 files changed, 119 insertions(+)
>   create mode 100644
> engine/tests/Benchmark.svsematest/chart_config.json
>   create mode 100644 engine/tests/Benchmark.svsematest/criteria.json
>   create mode 100755 engine/tests/Benchmark.svsematest/fuego_test.sh
>   create mode 100755 engine/tests/Benchmark.svsematest/parser.py
>   create mode 100644 engine/tests/Benchmark.svsematest/reference.json
>   create mode 100644 engine/tests/Benchmark.svsematest/spec.json
> 
> diff --git a/engine/tests/Benchmark.svsematest/chart_config.json
> b/engine/tests/Benchmark.svsematest/chart_config.json
> new file mode 100644
> index 0000000..cdaf6a2
> --- /dev/null
> +++ b/engine/tests/Benchmark.svsematest/chart_config.json
> @@ -0,0 +1,5 @@
> +{
> +    "chart_type": "measure_plot",
> +    "measures": ["default.latencies.max_latency",
> +        "default.latencies.avg_latency"]
> +}
> diff --git a/engine/tests/Benchmark.svsematest/criteria.json
> b/engine/tests/Benchmark.svsematest/criteria.json
> new file mode 100644
> index 0000000..a023558
> --- /dev/null
> +++ b/engine/tests/Benchmark.svsematest/criteria.json
> @@ -0,0 +1,26 @@
> +{
> +    "schema_version":"1.0",
> +    "criteria":[
> +        {
> +            "tguid":"default.latencies.max_latency",
> +            "reference":{
> +                "value":100,
> +                "operator":"le"
> +            }
> +        },
> +        {
> +            "tguid":"default.latencies.min_latency",
> +            "reference":{
> +                "value":100,
> +                "operator":"le"
> +            }
> +        },
> +        {
> +            "tguid":"default.latencies.avg_latency",
> +            "reference":{
> +                "value":100,
> +                "operator":"le"
> +            }
> +        }
> +    ]
> +}
> diff --git a/engine/tests/Benchmark.svsematest/fuego_test.sh
> b/engine/tests/Benchmark.svsematest/fuego_test.sh
> new file mode 100755
> index 0000000..9e90535
> --- /dev/null
> +++ b/engine/tests/Benchmark.svsematest/fuego_test.sh
> @@ -0,0 +1,25 @@
> +tarball=../rt-tests/rt-tests-v1.1.1.tar.gz
> +
> +NEED_ROOT=1
> +
> +function test_pre_check {
> +    assert_define BENCHMARK_SVSEMATEST_PARAMS
> +}
> +
> +function test_build {
> +    patch -p1 -N -s <
> $TEST_HOME/../rt-tests/0001-Add-scheduling-policies-for-old-kernels.patch
> +    make NUMA=0 svsematest
> +}
> +
> +function test_deploy {
> +    put svsematest  $BOARD_TESTDIR/fuego.$TESTDIR/
> +}
> +
> +function test_run {
> +    # svsematest does not support a option for printing a summary only
> on exit.
> +    # So, We get some lines at the end of the command's output.
> +    # The number for getting the lines depends on the cpu number of
> target machine.
> +    target_cpu_number=$(cmd "nproc")
> +    getting_line_number=$(( expr $target_cpu_number +
> $target_cpu_number ))

$(( )) is processed by bash to do arithmetic expansion.
There's no need to use 'expr' inside it.

I replaced with with:
getting_line_number=$(( $target_cpu_number +  $target_cpu_number ))

> +    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./svsematest
> $BENCHMARK_SVSEMATEST_PARAMS | tail -$getting_line_number"
> +}
> diff --git a/engine/tests/Benchmark.svsematest/parser.py
> b/engine/tests/Benchmark.svsematest/parser.py
> new file mode 100755
> index 0000000..edc77ff
> --- /dev/null
> +++ b/engine/tests/Benchmark.svsematest/parser.py
> @@ -0,0 +1,23 @@
> +#!/usr/bin/python
> +import os, re, sys
> +sys.path.insert(0, os.environ['FUEGO_CORE'] + '/engine/scripts/parser')
> +import common as plib
> +
> +regex_string = ".*, Min\s+(\d+).*, Avg\s+(\d+), Max\s+(\d+)"
> +measurements = {}
> +matches = plib.parse_log(regex_string)
> +
> +if matches:
> +    min_latencies = []
> +    avg_latencies = []
> +    max_latencies = []
> +    for thread in matches:
> +        min_latencies.append(float(thread[0]))
> +        avg_latencies.append(float(thread[1]))
> +        max_latencies.append(float(thread[2]))
> +    measurements['default.latencies'] = [
> +        {"name": "max_latency", "measure" : max(max_latencies)},
> +        {"name": "min_latency", "measure" : min(min_latencies)},
> +        {"name": "avg_latency", "measure" :
> sum(avg_latencies)/len(avg_latencies)}]
> +
> +sys.exit(plib.process(measurements))
> diff --git a/engine/tests/Benchmark.svsematest/reference.json
> b/engine/tests/Benchmark.svsematest/reference.json
> new file mode 100644
> index 0000000..415a8dd
> --- /dev/null
> +++ b/engine/tests/Benchmark.svsematest/reference.json
> @@ -0,0 +1,26 @@
> +{
> +    "test_sets":[
> +        {
> +            "name":"default",
> +            "test_cases":[
> +                {
> +                    "name":"latencies",
> +                    "measurements":[
> +                        {
> +                            "name":"max_latency",
> +                            "unit":"us"
> +                        },
> +                        {
> +                            "name":"min_latency",
> +                            "unit":"us"
> +                        },
> +                        {
> +                            "name":"avg_latency",
> +                            "unit":"us"
> +                        }
> +                    ]
> +                }
> +            ]
> +        }
> +    ]
> +}
> diff --git a/engine/tests/Benchmark.svsematest/spec.json
> b/engine/tests/Benchmark.svsematest/spec.json
> new file mode 100644
> index 0000000..1a9a767
> --- /dev/null
> +++ b/engine/tests/Benchmark.svsematest/spec.json
> @@ -0,0 +1,14 @@
> +{
> +    "testName": "Benchmark.svsematest",
> +    "specs": {
> +        "default": {
> +            "PARAMS": "-a -t -p99 -i100 -d25 -l10000"
> +        },
> +        "latest": {
> +            "PER_JOB_BUILD": "true",
> +            "gitrepo":
> "https://git.kernel.org/pub/scm/utils/rt-tests/rt-tests.git",
> +            "gitref": "unstable/devel/v1.1.1",
> +            "PARAMS": "-a -t -p99 -i100 -d25 -l10000"
> +        }
> +    }
> +}
> --
> 2.1.4
Looks good.  Applied.
 -- Tim


More information about the Fuego mailing list