[Linaro-TCWG-CI] glibc-2.39.9000-340-g176671f604: FAIL: 1 regressions on aarch64
Dear contributor, our automatic CI has detected problems related to your patch(es). Please find some details below. If you have any questions, please follow up on linaro-toolchain@lists.linaro.org mailing list, Libera's #linaro-tcwg channel, or ping your favourite Linaro toolchain developer on the usual project channel. We appreciate that it might be difficult to find the necessary logs or reproduce the issue locally. If you can't get what you need from our CI within minutes, let us know and we will be happy to help. We track this report status in https://linaro.atlassian.net/browse/GNU-1261 , please let us know if you are looking at the problem and/or when you have a fix. In glibc_check master-aarch64 after: | commit glibc-2.39.9000-340-g176671f604 | Author: Carlos Llamas | Date: Tue Jun 18 10:56:34 2024 +0200 | | linux: add definitions for hugetlb page size encodings | | A desired hugetlb page size can be encoded in the flags parameter of | system calls such as mmap() and shmget(). The Linux UAPI headers have | included explicit definitions for these encodings since v4.14. | | This patch adds these definitions that are used along with MAP_HUGETLB | ... 14 lines of the commit log omitted. FAIL: 1 regressions regressions.sum: === glibc tests === Running glibc:misc ... FAIL: misc/tst-mman-consts You can find the failure logs in *.log.1.xz files in - https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artifact/artifacts/00-sumfiles/ The full lists of regressions and progressions as well as configure and make commands are in - https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artifact/artifacts/notify/ The list of [ignored] baseline and flaky failures are in - https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artifact/artifacts/sumfiles/xfails.xfail The configuration of this build is: CI config tcwg_glibc_check master-aarch64 -8<--8<--8<-- The information below can be used to reproduce a debug environment: Current build : https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artifact/artifacts Reference build : https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1478/artifact/artifacts Reproduce last good and first bad builds: https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/glibc/sha1/176671f6042912200ea9733bb6cc8212e06bc85e/tcwg_glibc_check/master-aarch64/reproduction_instructions.txt Full commit : https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=176671f6042912200ea9733bb6cc8212e06bc85e List of configurations that regressed due to this commit : * tcwg_glibc_check ** master-aarch64 *** FAIL: 1 regressions *** https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/glibc/sha1/176671f6042912200ea9733bb6cc8212e06bc85e/tcwg_glibc_check/master-aarch64/details.txt *** https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artifact/artifacts ___ linaro-toolchain mailing list -- linaro-toolchain@lists.linaro.org To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org
Re: [Linaro-TCWG-CI] glibc-2.39.9000-340-g176671f604: FAIL: 1 regressions on aarch64
* ci notify: > In glibc_check master-aarch64 after: > > | commit glibc-2.39.9000-340-g176671f604 > | Author: Carlos Llamas > | Date: Tue Jun 18 10:56:34 2024 +0200 > | > | linux: add definitions for hugetlb page size encodings > | > | A desired hugetlb page size can be encoded in the flags parameter of > | system calls such as mmap() and shmget(). The Linux UAPI headers have > | included explicit definitions for these encodings since v4.14. > | > | This patch adds these definitions that are used along with MAP_HUGETLB > | ... 14 lines of the commit log omitted. > > FAIL: 1 regressions > > regressions.sum: > === glibc tests === > > Running glibc:misc ... > FAIL: misc/tst-mman-consts This must be the error I saw elsewhere: | First source: | #define _GNU_SOURCE 1 | #include | | | Second source: | #define _GNU_SOURCE 1 | #include | | | Only in first source: MAP_ABOVE4G | Different values for MAP_HUGE_16GB: 2281701376 != -2013265920 The reason is that the kernel changed the definition of MAP_HUGE_16GB. Older UAPI headers use an undefined expression which is treated by compilers as a signed int, hence t he discrepancy. This was fixed on the kernel side in this commit: commit 710bb68c2e3a24512e2d2bae470960d7488e97b1 Author: Matthias Goergens Date: Mon Sep 5 11:19:04 2022 +0800 hugetlb_encode.h: fix undefined behaviour (34 << 26) Left-shifting past the size of your datatype is undefined behaviour in C. The literal 34 gets the type `int`, and that one is not big enough to be left shifted by 26 bits. An `unsigned` is long enough (on any machine that has at least 32 bits for their ints.) For uniformity, we mark all the literals as unsigned. But it's only really needed for HUGETLB_FLAG_ENCODE_16GB. Thanks to Randy Dunlap for an initial review and suggestion. Link: https://lkml.kernel.org/r/20220905031904.150925-1-matthias.goerg...@gmail.com Signed-off-by: Matthias Goergens Acked-by: Randy Dunlap Cc: Mike Kravetz Cc: Muchun Song Signed-off-by: Andrew Morton Not sure what to do about this on the glibc side. Can we waive this in the CI, or should try to fix up the glibc test? Thanks, Florian ___ linaro-toolchain mailing list -- linaro-toolchain@lists.linaro.org To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org
Re: [Linaro-TCWG-CI] glibc-2.39.9000-340-g176671f604: FAIL: 1 regressions on aarch64
On Tue, Jun 18, 2024 at 06:37:29PM +0200, Florian Weimer wrote: > | First source: > | #define _GNU_SOURCE 1 > | #include > | > | > | Second source: > | #define _GNU_SOURCE 1 > | #include > | > | > | Only in first source: MAP_ABOVE4G > | Different values for MAP_HUGE_16GB: 2281701376 != -2013265920 > > The reason is that the kernel changed the definition of MAP_HUGE_16GB. > Older UAPI headers use an undefined expression which is treated by > compilers as a signed int, hence t he discrepancy. > Oops, sorry I never tested this against older kernel headers. > This was fixed on the kernel side in this commit: > > commit 710bb68c2e3a24512e2d2bae470960d7488e97b1 > Author: Matthias Goergens > Date: Mon Sep 5 11:19:04 2022 +0800 > > hugetlb_encode.h: fix undefined behaviour (34 << 26) > > Left-shifting past the size of your datatype is undefined behaviour in C. > The literal 34 gets the type `int`, and that one is not big enough to be > left shifted by 26 bits. > > An `unsigned` is long enough (on any machine that has at least 32 bits for > their ints.) > > For uniformity, we mark all the literals as unsigned. But it's only > really needed for HUGETLB_FLAG_ENCODE_16GB. > > Thanks to Randy Dunlap for an initial review and suggestion. > > Link: > https://lkml.kernel.org/r/20220905031904.150925-1-matthias.goerg...@gmail.com > Signed-off-by: Matthias Goergens > Acked-by: Randy Dunlap > Cc: Mike Kravetz > Cc: Muchun Song > Signed-off-by: Andrew Morton > Sounds like this kernel commit should have been backported to stable but never was. The glibc test caught the issue. I'll go ahead and backport the fix to the linux stable branches. > Not sure what to do about this on the glibc side. Can we waive this in > the CI, or should try to fix up the glibc test? Hopefully this can be done until the fix propagates to the older kernel's UAPI. Thanks Carlos Llamas ___ linaro-toolchain mailing list -- linaro-toolchain@lists.linaro.org To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org
