Re: [OE-core] [PATCH 2/3] strace: handle loaded test systems

[Alexandre Belloni <alexandre.belloni@bootlin.com>]

Hi Ross,

On 19/10/2021 20:26:13+0100, Ross Burton wrote:
> If the host is loaded when running the tests there is a good chance
> that strace-T.test will fail, as it does a sleep(1) and checks that
> the nanosleep() call lasted for 0.9-1.1 seconds.
> 
> Relax the test so that the test passes if the call takes less than two
> seconds.
> 
> [ YOCTO #14596 ]
> 
> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
>  meta/recipes-devtools/strace/strace/ptest-sleep.patch | 7 +++++++
>  meta/recipes-devtools/strace/strace_5.14.bb           | 1 +
>  2 files changed, 8 insertions(+)
>  create mode 100644 meta/recipes-devtools/strace/strace/ptest-sleep.patch
> 
> diff --git a/meta/recipes-devtools/strace/strace/ptest-sleep.patch b/meta/recipes-devtools/strace/strace/ptest-sleep.patch
> new file mode 100644
> index 0000000000..5629dec319
> --- /dev/null
> +++ b/meta/recipes-devtools/strace/strace/ptest-sleep.patch
> @@ -0,0 +1,7 @@
> +diff --git a/tests/strace-T.expected b/tests/strace-T.expected
> +index 1ea020465..f78d3e2c8 100644
> +--- a/tests/strace-T.expected
> ++++ b/tests/strace-T.expected
> +@@ -1 +1 @@
> +-nanosleep\(\{tv_sec=1, tv_nsec=0\}, NULL\) = 0 <(1\.[01]|0\.9)[[:digit:]]{5}>
> ++nanosleep\(\{tv_sec=1, tv_nsec=0\}, NULL\) = 0 <(1\.|0\.9)[[:digit:]]{5}>

Those two builds have the patch and reliably failed:

https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/2425/steps/12/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/2428/steps/12/logs/stdio

> diff --git a/meta/recipes-devtools/strace/strace_5.14.bb b/meta/recipes-devtools/strace/strace_5.14.bb
> index a81bda04ba..9ae1367337 100644
> --- a/meta/recipes-devtools/strace/strace_5.14.bb
> +++ b/meta/recipes-devtools/strace/strace_5.14.bb
> @@ -13,6 +13,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://ptest-sleep.patch \
>             "
>  SRC_URI[sha256sum] = "901bee6db5e17debad4530dd9ffb4dc9a96c4a656edbe1c3141b7cb307b11e73"
>  
> -- 
> 2.25.1
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#157202): https://lists.openembedded.org/g/openembedded-core/message/157202
> Mute This Topic: https://lists.openembedded.org/mt/86448553/3617179
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alexandre.belloni@bootlin.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com