| Message ID | 1731403483-21399-7-git-send-email-wangmy@fujitsu.com |
|---|---|
| State | New |
| Headers | show |
| Series | [01/47] barebox-tools: upgrade 2024.09.0 -> 2024.10.0 | expand |
On Tue, Nov 12, 2024 at 05:24:03PM +0800, wangmy via lists.openembedded.org wrote: > From: Wang Mingyu <wangmy@fujitsu.com> > > 0001-m4-readline-add-missing-includes.patch > removed since it's included in 5.3.1 > > Changelog: > =========== > - More subtle issues related to uninitialized array elements have > been fixed. > - A number of bugs in the debugger related to handling of arrays > have been fixed. > - Some subtle bugs in the API have been fixed. > - Use of MPFR is now possible again on 32-bit Power PC Mac systems. > - Race conditions around broken pipes for system() and read and write > pipes should now be closed off. > - Support for OSF/1 has been removed. > - The never-documented --nostalgia option has been removed. It was > causing bug reports. > - The implementation of printf/sprintf has been thoroughly reworked > in order to make the code more maintainable and to fix a goodly > number of corner cases. > - As usual, there have been several minor code cleanups and bug fixes. > It looks like this patch is breaking some ptests on the autobuilder: Failed ptests: {'gawk': ['printf-corners']} https://valkyrie.yoctoproject.org/#/builders/61/builds/388/steps/12/logs/stdio https://valkyrie.yoctoproject.org/#/builders/73/builds/390/steps/12/logs/stdio Can you have a look at this please?
On Mon, Nov 18, 2024 at 01:36:47PM +0000, Mingyu Wang (Fujitsu) wrote: > I didn't find ptest error of gawk from the log. > I did the ptest of gawk by running ptest-runner and there's no error occurred. > Can you tell me how to reproduce the error. > Ok, reproducing the ptest build locally is indeed not straightforward. I used the following procedure to reproduce something close to the setup we have on the autobuilder, and I did reproduce the ptest failure. First, if you want to use the exact same branch, you can take mathieu/master-next-e78c1f31b592 from git://git.yoctoproject.org/poky-contrib. Then, I created a new empty configuration and added entries in local.conf, to correspond roughly to what is described in https://git.yoctoproject.org/yocto-autobuilder2/tree/config.py. There is probably a lot of things not relevant here, but it was good enough to build and reproduce locally. rm -rf build source poky/oe-init-build-env QEMU_USE_KVM = 'True' INHERIT += 'report-error' PREMIRRORS = '' BB_GENERATE_MIRROR_TARBALLS = '1' BB_NUMBER_THREADS = '16' BB_NUMBER_PARSE_THREADS = '16' PARALLEL_MAKE = '-j 16 -l 75' BB_PRESSURE_MAX_CPU = '20000' BB_PRESSURE_MAX_IO = '20000' XZ_MEMLIMIT = '5%' XZ_THREADS = '8' ZSTD_THREADS = '8' BB_TASK_NICE_LEVEL = '5' BB_TASK_NICE_LEVEL:task-testimage = '0' BB_TASK_IONICE_LEVEL = '2.7' BB_TASK_IONICE_LEVEL:task-testimage = '2.1' IMAGE_CLASSES += 'testimage' TEST_QEMUBOOT_TIMEOUT = '1500' SANITY_TESTED_DISTROS = '' SDK_EXT_TYPE = 'minimal' SDK_INCLUDE_TOOLCHAIN = '1' ESDK_LOCALCONF_REMOVE:append = 'BB_HASHSERVE' BB_DISKMON_DIRS = 'STOPTASKS,${TMPDIR},1G,100K STOPTASKS,${DL_DIR},1G STOPTASKS,${SSTATE_DIR},1G STOPTASKS,/tmp,100M,30K HALT,${TMPDIR},100M,1K HALT,${DL_DIR},100M HALT,${SSTATE_DIR},100M HALT,/tmp,10M,1K' BB_HEARTBEAT_EVENT = '60' BB_LOG_HOST_STAT_ON_INTERVAL = '1' BB_LOG_HOST_STAT_CMDS_INTERVAL = 'oe-time-dd-test.sh -c 100 -t 3' BB_LOG_HOST_STAT_ON_FAILURE = '1' BB_LOG_HOST_STAT_CMDS_FAILURE = 'oe-time-dd-test.sh -l' SDK_TOOLCHAIN_LANGS += 'rust' BB_SERVER_TIMEOUT = '60' BB_LOADFACTOR_MAX = '1.5' TEST_SUITES = 'ping ssh ptest' INHERIT += 'image-buildinfo' IMAGE_BUILDINFO_VARS:append = ' IMAGE_BASENAME IMAGE_NAME' EOF Then built my image and ran ptests. The runqemu-gen-tapdevs might or might not be needed, more details in documentation/dev-manual/runtime-testing.rst. bitbake core-image-ptest-gawk sudo ${BUILDDIR}/../scripts/runqemu-gen-tapdevs 1000 4 bitbake core-image-ptest-gawk:do_testimage And I got this output: WARNING: core-image-ptest-gawk-1.0-r0 do_testimage: There were failing ptests. Traceback (most recent call last): File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/core/decorator/__init__.py", line 35, in wrapped_f return func(*args, **kwargs) ^^^^^^^^^^^^^^^^^^^^^ File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/core/decorator/__init__.py", line 35, in wrapped_f return func(*args, **kwargs) ^^^^^^^^^^^^^^^^^^^^^ File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/core/decorator/__init__.py", line 35, in wrapped_f return func(*args, **kwargs) ^^^^^^^^^^^^^^^^^^^^^ File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/runtime/cases/ptest.py", line 27, in test_ptestrunner_expectfail self.do_ptestrunner() File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/runtime/cases/ptest.py", line 120, in do_ptestrunner self.fail(failmsg) AssertionError: Failed ptests: {'gawk': ['printf-corners']}
I'd also note that the failure log shows where you can find the detailed testimage log: http://valkyrie.yocto.io/pub/non-release/20241115-76/testresults/qemux86-64-ptest/core-image-ptest-gawk/ There's a bit more detail there about the failure (and some strange binary dump from tar that shouldn't be there). Alex On Mon, 18 Nov 2024 at 17:21, Mathieu Dubois-Briand via lists.openembedded.org <mathieu.dubois-briand=bootlin.com@lists.openembedded.org> wrote: > > On Mon, Nov 18, 2024 at 01:36:47PM +0000, Mingyu Wang (Fujitsu) wrote: > > I didn't find ptest error of gawk from the log. > > I did the ptest of gawk by running ptest-runner and there's no error occurred. > > Can you tell me how to reproduce the error. > > > > Ok, reproducing the ptest build locally is indeed not straightforward. I > used the following procedure to reproduce something close to the setup we have > on the autobuilder, and I did reproduce the ptest failure. > > First, if you want to use the exact same branch, you can take > mathieu/master-next-e78c1f31b592 from > git://git.yoctoproject.org/poky-contrib. > > Then, I created a new empty configuration and added entries in > local.conf, to correspond roughly to what is described in > https://git.yoctoproject.org/yocto-autobuilder2/tree/config.py. There is > probably a lot of things not relevant here, but it was good enough to > build and reproduce locally. > > rm -rf build > source poky/oe-init-build-env > QEMU_USE_KVM = 'True' > INHERIT += 'report-error' > PREMIRRORS = '' > BB_GENERATE_MIRROR_TARBALLS = '1' > BB_NUMBER_THREADS = '16' > BB_NUMBER_PARSE_THREADS = '16' > PARALLEL_MAKE = '-j 16 -l 75' > BB_PRESSURE_MAX_CPU = '20000' > BB_PRESSURE_MAX_IO = '20000' > XZ_MEMLIMIT = '5%' > XZ_THREADS = '8' > ZSTD_THREADS = '8' > BB_TASK_NICE_LEVEL = '5' > BB_TASK_NICE_LEVEL:task-testimage = '0' > BB_TASK_IONICE_LEVEL = '2.7' > BB_TASK_IONICE_LEVEL:task-testimage = '2.1' > IMAGE_CLASSES += 'testimage' > TEST_QEMUBOOT_TIMEOUT = '1500' > SANITY_TESTED_DISTROS = '' > SDK_EXT_TYPE = 'minimal' > SDK_INCLUDE_TOOLCHAIN = '1' > ESDK_LOCALCONF_REMOVE:append = 'BB_HASHSERVE' > BB_DISKMON_DIRS = 'STOPTASKS,${TMPDIR},1G,100K STOPTASKS,${DL_DIR},1G > STOPTASKS,${SSTATE_DIR},1G STOPTASKS,/tmp,100M,30K > HALT,${TMPDIR},100M,1K HALT,${DL_DIR},100M HALT,${SSTATE_DIR},100M > HALT,/tmp,10M,1K' > BB_HEARTBEAT_EVENT = '60' > BB_LOG_HOST_STAT_ON_INTERVAL = '1' > BB_LOG_HOST_STAT_CMDS_INTERVAL = 'oe-time-dd-test.sh -c 100 -t 3' > BB_LOG_HOST_STAT_ON_FAILURE = '1' > BB_LOG_HOST_STAT_CMDS_FAILURE = 'oe-time-dd-test.sh -l' > SDK_TOOLCHAIN_LANGS += 'rust' > BB_SERVER_TIMEOUT = '60' > BB_LOADFACTOR_MAX = '1.5' > > TEST_SUITES = 'ping ssh ptest' > INHERIT += 'image-buildinfo' > IMAGE_BUILDINFO_VARS:append = ' IMAGE_BASENAME IMAGE_NAME' > EOF > > Then built my image and ran ptests. The runqemu-gen-tapdevs might or > might not be needed, more details in > documentation/dev-manual/runtime-testing.rst. > > bitbake core-image-ptest-gawk > sudo ${BUILDDIR}/../scripts/runqemu-gen-tapdevs 1000 4 > bitbake core-image-ptest-gawk:do_testimage > > And I got this output: > > WARNING: core-image-ptest-gawk-1.0-r0 do_testimage: There were failing ptests. > Traceback (most recent call last): > File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/core/decorator/__init__.py", line 35, in wrapped_f > return func(*args, **kwargs) > ^^^^^^^^^^^^^^^^^^^^^ > File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/core/decorator/__init__.py", line 35, in wrapped_f > return func(*args, **kwargs) > ^^^^^^^^^^^^^^^^^^^^^ > File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/core/decorator/__init__.py", line 35, in wrapped_f > return func(*args, **kwargs) > ^^^^^^^^^^^^^^^^^^^^^ > File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/runtime/cases/ptest.py", line 27, in test_ptestrunner_expectfail > self.do_ptestrunner() > File "/home/mathieu/projects/swat/builds/poky-contrib/meta/lib/oeqa/runtime/cases/ptest.py", line 120, in do_ptestrunner > self.fail(failmsg) > AssertionError: > Failed ptests: > {'gawk': ['printf-corners']} > > -- > Mathieu Dubois-Briand, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#207221): https://lists.openembedded.org/g/openembedded-core/message/207221 > Mute This Topic: https://lists.openembedded.org/mt/109531966/1686489 > Group Owner: openembedded-core+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- >
diff --git a/meta/recipes-extended/gawk/gawk/0001-m4-readline-add-missing-includes.patch b/meta/recipes-extended/gawk/gawk/0001-m4-readline-add-missing-includes.patch deleted file mode 100644 index 6fffd1f373..0000000000 --- a/meta/recipes-extended/gawk/gawk/0001-m4-readline-add-missing-includes.patch +++ /dev/null @@ -1,38 +0,0 @@ -From 4f4e84f139e2a8682f1374a592f2636c43ad857b Mon Sep 17 00:00:00 2001 -From: Ross Burton <ross.burton@arm.com> -Date: Tue, 21 May 2024 15:10:11 +0000 -Subject: [PATCH] m4/readline: add missing includes - -The cross-specific code fragment only includes stdio.h, where the native -fragment also includes fcntl.h and unistd.h. This is important because -GCC 14.1 has made the implicit definitions an error: - -conftest.c: In function 'main': -conftest.c:144:9: error: implicit declaration of function 'close'; did you mean 'pclose'? [-Wimplicit-function-declaration] -conftest.c:146:14: error: implicit declaration of function 'open'; did you mean 'popen'? [-Wimplicit-function-declaration] - -Add the missing includes so that the check doesn't always fail due to -these errors. - -Upstream-Status: Backport [https://git.savannah.gnu.org/cgit/gawk.git/commit/?id=c1613c310d71b87f509458e0259ecd10eda2b140] -Signed-off-by: Ross Burton <ross.burton@arm.com> ---- - m4/readline.m4 | 2 ++ - 1 file changed, 2 insertions(+) - -diff --git a/m4/readline.m4 b/m4/readline.m4 -index 38f96326..efd52d4e 100644 ---- a/m4/readline.m4 -+++ b/m4/readline.m4 -@@ -66,6 +66,8 @@ dnl action if false: - dnl action if cross compiling: - [AC_LINK_IFELSE( - [AC_LANG_PROGRAM([[#include <stdio.h> -+#include <fcntl.h> -+#include <unistd.h> - #include <readline/readline.h> - #include <readline/history.h>]], dnl includes - dnl function body --- -2.34.1 - diff --git a/meta/recipes-extended/gawk/gawk_5.3.0.bb b/meta/recipes-extended/gawk/gawk_5.3.1.bb similarity index 95% rename from meta/recipes-extended/gawk/gawk_5.3.0.bb rename to meta/recipes-extended/gawk/gawk_5.3.1.bb index ac9d8500d6..50b891ef14 100644 --- a/meta/recipes-extended/gawk/gawk_5.3.0.bb +++ b/meta/recipes-extended/gawk/gawk_5.3.1.bb @@ -16,11 +16,10 @@ PACKAGECONFIG[readline] = "--with-readline,--without-readline,readline" PACKAGECONFIG[mpfr] = "--with-mpfr,--without-mpfr, mpfr" SRC_URI = "${GNU_MIRROR}/gawk/gawk-${PV}.tar.gz \ - file://0001-m4-readline-add-missing-includes.patch \ file://run-ptest \ " -SRC_URI[sha256sum] = "378f8864ec21cfceaa048f7e1869ac9b4597b449087caf1eb55e440d30273336" +SRC_URI[sha256sum] = "fa41b3a85413af87fb5e3a7d9c8fa8d4a20728c67651185bb49c38a7f9382b1e" inherit autotools gettext texinfo update-alternatives