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?
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 > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#185037): https://lists.openembedded.org/g/openembedded-core/message/185037 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] -=-=-=-=-=-=-=-=-=-=-=-
