On 28/07/2023 17:25:46+0200, Alexander Kanavin wrote:
> I just locally ran the ptest against current abelloni/master-next
> (1db230c0191eefffd94be0e8f40312b76e8b8769) with arm-64 target on an
> x86 host, and that passed too. I can try to do that on the actual arm
> host in the AB cluster, but otherwise out of ideas at the moment. Can
> you bisect the patchset?
>
This is not 100% reproducible and so difficult to bisect. This is the
log:
diff-{{{
--- /usr/lib/util-linux/ptest/tests/expected/lsfd/option-inet
+++ /usr/lib/util-linux/ptest/tests/output/lsfd/option-inet
@@ -5,10 +5,10 @@
3 TCPv6 state=listen laddr=[::1]:34567
4 TCPv6 state=established laddr=[::1]:23456 raddr=[::1]:34567
5 TCPv6 state=established laddr=[::1]:34567 raddr=[::1]:23456
- 3 UDP state=close laddr=127.0.0.1:34567
- 4 UDP state=established laddr=127.0.0.1:23456 raddr=127.0.0.1:34567
- 3 UDPLITEv6 state=close laddr=[::1]:34567
- 4 UDPLITEv6 state=established laddr=[::1]:23456 raddr=[::1]:34567
+ 3 UDP socket:[33864]
+ 4 UDP socket:[33865]
+ 3 UDPLITEv6 socket:[31413]
+ 4 UDPLITEv6 socket:[31414]
# --inet=4
3 TCP state=listen laddr=127.0.0.1:34567
4 TCP state=established laddr=127.0.0.1:23456 raddr=127.0.0.1:34567
}}}-diff
FAIL: lsfd: --inet option
(lsfd/option-inet)
All the occurrences are on the arm workers, ubuntu1804-arm-1 and
ubuntu2204-arm-1 for now.
> Alex
>
> On Fri, 28 Jul 2023 at 15:38, Alexander Kanavin via
> lists.openembedded.org <alex.kanavin=gmail....@lists.openembedded.org>
> wrote:
> >
> > I don't know. Nothing stands out in particular here:
> > https://git.yoctoproject.org/poky-contrib/log/?h=abelloni/master-next
> >
> > Both failures happened on arm64 I think?
> >
> > Alex
> >
> > On Fri, 28 Jul 2023 at 15:13, Alexandre Belloni
> > <alexandre.bell...@bootlin.com> wrote:
> > >
> > > Hello,
> > >
> > > I got a new ptest failure twice with this patch:
> > >
> > > 'util-linux': ['lsfd:_--inet_option']
> > >
> > > I don't see it on your run, I'm wondering what is in my branch that
> > > causes it.
> > >
> > > On 27/07/2023 16:36:18+0200, Alexander Kanavin wrote:
> > > > License-update: added bsd-2-clause
> > > > https://github.com/util-linux/util-linux/commit/faeb1b64b36ededd1a0b62555cad65f002d47ac6
> > > >
> > > > Signed-off-by: Alexander Kanavin <a...@linutronix.de>
> > > > ---
> > > > ...2.38.1.bb => util-linux-libuuid_2.39.1.bb} | 0
> > > > meta/recipes-core/util-linux/util-linux.inc | 10 ++--
> > > > .../0001-check-for-sys-pidfd.h.patch | 53 -------------------
> > > > ...configure.ac-Improve-check-for-magic.patch | 40 --------------
> > > > .../util-linux/util-linux/run-ptest | 6 +++
> > > > ...l-linux_2.38.1.bb => util-linux_2.39.1.bb} | 0
> > > > 6 files changed, 10 insertions(+), 99 deletions(-)
> > > > rename meta/recipes-core/util-linux/{util-linux-libuuid_2.38.1.bb =>
> > > > util-linux-libuuid_2.39.1.bb} (100%)
> > > > delete mode 100644
> > > > meta/recipes-core/util-linux/util-linux/0001-check-for-sys-pidfd.h.patch
> > > > delete mode 100644
> > > > meta/recipes-core/util-linux/util-linux/0001-configure.ac-Improve-check-for-magic.patch
> > > > rename meta/recipes-core/util-linux/{util-linux_2.38.1.bb =>
> > > > util-linux_2.39.1.bb} (100%)
> > > >
> > > > diff --git a/meta/recipes-core/util-linux/util-linux-libuuid_2.38.1.bb
> > > > b/meta/recipes-core/util-linux/util-linux-libuuid_2.39.1.bb
> > > > similarity index 100%
> > > > rename from meta/recipes-core/util-linux/util-linux-libuuid_2.38.1.bb
> > > > rename to meta/recipes-core/util-linux/util-linux-libuuid_2.39.1.bb
> > > > diff --git a/meta/recipes-core/util-linux/util-linux.inc
> > > > b/meta/recipes-core/util-linux/util-linux.inc
> > > > index 3868b1c41d3..99ce2a24b0a 100644
> > > > --- a/meta/recipes-core/util-linux/util-linux.inc
> > > > +++ b/meta/recipes-core/util-linux/util-linux.inc
> > > > @@ -6,13 +6,13 @@ disk partitioning, kernel message management,
> > > > filesystem creation, and system lo
> > > >
> > > > SECTION = "base"
> > > >
> > > > -LICENSE = "GPL-2.0-or-later & LGPL-2.1-or-later & BSD-3-Clause &
> > > > BSD-4-Clause"
> > > > +LICENSE = "GPL-2.0-or-later & LGPL-2.1-or-later & BSD-2-Clause &
> > > > BSD-3-Clause & BSD-4-Clause"
> > > > LICENSE:${PN}-libblkid = "LGPL-2.1-or-later"
> > > > LICENSE:${PN}-libfdisk = "LGPL-2.1-or-later"
> > > > LICENSE:${PN}-libmount = "LGPL-2.1-or-later"
> > > > LICENSE:${PN}-libsmartcols = "LGPL-2.1-or-later"
> > > >
> > > > -LIC_FILES_CHKSUM =
> > > > "file://README.licensing;md5=0fd5c050c6187d2bf0a4492b7f4e33da \
> > > > +LIC_FILES_CHKSUM =
> > > > "file://README.licensing;md5=12ae7768a65ec8f2e44d930df9cb43fa \
> > > >
> > > > file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
> > > >
> > > > file://Documentation/licenses/COPYING.GPL-2.0-or-later;md5=b234ee4d69f5fce4486a80fdaf4a4263
> > > > \
> > > >
> > > > file://Documentation/licenses/COPYING.LGPL-2.1-or-later;md5=4fbd65380cdd255951079008b364516c
> > > > \
> > > > @@ -23,7 +23,7 @@ LIC_FILES_CHKSUM =
> > > > "file://README.licensing;md5=0fd5c050c6187d2bf0a4492b7f4e33da
> > > >
> > > > file://libblkid/COPYING;md5=693bcbbe16d3a4a4b37bc906bc01cc04 \
> > > >
> > > > file://libfdisk/COPYING;md5=693bcbbe16d3a4a4b37bc906bc01cc04 \
> > > >
> > > > file://libsmartcols/COPYING;md5=693bcbbe16d3a4a4b37bc906bc01cc04 \
> > > > -"
> > > > + "
> > > >
> > > > FILESEXTRAPATHS:prepend := "${THISDIR}/util-linux:"
> > > > MAJOR_VERSION = "${@'.'.join(d.getVar('PV').split('.')[0:2])}"
> > > > @@ -35,8 +35,6 @@ SRC_URI =
> > > > "${KERNELORG_MIRROR}/linux/utils/util-linux/v${MAJOR_VERSION}/util-lin
> > > > file://run-ptest \
> > > > file://display_testname_for_subtest.patch \
> > > > file://avoid_parallel_tests.patch \
> > > > - file://0001-check-for-sys-pidfd.h.patch \
> > > > - file://0001-configure.ac-Improve-check-for-magic.patch \
> > > > "
> > > >
> > > > -SRC_URI[sha256sum] =
> > > > "60492a19b44e6cf9a3ddff68325b333b8b52b6c59ce3ebd6a0ecaa4c5117e84f"
> > > > +SRC_URI[sha256sum] =
> > > > "890ae8ff810247bd19e274df76e8371d202cda01ad277681b0ea88eeaa00286b"
> > > > diff --git
> > > > a/meta/recipes-core/util-linux/util-linux/0001-check-for-sys-pidfd.h.patch
> > > >
> > > > b/meta/recipes-core/util-linux/util-linux/0001-check-for-sys-pidfd.h.patch
> > > > deleted file mode 100644
> > > > index 19f57f14bc7..00000000000
> > > > ---
> > > > a/meta/recipes-core/util-linux/util-linux/0001-check-for-sys-pidfd.h.patch
> > > > +++ /dev/null
> > > > @@ -1,53 +0,0 @@
> > > > -From 548bc568f3c735e53fb5b0a5ab6473a3f1457b91 Mon Sep 17 00:00:00 2001
> > > > -From: Khem Raj <raj.k...@gmail.com>
> > > > -Date: Sun, 7 Aug 2022 14:39:19 -0700
> > > > -Subject: [PATCH] check for sys/pidfd.h
> > > > -
> > > > -This header in newer glibc defines the signatures of functions
> > > > -pidfd_send_signal() and pidfd_open() and when these functions are
> > > > -defined by libc then we need to include the relevant header to get
> > > > -the definitions. Clang 15+ has started to error out when function
> > > > -signatures are missing.
> > > > -
> > > > -Fixes errors like
> > > > -misc-utils/kill.c:402:6: error: call to undeclared function
> > > > 'pidfd_send_signal'; ISO C99 and later do not support implicit function
> > > > declarations [-Wimplicit-function-declaration]
> > > > - if (pidfd_send_signal(pfd, ctl->numsig, &info, 0) < 0)
> > > > -
> > > > -Upstream-Status: Submitted
> > > > [https://github.com/util-linux/util-linux/pull/1769]
> > > > -Signed-off-by: Khem Raj <raj.k...@gmail.com>
> > > > ----
> > > > - configure.ac | 1 +
> > > > - include/pidfd-utils.h | 4 +++-
> > > > - 2 files changed, 4 insertions(+), 1 deletion(-)
> > > > -
> > > > -diff --git a/configure.ac b/configure.ac
> > > > -index a511e93de..fd7d9245f 100644
> > > > ---- a/configure.ac
> > > > -+++ b/configure.ac
> > > > -@@ -342,6 +342,7 @@ AC_CHECK_HEADERS([ \
> > > > - sys/mkdev.h \
> > > > - sys/mount.h \
> > > > - sys/param.h \
> > > > -+ sys/pidfd.h \
> > > > - sys/prctl.h \
> > > > - sys/resource.h \
> > > > - sys/sendfile.h \
> > > > -diff --git a/include/pidfd-utils.h b/include/pidfd-utils.h
> > > > -index eddede976..d9e33cbc5 100644
> > > > ---- a/include/pidfd-utils.h
> > > > -+++ b/include/pidfd-utils.h
> > > > -@@ -4,8 +4,10 @@
> > > > - #ifdef HAVE_SYS_SYSCALL_H
> > > > - # include <sys/syscall.h>
> > > > - # if defined(SYS_pidfd_send_signal) && defined(SYS_pidfd_open)
> > > > -+# ifdef HAVE_SYS_PIDFD_H
> > > > -+# include <sys/pidfd.h>
> > > > -+# endif
> > > > - # include <sys/types.h>
> > > > --
> > > > - # ifndef HAVE_PIDFD_SEND_SIGNAL
> > > > - static inline int pidfd_send_signal(int pidfd, int sig, siginfo_t
> > > > *info,
> > > > - unsigned int flags)
> > > > ---
> > > > -2.37.1
> > > > -
> > > > diff --git
> > > > a/meta/recipes-core/util-linux/util-linux/0001-configure.ac-Improve-check-for-magic.patch
> > > >
> > > > b/meta/recipes-core/util-linux/util-linux/0001-configure.ac-Improve-check-for-magic.patch
> > > > deleted file mode 100644
> > > > index 00611fe5788..00000000000
> > > > ---
> > > > a/meta/recipes-core/util-linux/util-linux/0001-configure.ac-Improve-check-for-magic.patch
> > > > +++ /dev/null
> > > > @@ -1,40 +0,0 @@
> > > > -From 263381ddd46eea2293c70bc811273b66bc52087b Mon Sep 17 00:00:00 2001
> > > > -From: Mateusz Marciniec <mateuszm...@gmail.com>
> > > > -Date: Fri, 19 Aug 2022 14:47:49 +0200
> > > > -Subject: [PATCH] configure.ac: Improve check for magic
> > > > -
> > > > -Check whether magic.h header exists before defining HAVE_MAGIC.
> > > > -
> > > > -Despite library availability there still can be missing header.
> > > > -Current test doesn't cover that possibility which will lead compilation
> > > > -to fail in case of separate sysroot.
> > > > -
> > > > -Upstream-Status: Backport
> > > > -[https://github.com/util-linux/util-linux/commit/263381ddd46eea2293c70bc811273b66bc52087b]
> > > > -
> > > > -Signed-off-by: Mateusz Marciniec <mateuszm...@gmail.com>
> > > > -Signed-off-by: Tomasz Dziendzielski <tomasz.dziendziel...@gmail.com>
> > > > ----
> > > > - configure.ac | 6 ++++--
> > > > - 1 file changed, 4 insertions(+), 2 deletions(-)
> > > > -
> > > > -diff --git a/configure.ac b/configure.ac
> > > > -index daa8f0dca..968a0daf0 100644
> > > > ---- a/configure.ac
> > > > -+++ b/configure.ac
> > > > -@@ -1570,8 +1570,10 @@ AC_ARG_WITH([libmagic],
> > > > - )
> > > > - AS_IF([test "x$with_libmagic" = xno], [have_magic=no], [
> > > > - AC_CHECK_LIB([magic], [magic_open], [
> > > > -- AC_DEFINE([HAVE_MAGIC], [1], [Define to 1 if you have the
> > > > libmagic present.])
> > > > -- MAGIC_LIBS="-lmagic"
> > > > -+ AC_CHECK_HEADER(magic.h, [
> > > > -+ AC_DEFINE([HAVE_MAGIC], [1], [Define to 1 if you have the
> > > > libmagic present.])
> > > > -+ MAGIC_LIBS="-lmagic"
> > > > -+ ])
> > > > - ])
> > > > - ])
> > > > - AC_SUBST([MAGIC_LIBS])
> > > > ---
> > > > -2.37.1
> > > > -
> > > > diff --git a/meta/recipes-core/util-linux/util-linux/run-ptest
> > > > b/meta/recipes-core/util-linux/util-linux/run-ptest
> > > > index 097107cd095..e25e9d10823 100644
> > > > --- a/meta/recipes-core/util-linux/util-linux/run-ptest
> > > > +++ b/meta/recipes-core/util-linux/util-linux/run-ptest
> > > > @@ -16,6 +16,12 @@ export PATH=$bindir/bin:$PATH
> > > > # losetup tests will be skipped and/or fail otherwise
> > > > modprobe loop
> > > >
> > > > +# required for mount/fallback test to pass
> > > > +# systemd does this by default, but ptest images do not use it
> > > > +# see https://man7.org/linux/man-pages/man7/mount_namespaces.7.html
> > > > +# for a long description of mount namespaces in Linux
> > > > +mount --make-shared /
> > > > +
> > > > ./tests/run.sh --use-system-commands --parsable --show-diff | sed -u '{
> > > > s/^\(.*\):\(.*\) \.\.\. OK$/PASS: \1:\2/
> > > > s/^\(.*\):\(.*\) \.\.\. FAILED \(.*\)$/FAIL: \1:\2 \3/
> > > > diff --git a/meta/recipes-core/util-linux/util-linux_2.38.1.bb
> > > > b/meta/recipes-core/util-linux/util-linux_2.39.1.bb
> > > > similarity index 100%
> > > > rename from meta/recipes-core/util-linux/util-linux_2.38.1.bb
> > > > rename to meta/recipes-core/util-linux/util-linux_2.39.1.bb
> > > > --
> > > > 2.30.2
> > > >
> > >
> > > >
> > > >
> > > >
> > >
> > >
> > > --
> > > 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 (#185113):
https://lists.openembedded.org/g/openembedded-core/message/185113
Mute This Topic: https://lists.openembedded.org/mt/100391613/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
