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 (#157285):
https://lists.openembedded.org/g/openembedded-core/message/157285
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]
-=-=-=-=-=-=-=-=-=-=-=-
