All of lore.kernel.org
 help / color / mirror / Atom feed
* [Fuego] [PATCH] tests: Use get_program_path in test_run function
@ 2021-10-28 12:58 venkata.pyla
  2021-10-28 13:00 ` Venkata.Pyla
  2021-10-29  8:34 ` [Fuego] [PATCH v2] " venkata.pyla
  0 siblings, 2 replies; 4+ messages in thread
From: venkata.pyla @ 2021-10-28 12:58 UTC (permalink / raw)
  To: tim.bird; +Cc: dinesh.kumar, fuego

From: venkata pyla <venkata.pyla@toshiba-tsip.com>

In order to use the test programs installed in the board's
default PATH use get_program_path function, which will
find the program in the PATH location or in the
board test directory.

Signed-off-by: venkata pyla <venkata.pyla@toshiba-tsip.com>
---
 tests/Benchmark.migratetest/fuego_test.sh | 3 ++-
 tests/Benchmark.pmqtest/fuego_test.sh     | 3 ++-
 tests/Benchmark.ptsematest/fuego_test.sh  | 3 ++-
 tests/Benchmark.signaltest/fuego_test.sh  | 3 ++-
 tests/Benchmark.sigwaittest/fuego_test.sh | 3 ++-
 tests/Benchmark.svsematest/fuego_test.sh  | 3 ++-
 tests/Functional.pi_tests/fuego_test.sh   | 3 ++-
 7 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/tests/Benchmark.migratetest/fuego_test.sh b/tests/Benchmark.migratetest/fuego_test.sh
index eeaa4f6..af5d8a5 100755
--- a/tests/Benchmark.migratetest/fuego_test.sh
+++ b/tests/Benchmark.migratetest/fuego_test.sh
@@ -23,5 +23,6 @@ function test_deploy {
 }
 
 function test_run {
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./${TEST_COMMAND} $BENCHMARK_MIGRATETEST_PARAMS"
+    get_program_path rt-migrate-test
+    report "$PROGRAM_RT_MIGRATE_TEST $BENCHMARK_MIGRATETEST_PARAMS"
 }
diff --git a/tests/Benchmark.pmqtest/fuego_test.sh b/tests/Benchmark.pmqtest/fuego_test.sh
index 6733e78..72a8fce 100755
--- a/tests/Benchmark.pmqtest/fuego_test.sh
+++ b/tests/Benchmark.pmqtest/fuego_test.sh
@@ -27,5 +27,6 @@ function test_run {
     # The number for getting the lines depends on the cpu number of target machine.
     target_cpu_number=$(cmd "nproc")
     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./pmqtest $BENCHMARK_PMQTEST_PARAMS | tail -$getting_line_number"
+    get_program_path pmqtest
+    report "$PROGRAM_PMQTEST $BENCHMARK_PMQTEST_PARAMS | tail -$getting_line_number"
 }
diff --git a/tests/Benchmark.ptsematest/fuego_test.sh b/tests/Benchmark.ptsematest/fuego_test.sh
index a626d44..8ddb53c 100755
--- a/tests/Benchmark.ptsematest/fuego_test.sh
+++ b/tests/Benchmark.ptsematest/fuego_test.sh
@@ -27,5 +27,6 @@ function test_run {
     # The number for getting the lines depends on the cpu number of target machine.
     target_cpu_number=$(cmd "cat /proc/cpuinfo | grep processor | wc -l")
     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./ptsematest $BENCHMARK_PTSEMATEST_PARAMS | tail -$getting_line_number"
+    get_program_path ptsematest
+    report "$PROGRAM_PTSEMATEST $BENCHMARK_PTSEMATEST_PARAMS | tail -$getting_line_number"
 }
