]> code.ossystems Code Review - openembedded-core.git/commitdiff
strace: skip a number of load-sensitive tests
authorRoss Burton <ross@burtonini.com>
Mon, 31 Jan 2022 17:40:13 +0000 (17:40 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 3 Feb 2022 09:01:58 +0000 (09:01 +0000)
Some tests are very load-sensitive as they depend on sleep() calls to
take a precise amount of time. These will randomly fail in the automated
QA, so skip them.

Signed-off-by: Ross Burton <ross.burton@arm.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/recipes-devtools/strace/strace/skip-load.patch [new file with mode: 0644]
meta/recipes-devtools/strace/strace_5.16.bb

diff --git a/meta/recipes-devtools/strace/strace/skip-load.patch b/meta/recipes-devtools/strace/strace/skip-load.patch
new file mode 100644 (file)
index 0000000..fa3ddb8
--- /dev/null
@@ -0,0 +1,55 @@
+Skip tests which are known to be unreliable under load, typically because they
+care about timing.
+
+Upstream-Status: Inappropriate
+Signed-off-by: Ross Burton <ross.burton@arm.com>
+
+diff --git a/tests/clock_nanosleep.gen.test b/tests/clock_nanosleep.gen.test
+index 7a6025b..f0e6dbb 100755
+--- a/tests/clock_nanosleep.gen.test
++++ b/tests/clock_nanosleep.gen.test
+@@ -1,4 +1,5 @@
+ #!/bin/sh -efu
+ # Generated by ./tests/gen_tests.sh from ./tests/gen_tests.in (clock_nanosleep -e trace=clock_nanosleep,clock_gettime); do not edit.
+ . "${srcdir=.}/init.sh"
++skip_ "Test not reliable under load"
+ run_strace_match_diff -e trace=clock_nanosleep,clock_gettime
+diff --git a/tests/delay.test b/tests/delay.test
+index f74e27f..328087e 100755
+--- a/tests/delay.test
++++ b/tests/delay.test
+@@ -9,6 +9,8 @@
+ . "${srcdir=.}/init.sh"
++skip_ "Test not reliable under load"
++
+ while read -r denter dexit denter_us dexit_us; do
+       [ -n "$denter" ] || continue
+diff --git a/tests/strace-T.test b/tests/strace-T.test
+index adca3e8..d4acfe9 100755
+--- a/tests/strace-T.test
++++ b/tests/strace-T.test
+@@ -9,6 +9,8 @@
+ . "${srcdir=.}/init.sh"
++skip_ "Test not reliable under load"
++
+ T_opt="${1:--T}"
+ run_prog ../sleep 0
+diff --git a/tests/strace-r.test b/tests/strace-r.test
+index 8299737..c360344 100755
+--- a/tests/strace-r.test
++++ b/tests/strace-r.test
+@@ -9,6 +9,8 @@
+ . "${srcdir=.}/init.sh"
++skip_ "Test not reliable under load"
++
+ r_opt="${1:--r}"
+ run_prog ../sleep 0
index 1574b8e7773fb24725bde11834380e66b475dc73..a9c22d9ae0ac8c4c57562c429bf586044560cfa6 100644 (file)
@@ -12,6 +12,7 @@ SRC_URI = "https://strace.io/files/${PV}/strace-${PV}.tar.xz \
            file://0001-caps-abbrev.awk-fix-gawk-s-path.patch \
            file://ptest-spacesave.patch \
            file://0001-strace-fix-reproducibilty-issues.patch \
+           file://skip-load.patch \
            "
 SRC_URI[sha256sum] = "dc7db230ff3e57c249830ba94acab2b862da1fcaac55417e9b85041a833ca285"