Successfully identified regression in *glibc* in CI configuration 
tcwg_cross/gnu-master-aarch64-build_cross.  So far, this commit has regressed 
CI configurations:
 - tcwg_cross/gnu-master-aarch64-build_cross

Culprit:
<cut>
commit 832f50be6c9c010e46180d14126bbb81f35e808c
Author: Florian Weimer <fwei...@redhat.com>
Date:   Tue Jul 6 13:22:01 2021 +0200

    elf: Call free from base namespace on error in dl-libc.c [BZ #27646]
    
    In dlerror_run, free corresponds to the local malloc in the
    namespace, but GLRO (dl_catch_error) uses the malloc from the base
    namespace.  elf/tst-dlmopen-gethostbyname triggers this mismatch,
    but it does not crash, presumably because of a fastbin deallocation.
    
    Fixes commit c2059edce20c124d1a99f1a94cc52e83b77a917a ("elf: Use
    _dl_catch_error from base namespace in dl-libc.c [BZ #27646]") and
    commit b2964eb1d9a6b8ab1250e8a881cf406182da5875 ("dlfcn: Failures
    after dlmopen should not terminate process [BZ #24772]").
</cut>

Results regressed to (for first_bad == 832f50be6c9c010e46180d14126bbb81f35e808c)
# reset_artifacts:
-10
# true:
0
# build_abe binutils:
1
# build_abe stage1:
2
# build_abe linux:
3
# First few build errors in logs:
# 00:00:17 make[2]: *** 
[/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/sysd-rules:499:
 
/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/csu/start.os]
 Error 1
# 00:00:17 make[2]: *** 
[/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/sysd-rules:483:
 
/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/csu/start.o]
 Error 1
# 00:00:17 make[2]: *** 
[/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/sysd-rules:483:
 
/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/csu/crtn.o]
 Error 1
# 00:00:17 make[2]: *** 
[/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/sysd-rules:483:
 
/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/csu/crti.o]
 Error 1
# 00:00:17 make[2]: *** [../o-iterator.mk:9: 
/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/csu/check_fds.o]
 Error 1
# 00:00:17 make[2]: *** [../o-iterator.mk:9: 
/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/csu/check_fds.os]
 Error 1
# 00:00:17 ../sysdeps/unix/sysv/linux/aarch64/sysdep.h:190:17: error: invalid 
register name for ‘_x0’
# 00:00:17 ../sysdeps/unix/sysv/linux/aarch64/sysdep.h:174:22: error: invalid 
register name for ‘_x8’
# 00:00:17 ../sysdeps/aarch64/nptl/tls.h:95:21: error: 
‘__builtin_thread_pointer’ is not supported on this target
# 00:00:17 make[2]: *** 
[/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/sysd-rules:25:
 
/home/tcwg-buildslave/workspace/tcwg_gnu_1/abe/builds/x86_64-pc-linux-gnu/aarch64-linux-gnu/glibc-glibc.git~master/csu/libc-start.os]
 Error 1

from (for last_good == c8935581de4ff931bc32fb03da5d87f0ee2378a1)
# reset_artifacts:
-10
# true:
0
# build_abe binutils:
1
# build_abe stage1:
2
# build_abe linux:
3
# build_abe glibc:
4
# build_abe stage2:
5
# build_abe qemu:
6

Artifacts of last_good build: 
https://ci.linaro.org/job/tcwg_cross-bisect-gnu-master-aarch64-build_cross/21/artifact/artifacts/build-c8935581de4ff931bc32fb03da5d87f0ee2378a1/
Artifacts of first_bad build: 
https://ci.linaro.org/job/tcwg_cross-bisect-gnu-master-aarch64-build_cross/21/artifact/artifacts/build-832f50be6c9c010e46180d14126bbb81f35e808c/
Build top page/logs: 
https://ci.linaro.org/job/tcwg_cross-bisect-gnu-master-aarch64-build_cross/21/

Configuration details:


Reproduce builds:
<cut>
mkdir investigate-glibc-832f50be6c9c010e46180d14126bbb81f35e808c
cd investigate-glibc-832f50be6c9c010e46180d14126bbb81f35e808c

git clone https://git.linaro.org/toolchain/jenkins-scripts

mkdir -p artifacts/manifests
curl -o artifacts/manifests/build-baseline.sh 
https://ci.linaro.org/job/tcwg_cross-bisect-gnu-master-aarch64-build_cross/21/artifact/artifacts/manifests/build-baseline.sh
 --fail
curl -o artifacts/manifests/build-parameters.sh 
https://ci.linaro.org/job/tcwg_cross-bisect-gnu-master-aarch64-build_cross/21/artifact/artifacts/manifests/build-parameters.sh
 --fail
curl -o artifacts/test.sh 
https://ci.linaro.org/job/tcwg_cross-bisect-gnu-master-aarch64-build_cross/21/artifact/artifacts/test.sh
 --fail
chmod +x artifacts/test.sh

# Reproduce the baseline build (build all pre-requisites)
./jenkins-scripts/tcwg_gnu-build.sh @@ artifacts/manifests/build-baseline.sh

cd glibc

# Reproduce first_bad build
git checkout --detach 832f50be6c9c010e46180d14126bbb81f35e808c
../artifacts/test.sh

# Reproduce last_good build
git checkout --detach c8935581de4ff931bc32fb03da5d87f0ee2378a1
../artifacts/test.sh

cd ..
</cut>

History of pending regressions and results: 
https://git.linaro.org/toolchain/ci/base-artifacts.git/log/?h=linaro-local/ci/tcwg_cross/gnu-master-aarch64-build_cross

Artifacts: 
https://ci.linaro.org/job/tcwg_cross-bisect-gnu-master-aarch64-build_cross/21/artifact/artifacts/
Build log: 
https://ci.linaro.org/job/tcwg_cross-bisect-gnu-master-aarch64-build_cross/21/consoleText

Full commit (up to 1000 lines):
<cut>
commit 832f50be6c9c010e46180d14126bbb81f35e808c
Author: Florian Weimer <fwei...@redhat.com>
Date:   Tue Jul 6 13:22:01 2021 +0200

    elf: Call free from base namespace on error in dl-libc.c [BZ #27646]
    
    In dlerror_run, free corresponds to the local malloc in the
    namespace, but GLRO (dl_catch_error) uses the malloc from the base
    namespace.  elf/tst-dlmopen-gethostbyname triggers this mismatch,
    but it does not crash, presumably because of a fastbin deallocation.
    
    Fixes commit c2059edce20c124d1a99f1a94cc52e83b77a917a ("elf: Use
    _dl_catch_error from base namespace in dl-libc.c [BZ #27646]") and
    commit b2964eb1d9a6b8ab1250e8a881cf406182da5875 ("dlfcn: Failures
    after dlmopen should not terminate process [BZ #24772]").
---
 elf/dl-libc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/elf/dl-libc.c b/elf/dl-libc.c
index a49df22029..ba792e9589 100644
--- a/elf/dl-libc.c
+++ b/elf/dl-libc.c
@@ -48,7 +48,7 @@ dlerror_run (void (*operate) (void *), void *args)
                ?: last_errstring != NULL);
 
   if (result && malloced)
-    free ((char *) last_errstring);
+    GLRO (dl_error_free) ((char *) last_errstring);
 
   return result;
 }
</cut>
_______________________________________________
linaro-toolchain mailing list
linaro-toolchain@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/linaro-toolchain

Reply via email to