diff --git a/tests/Benchmark.signaltest/fuego_test.sh b/tests/Benchmark.signaltest/fuego_test.sh
index 54e08b1..b70f99d 100755
--- a/tests/Benchmark.signaltest/fuego_test.sh
+++ b/tests/Benchmark.signaltest/fuego_test.sh
@@ -22,5 +22,6 @@ function test_deploy {
 }
 
 function test_run {
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./signaltest $BENCHMARK_SIGNALTEST_PARAMS"
+    get_program_path signaltest
+    report "$PROGRAM_SIGNALTEST $BENCHMARK_SIGNALTEST_PARAMS"
 }
diff --git a/tests/Benchmark.sigwaittest/fuego_test.sh b/tests/Benchmark.sigwaittest/fuego_test.sh
index e419f17..57b118a 100755
--- a/tests/Benchmark.sigwaittest/fuego_test.sh
+++ b/tests/Benchmark.sigwaittest/fuego_test.sh
@@ -27,5 +27,6 @@ function test_run {
     # The number for getting the lines depends on the cpu number of target machine.
     target_cpu_number=$(cmd "nproc")
     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./sigwaittest $BENCHMARK_SIGWAITTEST_PARAMS | tail -$getting_line_number"
+    get_program_path sigwaittest
+    report "$PROGRAM_SIGWAITTEST $BENCHMARK_SIGWAITTEST_PARAMS | tail -$getting_line_number"
 }
diff --git a/tests/Benchmark.svsematest/fuego_test.sh b/tests/Benchmark.svsematest/fuego_test.sh
index 2f6e914..89d8335 100755
--- a/tests/Benchmark.svsematest/fuego_test.sh
+++ b/tests/Benchmark.svsematest/fuego_test.sh
@@ -27,5 +27,6 @@ function test_run {
     # The number for getting the lines depends on the cpu number of target machine.
     target_cpu_number=$(cmd "cat /proc/cpuinfo | grep processor | wc -l")
     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./svsematest $BENCHMARK_SVSEMATEST_PARAMS | tail -$getting_line_number"
+    get_program_path svsematest
+    report "$PROGRAM_SVSEMATEST $BENCHMARK_SVSEMATEST_PARAMS | tail -$getting_line_number"
 }
