Ha, idiot me.  I didn't see that bit of the regex and the test suite
takes so long I didn't actually test it...

Other opinions on whether to we skip the test or relax the bounds?

Ross

On Thu, 21 Oct 2021 at 16:10, Alexandre Belloni
<alexandre.bell...@bootlin.com> wrote:
>
> On 21/10/2021 15:20:04+0100, Ross Burton wrote:
> > Sigh.  I'll just remove the test entirely then.
>
> Actually, I think your regex is not correct because it also failed on
> qemux86. I believe you are missing a digit in the case it takes more
> than one second, before you had '1.[01]' then 5 digits, you changed it to
> '1.' and 5 digits, this should be 6
>
> >
> > Ross
> >
> > On Thu, 21 Oct 2021 at 14:48, Alexandre Belloni
> > <alexandre.bell...@bootlin.com> wrote:
> > >
> > > 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.bur...@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
> > > >
> > >
> > > >
> > > > 
> > > >
> > >
> > >
> > > --
> > > Alexandre Belloni, co-owner and COO, Bootlin
> > > Embedded Linux and Kernel engineering
> > > https://bootlin.com
>
> --
> Alexandre Belloni, co-owner and COO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#157287): 
https://lists.openembedded.org/g/openembedded-core/message/157287
Mute This Topic: https://lists.openembedded.org/mt/86448553/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to