Hello,
# TCWG CI
- Working on Abe change to always add detected flaky tests to the
flaky.xfail file, even if they are already present in the baseline
results. Sent a few versions of Gerrit review request already.
# [GNU-981] FEAT_GCS (Guarded Control Stack) support in GDB
- Prepared environment to develop and test GDB: built Linux kernel from
Mark Brown's arm64-gcs branch and booted it in an Arm FVP supporting
the feature. Successfully ran native GDB in it to debug a program
(though I didn't use any GCS feature for this test).
--
Thiago
Progress:
* UM-2 [QEMU upstream maintainership]
- investigated and fixed a few bugs in FEAT_MOPS; patches sent
- reviewed patch fixing bug in virt ACPI table for debug UART; added
the necessary update-the-reference-files patches for 'make check'.
Debugged why it broke console output on Linux -- turns out that Linux
has a bug where acpi_parse_spcr() and pl011_console_match() disagree
about how the former should tell the latter "use 16-bit MMIO"...
We'll work around this by having the ACPI table report 32-bit MMIO;
Raspberry Pi 4 firmware on real hardware already does that, so it
should be a safe choice.
- sent patch fixing failure to report BTI GP state correctly when
doing a two-stage page table walk
- code review and assembly of a pull request (softfreeze next week...)
* QEMU-292 [ARMv8.3 FEAT_NV, Nested Virtualization]
- got to a (hopefully) code-complete but untested state; will probably
put this to one side for the moment leading up to softfreeze
- tracked down the kernel and kvmtool patchsets that will be needed
to test this (they are not upstream yet)
thanks
-- PMM
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(a)lists.linaro.org mailing list, Libera's #linaro-tcwg channel, or ping your favourite Linaro toolchain developer on the usual project channel.
In CI config tcwg_kernel/gnu-master-aarch64-mainline-allyesconfig after:
| commit gcc-14-5059-gd880e093d92
| Author: Martin Uecker <uecker(a)tugraz.at>
| Date: Thu Jul 27 13:36:05 2023 +0200
|
| c: Add Walloc-size to warn about insufficient size in allocations [PR71219]
|
| Add option Walloc-size that warns about allocations that have
| insufficient storage for the target type of the pointer the
| storage is assigned to. Added to Wextra.
|
| PR c/71219
| ... 14 lines of the commit log omitted.
Results changed to
# reset_artifacts:
-10
# build_abe binutils:
-9
# build_abe stage1:
-5
# build_abe qemu:
-2
# linux_n_obj:
22669
# First few build errors in logs:
# 00:33:49 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v8.c:241:15: error: allocation of insufficient size ‘4’ for type ‘uint32_t[2]’ {aka ‘unsigned int[2]’} with size ‘8’ [-Werror=alloc-size]
# 00:33:49 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v8.c:327:15: error: allocation of insufficient size ‘4’ for type ‘uint32_t[2]’ {aka ‘unsigned int[2]’} with size ‘8’ [-Werror=alloc-size]
# 00:33:49 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v9.c:368:15: error: allocation of insufficient size ‘4’ for type ‘uint32_t[2]’ {aka ‘unsigned int[2]’} with size ‘8’ [-Werror=alloc-size]
# 00:33:49 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v9.c:465:15: error: allocation of insufficient size ‘4’ for type ‘uint32_t[2]’ {aka ‘unsigned int[2]’} with size ‘8’ [-Werror=alloc-size]
# 00:33:50 make[6]: *** [scripts/Makefile.build:243: drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v8.o] Error 1
# 00:33:52 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_arcturus.c:203:15: error: allocation of insufficient size ‘4’ for type ‘uint32_t[2]’ {aka ‘unsigned int[2]’} with size ‘8’ [-Werror=alloc-size]
# 00:33:53 make[6]: *** [scripts/Makefile.build:243: drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_arcturus.o] Error 1
# 00:33:53 make[6]: *** [scripts/Makefile.build:243: drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v9.o] Error 1
# 00:33:56 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gc_9_4_3.c:144:15: error: allocation of insufficient size ‘4’ for type ‘uint32_t[2]’ {aka ‘unsigned int[2]’} with size ‘8’ [-Werror=alloc-size]
# 00:33:57 make[6]: *** [scripts/Makefile.build:243: drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gc_9_4_3.o] Error 1
From
# reset_artifacts:
-10
# build_abe binutils:
-9
# build_abe stage1:
-5
# build_abe qemu:
-2
# linux_n_obj:
22751
# linux build successful:
all
The configuration of this build is:
CI config tcwg_kernel/gnu-master-aarch64-mainline-allyesconfig
-----------------8<--------------------------8<--------------------------8<--------------------------
The information below can be used to reproduce a debug environment:
Current build : https://ci.linaro.org/job/tcwg_kernel--gnu-master-aarch64-mainline-allyesco…
Reference build : https://ci.linaro.org/job/tcwg_kernel--gnu-master-aarch64-mainline-allyesco…
Reproduce last good and first bad builds: https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/gcc/sh…
Full commit : https://github.com/gcc-mirror/gcc/commit/d880e093d92084f55b10626610ef059fd9…
Latest bug report status : https://linaro.atlassian.net/browse/GNU-990
List of configurations that regressed due to this commit :
* tcwg_kernel
** gnu-master-aarch64-mainline-allyesconfig
*** Failure
*** https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/gcc/sh…
*** https://ci.linaro.org/job/tcwg_kernel--gnu-master-aarch64-mainline-allyesco…
On Wed, Nov 1, 2023 at 7:42 AM <ci_notify(a)linaro.org> wrote:
>
> 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(a)lists.linaro.org mailing list, Libera's #linaro-tcwg channel, or ping your favourite Linaro toolchain developer on the usual project channel.
>
> In CI config tcwg_kernel/llvm-master-arm-mainline-allmodconfig after:
ok, so ARCH=arm allmodconfig on mainline...
>
> | commit v6.6-rc1-17-g1c6fdbd8f246
> | Author: Kent Overstreet <kent.overstreet(a)gmail.com>
> | Date: Thu Mar 16 22:18:50 2017 -0800
> |
> | bcachefs: Initial commit
> |
> | Initially forked from drivers/md/bcache, bcachefs is a new copy-on-write
> | filesystem with every feature you could possibly want.
> |
> | Website: https://bcachefs.org
> |
> | ... 1 lines of the commit log omitted.
>
> Results changed to
> # reset_artifacts:
> -10
> # build_abe binutils:
> -9
> # build_kernel_llvm:
> -5
> # build_abe qemu:
> -2
> # linux_n_obj:
> 23730
> # First few build errors in logs:
>
> # 00:23:16 fs/bcachefs/btree_cache.h:45:43: error: array index 0 is past the end of the array (that has type 'const __u64[0]' (aka 'const unsigned long long[0]')) [-Werror,-Warray-bounds]
> # 00:23:17 fs/bcachefs/alloc.c:332:9: error: call to undeclared function 'COUNT_ARGS'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
^
$ file fs/bcachefs/alloc.c
fs/bcachefs/alloc.c: cannot open `fs/bcachefs/alloc.c' (No such file
or directory)
> # 00:23:17 make[4]: *** [scripts/Makefile.build:243: fs/bcachefs/alloc.o] Error 1
> # 00:23:29 fs/bcachefs/btree_cache.h:45:43: error: array index 0 is past the end of the array (that has type 'const __u64[0]' (aka 'const unsigned long long[0]')) [-Werror,-Warray-bounds]
> # 00:23:30 make[4]: *** [scripts/Makefile.build:243: fs/bcachefs/bset.o] Error 1
^
$ make LLVM=1 ARCH=arm allmodconfig fs/bcachefs/bset.o
CC [M] fs/bcachefs/bset.o
$
> # 00:23:33 fs/bcachefs/btree_cache.h:45:43: error: array index 0 is past the end of the array (that has type 'const __u64[0]' (aka 'const unsigned long long[0]')) [-Werror,-Warray-bounds]
> # 00:23:33 fs/bcachefs/btree_cache.h:45:43: error: array index 0 is past the end of the array (that has type 'const __u64[0]' (aka 'const unsigned long long[0]')) [-Werror,-Warray-bounds]
> # 00:23:33 fs/bcachefs/btree_cache.c:67:9: error: array index 0 is past the end of the array (that has type 'const __u64[0]' (aka 'const unsigned long long[0]')) [-Werror,-Warray-bounds]
> # 00:23:33 fs/bcachefs/btree_cache.c:120:2: error: array index 0 is past the end of the array (that has type '__u64[0]' (aka 'unsigned long long[0]')) [-Werror,-Warray-bounds]
> # 00:23:33 fs/bcachefs/btree_cache.c:201:6: error: array index 0 is past the end of the array (that has type 'const __u64[0]' (aka 'const unsigned long long[0]')) [-Werror,-Warray-bounds]
$ make LLVM=1 ARCH=arm allmodconfig fs/bcachefs/btree_cache.o
CC [M] fs/bcachefs/btree_cache.o
$
... what's up with this report?
>
> From
> # reset_artifacts:
> -10
> # build_abe binutils:
> -9
> # build_kernel_llvm:
> -5
> # build_abe qemu:
> -2
> # linux_n_obj:
> 32815
> # linux build successful:
> all
> # linux boot successful:
> boot
>
> The configuration of this build is:
> CI config tcwg_kernel/llvm-master-arm-mainline-allmodconfig
>
> -----------------8<--------------------------8<--------------------------8<--------------------------
> The information below can be used to reproduce a debug environment:
>
> Current build : https://ci.linaro.org/job/tcwg_kernel--llvm-master-arm-mainline-allmodconfi…
> Reference build : https://ci.linaro.org/job/tcwg_kernel--llvm-master-arm-mainline-allmodconfi…
>
> Reproduce last good and first bad builds: https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/linux/…
>
> Full commit : https://git.linaro.org/kernel-org/linux.git/commit/?id=1c6fdbd8f2465ddfb73a…
>
> Latest bug report status : https://linaro.atlassian.net/browse/LLVM-1015
>
> List of configurations that regressed due to this commit :
> * tcwg_kernel
> ** llvm-master-arm-mainline-allmodconfig
> *** Failure
> *** https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/linux/…
> *** https://ci.linaro.org/job/tcwg_kernel--llvm-master-arm-mainline-allmodconfi…
--
Thanks,
~Nick Desaulniers
Progress:
* QEMU-610 [Remove qemu_get_cpu() from hardware models]
- Big rework of Cortex-A MPCore models. A common class is now used
and various duplicated code from ARM boards got removed.
Possible follow-up with QEMU-601 [Implement TYPE_CORTEXR52_PRIV].
Hello,
# [GNU-981] FEAT_GCS (Guarded Control Stack) support in GDB
- Started working on this feature. Currently adding ptrace and core file
support for the NT_ARM_GCS regset.
# Upstream community
- Implemented fix for ‘--with-auto-load-dir’ and
‘--with-auto-load-safe-path’ configure options on Windows (since Eli
mentioned it earlier this month) and posted “[PATCH] gdb/configure.ac:
Fix auto-load options to work with Windows path separator”.
--
Thiago
Progress:
* UM-2 [QEMU upstream maintainership]
- Took a prototype set of patches to add a second UART to the
virt board which I'd written before going off on holiday; cleaned
them up and got them into shape to send out for review. This is
likely to be QEMU 9.0 material because we want to wait for the
fix to EDK2 to make it handle the second uart more sensibly to
get into an upstream EDK2 before landing the QEMU patches.
- More code review work; sent this week's target-arm pull request
* QEMU-292 [ARMv8.3 FEAT_NV, Nested Virtualization]
- Started on implementation. This seems to be shaping up to be a less
difficult task than our initial estimate suggested; I have
code-complete patches for most of the necessary behaviour
changes already. (Testing may be a more drawn-out effort :-))
-- PMM