diff --git a/tests/Functional.pi_tests/fuego_test.sh b/tests/Functional.pi_tests/fuego_test.sh
index bf94a63..e0783ea 100755
--- a/tests/Functional.pi_tests/fuego_test.sh
+++ b/tests/Functional.pi_tests/fuego_test.sh
@@ -22,7 +22,8 @@ function test_deploy {
 }
 
 function test_run {
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./pi_stress $FUNCTIONAL_PI_TESTS_PARAMS"
+    get_program_path pi_stress
+    report "$PROGRAM_PI_STRESS $FUNCTIONAL_PI_TESTS_PARAMS"
 }
 
 function test_processing {
-- 
2.20.1



^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [Fuego] [PATCH] tests: Use get_program_path in test_run function
  2021-10-28 12:58 [Fuego] [PATCH] tests: Use get_program_path in test_run function venkata.pyla
@ 2021-10-28 13:00 ` Venkata.Pyla
  2021-10-29  8:34 ` [Fuego] [PATCH v2] " venkata.pyla
  1 sibling, 0 replies; 4+ messages in thread
From: Venkata.Pyla @ 2021-10-28 13:00 UTC (permalink / raw)
  To: tim.bird; +Cc: dinesh.kumar, fuego

Hi Tim,

Below are some more tests that I have used `get_program_path`, 
kindly please review and give your feedback.


Thanks,
Venkata.

>-----Original Message-----
>From: venkata.pyla@toshiba-tsip.com <venkata.pyla@toshiba-tsip.com>
>Sent: 28 October 2021 18:29
>To: tim.bird@sony.com
>Cc: pyla venkata(ＴＳＩＰ) <Venkata.Pyla@toshiba-tsip.com>; sangorrin
>daniel(サンゴリン ダニエル □ＳＷＣ◯ＡＣＴ) <daniel.sangorrin@toshiba.co.jp>;
>fuego@lists.linuxfoundation.org; dinesh kumar(ＴＳＩＰ)
><dinesh.kumar@toshiba-tsip.com>
>Subject: [PATCH] tests: Use get_program_path in test_run function
>
>From: venkata pyla <venkata.pyla@toshiba-tsip.com>
>
>In order to use the test programs installed in the board's default PATH use
>get_program_path function, which will find the program in the PATH location or
>in the board test directory.
>
>Signed-off-by: venkata pyla <venkata.pyla@toshiba-tsip.com>
>---
> tests/Benchmark.migratetest/fuego_test.sh | 3 ++-
> tests/Benchmark.pmqtest/fuego_test.sh     | 3 ++-
> tests/Benchmark.ptsematest/fuego_test.sh  | 3 ++-
>tests/Benchmark.signaltest/fuego_test.sh  | 3 ++-
>tests/Benchmark.sigwaittest/fuego_test.sh | 3 ++-
>tests/Benchmark.svsematest/fuego_test.sh  | 3 ++-
> tests/Functional.pi_tests/fuego_test.sh   | 3 ++-
> 7 files changed, 14 insertions(+), 7 deletions(-)
>
>diff --git a/tests/Benchmark.migratetest/fuego_test.sh
>b/tests/Benchmark.migratetest/fuego_test.sh
>index eeaa4f6..af5d8a5 100755
>--- a/tests/Benchmark.migratetest/fuego_test.sh
>+++ b/tests/Benchmark.migratetest/fuego_test.sh
>@@ -23,5 +23,6 @@ function test_deploy {  }
>
> function test_run {
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./${TEST_COMMAND}
>$BENCHMARK_MIGRATETEST_PARAMS"
>+    get_program_path rt-migrate-test
>+    report "$PROGRAM_RT_MIGRATE_TEST
>$BENCHMARK_MIGRATETEST_PARAMS"
> }
>diff --git a/tests/Benchmark.pmqtest/fuego_test.sh
>b/tests/Benchmark.pmqtest/fuego_test.sh
>index 6733e78..72a8fce 100755
>--- a/tests/Benchmark.pmqtest/fuego_test.sh
>+++ b/tests/Benchmark.pmqtest/fuego_test.sh
>@@ -27,5 +27,6 @@ function test_run {
>     # The number for getting the lines depends on the cpu number of target
>machine.
>     target_cpu_number=$(cmd "nproc")
>     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./pmqtest
>$BENCHMARK_PMQTEST_PARAMS | tail -$getting_line_number"
>+    get_program_path pmqtest
>+    report "$PROGRAM_PMQTEST $BENCHMARK_PMQTEST_PARAMS | tail -
>$getting_line_number"
> }
>diff --git a/tests/Benchmark.ptsematest/fuego_test.sh
>b/tests/Benchmark.ptsematest/fuego_test.sh
>index a626d44..8ddb53c 100755
>--- a/tests/Benchmark.ptsematest/fuego_test.sh
>+++ b/tests/Benchmark.ptsematest/fuego_test.sh
>@@ -27,5 +27,6 @@ function test_run {
>     # The number for getting the lines depends on the cpu number of target
>machine.
>     target_cpu_number=$(cmd "cat /proc/cpuinfo | grep processor | wc -l")
>     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./ptsematest
>$BENCHMARK_PTSEMATEST_PARAMS | tail -$getting_line_number"
>+    get_program_path ptsematest
>+    report "$PROGRAM_PTSEMATEST $BENCHMARK_PTSEMATEST_PARAMS |
>tail -$getting_line_number"
> }
>diff --git a/tests/Benchmark.signaltest/fuego_test.sh
>b/tests/Benchmark.signaltest/fuego_test.sh
>index 54e08b1..b70f99d 100755
>--- a/tests/Benchmark.signaltest/fuego_test.sh
>+++ b/tests/Benchmark.signaltest/fuego_test.sh
>@@ -22,5 +22,6 @@ function test_deploy {  }
>
> function test_run {
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./signaltest
>$BENCHMARK_SIGNALTEST_PARAMS"
>+    get_program_path signaltest
>+    report "$PROGRAM_SIGNALTEST $BENCHMARK_SIGNALTEST_PARAMS"
> }
>diff --git a/tests/Benchmark.sigwaittest/fuego_test.sh
>b/tests/Benchmark.sigwaittest/fuego_test.sh
>index e419f17..57b118a 100755
>--- a/tests/Benchmark.sigwaittest/fuego_test.sh
>+++ b/tests/Benchmark.sigwaittest/fuego_test.sh
>@@ -27,5 +27,6 @@ function test_run {
>     # The number for getting the lines depends on the cpu number of target
>machine.
>     target_cpu_number=$(cmd "nproc")
>     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./sigwaittest
>$BENCHMARK_SIGWAITTEST_PARAMS | tail -$getting_line_number"
>+    get_program_path sigwaittest
>+    report "$PROGRAM_SIGWAITTEST $BENCHMARK_SIGWAITTEST_PARAMS |
>tail -$getting_line_number"
> }
>diff --git a/tests/Benchmark.svsematest/fuego_test.sh
>b/tests/Benchmark.svsematest/fuego_test.sh
>index 2f6e914..89d8335 100755
>--- a/tests/Benchmark.svsematest/fuego_test.sh
>+++ b/tests/Benchmark.svsematest/fuego_test.sh
>@@ -27,5 +27,6 @@ function test_run {
>     # The number for getting the lines depends on the cpu number of target
>machine.
>     target_cpu_number=$(cmd "cat /proc/cpuinfo | grep processor | wc -l")
>     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./svsematest
>$BENCHMARK_SVSEMATEST_PARAMS | tail -$getting_line_number"
>+    get_program_path svsematest
>+    report "$PROGRAM_SVSEMATEST $BENCHMARK_SVSEMATEST_PARAMS |
>tail -$getting_line_number"
> }
>diff --git a/tests/Functional.pi_tests/fuego_test.sh
>b/tests/Functional.pi_tests/fuego_test.sh
>index bf94a63..e0783ea 100755
>--- a/tests/Functional.pi_tests/fuego_test.sh
>+++ b/tests/Functional.pi_tests/fuego_test.sh
>@@ -22,7 +22,8 @@ function test_deploy {  }
>
> function test_run {
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./pi_stress
>$FUNCTIONAL_PI_TESTS_PARAMS"
>+    get_program_path pi_stress
>+    report "$PROGRAM_PI_STRESS $FUNCTIONAL_PI_TESTS_PARAMS"
> }
>
> function test_processing {
>--
>2.20.1



^ permalink raw reply	[flat|nested] 4+ messages in thread
* [Fuego] [PATCH v2] tests: Use get_program_path in test_run function
  2021-10-28 12:58 [Fuego] [PATCH] tests: Use get_program_path in test_run function venkata.pyla
  2021-10-28 13:00 ` Venkata.Pyla
@ 2021-10-29  8:34 ` venkata.pyla
  2021-10-29  8:41   ` Venkata.Pyla
  1 sibling, 1 reply; 4+ messages in thread
From: venkata.pyla @ 2021-10-29  8:34 UTC (permalink / raw)
  To: tim.bird; +Cc: dinesh.kumar, fuego

From: venkata pyla <venkata.pyla@toshiba-tsip.com>

In order to use the test programs installed in the board's
default PATH use get_program_path function, which will
find the program in the PATH location or in the
board test directory.

Signed-off-by: venkata pyla <venkata.pyla@toshiba-tsip.com>
---
 tests/Benchmark.fio/fuego_test.sh         | 5 +++--
 tests/Benchmark.migratetest/fuego_test.sh | 3 ++-
 tests/Benchmark.pmqtest/fuego_test.sh     | 3 ++-
 tests/Benchmark.ptsematest/fuego_test.sh  | 3 ++-
 tests/Benchmark.signaltest/fuego_test.sh  | 3 ++-
 tests/Benchmark.sigwaittest/fuego_test.sh | 3 ++-
 tests/Benchmark.svsematest/fuego_test.sh  | 3 ++-
 tests/Functional.pi_tests/fuego_test.sh   | 3 ++-
 8 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/tests/Benchmark.fio/fuego_test.sh b/tests/Benchmark.fio/fuego_test.sh
index 039ec15..363a5ef 100755
--- a/tests/Benchmark.fio/fuego_test.sh
+++ b/tests/Benchmark.fio/fuego_test.sh
@@ -36,8 +36,9 @@ function test_deploy {
 function test_run {
     hd_test_mount_prepare $BENCHMARK_FIO_MOUNT_BLOCKDEV $BENCHMARK_FIO_MOUNT_POINT
 
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./fio ssd-test-current --timeout=$BENCHMARK_FIO_TIMEOUT"
-    report_append "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./fio fsx-current --timeout=$BENCHMARK_FIO_TIMEOUT"
+    get_program_path fio
+    report "$PROGRAM_FIO ssd-test-current --timeout=$BENCHMARK_FIO_TIMEOUT"
+    report_append "$PROGRAM_FIO fsx-current --timeout=$BENCHMARK_FIO_TIMEOUT"
 
     hd_test_clean_umount $BENCHMARK_FIO_MOUNT_BLOCKDEV $BENCHMARK_FIO_MOUNT_POINT
 }
diff --git a/tests/Benchmark.migratetest/fuego_test.sh b/tests/Benchmark.migratetest/fuego_test.sh
index eeaa4f6..af5d8a5 100755
--- a/tests/Benchmark.migratetest/fuego_test.sh
+++ b/tests/Benchmark.migratetest/fuego_test.sh
@@ -23,5 +23,6 @@ function test_deploy {
 }
 
 function test_run {
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./${TEST_COMMAND} $BENCHMARK_MIGRATETEST_PARAMS"
+    get_program_path rt-migrate-test
+    report "$PROGRAM_RT_MIGRATE_TEST $BENCHMARK_MIGRATETEST_PARAMS"
 }
diff --git a/tests/Benchmark.pmqtest/fuego_test.sh b/tests/Benchmark.pmqtest/fuego_test.sh
index 6733e78..72a8fce 100755
--- a/tests/Benchmark.pmqtest/fuego_test.sh
+++ b/tests/Benchmark.pmqtest/fuego_test.sh
@@ -27,5 +27,6 @@ function test_run {
     # The number for getting the lines depends on the cpu number of target machine.
     target_cpu_number=$(cmd "nproc")
     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./pmqtest $BENCHMARK_PMQTEST_PARAMS | tail -$getting_line_number"
+    get_program_path pmqtest
+    report "$PROGRAM_PMQTEST $BENCHMARK_PMQTEST_PARAMS | tail -$getting_line_number"
 }
diff --git a/tests/Benchmark.ptsematest/fuego_test.sh b/tests/Benchmark.ptsematest/fuego_test.sh
index a626d44..8ddb53c 100755
--- a/tests/Benchmark.ptsematest/fuego_test.sh
+++ b/tests/Benchmark.ptsematest/fuego_test.sh
@@ -27,5 +27,6 @@ function test_run {
     # The number for getting the lines depends on the cpu number of target machine.
     target_cpu_number=$(cmd "cat /proc/cpuinfo | grep processor | wc -l")
     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./ptsematest $BENCHMARK_PTSEMATEST_PARAMS | tail -$getting_line_number"
+    get_program_path ptsematest
+    report "$PROGRAM_PTSEMATEST $BENCHMARK_PTSEMATEST_PARAMS | tail -$getting_line_number"
 }
diff --git a/tests/Benchmark.signaltest/fuego_test.sh b/tests/Benchmark.signaltest/fuego_test.sh
index 54e08b1..b70f99d 100755
--- a/tests/Benchmark.signaltest/fuego_test.sh
+++ b/tests/Benchmark.signaltest/fuego_test.sh
@@ -22,5 +22,6 @@ function test_deploy {
 }
 
 function test_run {
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./signaltest $BENCHMARK_SIGNALTEST_PARAMS"
+    get_program_path signaltest
+    report "$PROGRAM_SIGNALTEST $BENCHMARK_SIGNALTEST_PARAMS"
 }
diff --git a/tests/Benchmark.sigwaittest/fuego_test.sh b/tests/Benchmark.sigwaittest/fuego_test.sh
index e419f17..57b118a 100755
--- a/tests/Benchmark.sigwaittest/fuego_test.sh
+++ b/tests/Benchmark.sigwaittest/fuego_test.sh
@@ -27,5 +27,6 @@ function test_run {
     # The number for getting the lines depends on the cpu number of target machine.
     target_cpu_number=$(cmd "nproc")
     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./sigwaittest $BENCHMARK_SIGWAITTEST_PARAMS | tail -$getting_line_number"
+    get_program_path sigwaittest
+    report "$PROGRAM_SIGWAITTEST $BENCHMARK_SIGWAITTEST_PARAMS | tail -$getting_line_number"
 }
diff --git a/tests/Benchmark.svsematest/fuego_test.sh b/tests/Benchmark.svsematest/fuego_test.sh
index 2f6e914..89d8335 100755
--- a/tests/Benchmark.svsematest/fuego_test.sh
+++ b/tests/Benchmark.svsematest/fuego_test.sh
@@ -27,5 +27,6 @@ function test_run {
     # The number for getting the lines depends on the cpu number of target machine.
     target_cpu_number=$(cmd "cat /proc/cpuinfo | grep processor | wc -l")
     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./svsematest $BENCHMARK_SVSEMATEST_PARAMS | tail -$getting_line_number"
+    get_program_path svsematest
+    report "$PROGRAM_SVSEMATEST $BENCHMARK_SVSEMATEST_PARAMS | tail -$getting_line_number"
 }
diff --git a/tests/Functional.pi_tests/fuego_test.sh b/tests/Functional.pi_tests/fuego_test.sh
index bf94a63..e0783ea 100755
--- a/tests/Functional.pi_tests/fuego_test.sh
+++ b/tests/Functional.pi_tests/fuego_test.sh
@@ -22,7 +22,8 @@ function test_deploy {
 }
 
 function test_run {
-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./pi_stress $FUNCTIONAL_PI_TESTS_PARAMS"
+    get_program_path pi_stress
+    report "$PROGRAM_PI_STRESS $FUNCTIONAL_PI_TESTS_PARAMS"
 }
 
 function test_processing {
-- 
2.20.1



^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [Fuego] [PATCH v2] tests: Use get_program_path in test_run function
  2021-10-29  8:34 ` [Fuego] [PATCH v2] " venkata.pyla
@ 2021-10-29  8:41   ` Venkata.Pyla
  0 siblings, 0 replies; 4+ messages in thread
From: Venkata.Pyla @ 2021-10-29  8:41 UTC (permalink / raw)
  To: Venkata.Pyla, tim.bird; +Cc: dinesh.kumar, fuego

Hi Tim,

I have update my previous patch, with adding one more test (Benchmark.fio) for using get_program_path function.
Please review this latest patch.

I will share one more patch for lmbench2 for using get_program_path, as it has slightly different implementation to use distribution installed binaries.

Thanks,
Venkata.

>-----Original Message-----
>From: venkata.pyla@toshiba-tsip.com <venkata.pyla@toshiba-tsip.com>
>Sent: 29 October 2021 14:04
>To: tim.bird@sony.com
>Cc: pyla venkata(ＴＳＩＰ) <Venkata.Pyla@toshiba-tsip.com>; sangorrin
>daniel(サンゴリン ダニエル □ＳＷＣ◯ＡＣＴ) <daniel.sangorrin@toshiba.co.jp>;
>fuego@lists.linuxfoundation.org; dinesh kumar(ＴＳＩＰ)
><dinesh.kumar@toshiba-tsip.com>
>Subject: [PATCH v2] tests: Use get_program_path in test_run function
>
>From: venkata pyla <venkata.pyla@toshiba-tsip.com>
>
>In order to use the test programs installed in the board's default PATH use
>get_program_path function, which will find the program in the PATH location or
>in the board test directory.
>
>Signed-off-by: venkata pyla <venkata.pyla@toshiba-tsip.com>
>---
> tests/Benchmark.fio/fuego_test.sh         | 5 +++--
> tests/Benchmark.migratetest/fuego_test.sh | 3 ++-
> tests/Benchmark.pmqtest/fuego_test.sh     | 3 ++-
> tests/Benchmark.ptsematest/fuego_test.sh  | 3 ++-
>tests/Benchmark.signaltest/fuego_test.sh  | 3 ++-
>tests/Benchmark.sigwaittest/fuego_test.sh | 3 ++-
>tests/Benchmark.svsematest/fuego_test.sh  | 3 ++-
> tests/Functional.pi_tests/fuego_test.sh   | 3 ++-
> 8 files changed, 17 insertions(+), 9 deletions(-)
>
>diff --git a/tests/Benchmark.fio/fuego_test.sh
>b/tests/Benchmark.fio/fuego_test.sh
>index 039ec15..363a5ef 100755
>--- a/tests/Benchmark.fio/fuego_test.sh
>+++ b/tests/Benchmark.fio/fuego_test.sh
>@@ -36,8 +36,9 @@ function test_deploy {  function test_run {
>     hd_test_mount_prepare $BENCHMARK_FIO_MOUNT_BLOCKDEV
>$BENCHMARK_FIO_MOUNT_POINT
>
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./fio ssd-test-current --
>timeout=$BENCHMARK_FIO_TIMEOUT"
>-    report_append "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./fio fsx-current --
>timeout=$BENCHMARK_FIO_TIMEOUT"
>+    get_program_path fio
>+    report "$PROGRAM_FIO ssd-test-current --
>timeout=$BENCHMARK_FIO_TIMEOUT"
>+    report_append "$PROGRAM_FIO fsx-current --
>timeout=$BENCHMARK_FIO_TIMEOUT"
>
>     hd_test_clean_umount $BENCHMARK_FIO_MOUNT_BLOCKDEV
>$BENCHMARK_FIO_MOUNT_POINT  } diff --git
>a/tests/Benchmark.migratetest/fuego_test.sh
>b/tests/Benchmark.migratetest/fuego_test.sh
>index eeaa4f6..af5d8a5 100755
>--- a/tests/Benchmark.migratetest/fuego_test.sh
>+++ b/tests/Benchmark.migratetest/fuego_test.sh
>@@ -23,5 +23,6 @@ function test_deploy {  }
>
> function test_run {
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./${TEST_COMMAND}
>$BENCHMARK_MIGRATETEST_PARAMS"
>+    get_program_path rt-migrate-test
>+    report "$PROGRAM_RT_MIGRATE_TEST
>$BENCHMARK_MIGRATETEST_PARAMS"
> }
>diff --git a/tests/Benchmark.pmqtest/fuego_test.sh
>b/tests/Benchmark.pmqtest/fuego_test.sh
>index 6733e78..72a8fce 100755
>--- a/tests/Benchmark.pmqtest/fuego_test.sh
>+++ b/tests/Benchmark.pmqtest/fuego_test.sh
>@@ -27,5 +27,6 @@ function test_run {
>     # The number for getting the lines depends on the cpu number of target
>machine.
>     target_cpu_number=$(cmd "nproc")
>     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./pmqtest
>$BENCHMARK_PMQTEST_PARAMS | tail -$getting_line_number"
>+    get_program_path pmqtest
>+    report "$PROGRAM_PMQTEST $BENCHMARK_PMQTEST_PARAMS | tail -
>$getting_line_number"
> }
>diff --git a/tests/Benchmark.ptsematest/fuego_test.sh
>b/tests/Benchmark.ptsematest/fuego_test.sh
>index a626d44..8ddb53c 100755
>--- a/tests/Benchmark.ptsematest/fuego_test.sh
>+++ b/tests/Benchmark.ptsematest/fuego_test.sh
>@@ -27,5 +27,6 @@ function test_run {
>     # The number for getting the lines depends on the cpu number of target
>machine.
>     target_cpu_number=$(cmd "cat /proc/cpuinfo | grep processor | wc -l")
>     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./ptsematest
>$BENCHMARK_PTSEMATEST_PARAMS | tail -$getting_line_number"
>+    get_program_path ptsematest
>+    report "$PROGRAM_PTSEMATEST $BENCHMARK_PTSEMATEST_PARAMS |
>tail -$getting_line_number"
> }
>diff --git a/tests/Benchmark.signaltest/fuego_test.sh
>b/tests/Benchmark.signaltest/fuego_test.sh
>index 54e08b1..b70f99d 100755
>--- a/tests/Benchmark.signaltest/fuego_test.sh
>+++ b/tests/Benchmark.signaltest/fuego_test.sh
>@@ -22,5 +22,6 @@ function test_deploy {  }
>
> function test_run {
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./signaltest
>$BENCHMARK_SIGNALTEST_PARAMS"
>+    get_program_path signaltest
>+    report "$PROGRAM_SIGNALTEST $BENCHMARK_SIGNALTEST_PARAMS"
> }
>diff --git a/tests/Benchmark.sigwaittest/fuego_test.sh
>b/tests/Benchmark.sigwaittest/fuego_test.sh
>index e419f17..57b118a 100755
>--- a/tests/Benchmark.sigwaittest/fuego_test.sh
>+++ b/tests/Benchmark.sigwaittest/fuego_test.sh
>@@ -27,5 +27,6 @@ function test_run {
>     # The number for getting the lines depends on the cpu number of target
>machine.
>     target_cpu_number=$(cmd "nproc")
>     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./sigwaittest
>$BENCHMARK_SIGWAITTEST_PARAMS | tail -$getting_line_number"
>+    get_program_path sigwaittest
>+    report "$PROGRAM_SIGWAITTEST $BENCHMARK_SIGWAITTEST_PARAMS |
>tail -$getting_line_number"
> }
>diff --git a/tests/Benchmark.svsematest/fuego_test.sh
>b/tests/Benchmark.svsematest/fuego_test.sh
>index 2f6e914..89d8335 100755
>--- a/tests/Benchmark.svsematest/fuego_test.sh
>+++ b/tests/Benchmark.svsematest/fuego_test.sh
>@@ -27,5 +27,6 @@ function test_run {
>     # The number for getting the lines depends on the cpu number of target
>machine.
>     target_cpu_number=$(cmd "cat /proc/cpuinfo | grep processor | wc -l")
>     getting_line_number=$(( $target_cpu_number + $target_cpu_number ))
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./svsematest
>$BENCHMARK_SVSEMATEST_PARAMS | tail -$getting_line_number"
>+    get_program_path svsematest
>+    report "$PROGRAM_SVSEMATEST $BENCHMARK_SVSEMATEST_PARAMS |
>tail -$getting_line_number"
> }
>diff --git a/tests/Functional.pi_tests/fuego_test.sh
>b/tests/Functional.pi_tests/fuego_test.sh
>index bf94a63..e0783ea 100755
>--- a/tests/Functional.pi_tests/fuego_test.sh
>+++ b/tests/Functional.pi_tests/fuego_test.sh
>@@ -22,7 +22,8 @@ function test_deploy {  }
>
> function test_run {
>-    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; ./pi_stress
>$FUNCTIONAL_PI_TESTS_PARAMS"
>+    get_program_path pi_stress
>+    report "$PROGRAM_PI_STRESS $FUNCTIONAL_PI_TESTS_PARAMS"
> }
>
> function test_processing {
>--
>2.20.1



^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-10-29  8:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-28 12:58 [Fuego] [PATCH] tests: Use get_program_path in test_run function venkata.pyla
2021-10-28 13:00 ` Venkata.Pyla
2021-10-29  8:34 ` [Fuego] [PATCH v2] " venkata.pyla
2021-10-29  8:41   ` Venkata.Pyla

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.