Progress:
* UM-2 [QEMU upstream maintainership]
- Noticed that if the guest uses the HVC insn at EL3 we incorrectly take
the exception to EL2, not EL3; sent a patch to fix this
- looking into an issue with FEAT_MOPS copies going backwards across
page boundaries
* QEMU-292 [ARMv8.3 FEAT_NV, Nested Virtualization]
- realised we need to do some cleanup of how we handle the FEAT_VHE register
aliases, as the way we currently do them makes the FEAT_NV register
trapping a little awkward. Wrote the necessary patches; they'll go
into the FEAT_NV patchset when it's eventually ready to send out.
-- PMM
On Wed, 8 Nov 2023, ci_notify--- via Gcc-regression 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 gcc_check master-arm after:
>
> | commit gcc-14-5237-gf27246b0096
> | Author: Joseph Myers <joseph(a)codesourcery.com>
> | Date: Wed Nov 8 00:15:22 2023 +0000
> |
> | testsuite: Rename c2x-*, gnu2x-* tests to c23-*, gnu23-*
> |
> | Completing the move to refer to C23 in place of C2X, rename all tests
> | with "c2x" or "gnu2x" in their names to use "c23" or "gnu23" instead.
> | 17 files in the testsuite that referred to such tests (or, in one
> | case, a generated .i file to be scanned) by those names are updated
> | for the renaming.
> | ... 467 lines of the commit log omitted.
>
> FAIL: 8 regressions: 8 progressions
>
> regressions.sum:
> === gcc tests ===
>
> Running gcc:gcc.dg/dg.exp ...
> FAIL: gcc.dg/c23-stdarg-4.c execution test
>
> Running gcc:gcc.dg/torture/dg-torture.exp ...
> FAIL: gcc.dg/torture/c23-stdarg-split-1a.c -O0 execution test
> FAIL: gcc.dg/torture/c23-stdarg-split-1a.c -O1 execution test
> FAIL: gcc.dg/torture/c23-stdarg-split-1a.c -O2 execution test
> FAIL: gcc.dg/torture/c23-stdarg-split-1a.c -O2 -flto -fno-use-linker-plugin -flto-partition=none execution test
> ... and 5 more entries
This is clearly just a renaming of tests that were already failing. I'd
certainly encourage the relevant target maintainers to investigate and fix
the failing tests (which probably indicate a back-end change is needed to
make variable argument access work correctly in a function with no named
arguments - some architectures were fixed by the change in bug 107453, but
it wouldn't be surprising if other architectures need back-end changes).
--
Joseph S. Myers
joseph(a)codesourcery.com
It also caused x86 backend regression. Please help verify it.
Thx,
Haochen
> -----Original Message-----
> From: Gcc-regression <gcc-regression-
> bounces+haochen.jiang=intel.com(a)gcc.gnu.org> On Behalf Of ci_notify--- via
> Gcc-regression
> Sent: Saturday, November 4, 2023 2:43 PM
> To: uecker(a)tugraz.at
> Cc: ci_notify(a)linaro.org; gcc-regression(a)gcc.gnu.org
> Subject: [Linaro-TCWG-CI] gcc-14-5115-g6e9ee44d96e: FAIL: 1 regressions
> on arm
>
> 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 gcc_check master-arm after:
>
> | commit gcc-14-5115-g6e9ee44d96e
> | Author: Martin Uecker <uecker(a)tugraz.at>
> | Date: Thu Apr 13 19:35:15 2023 +0200
> |
> | Reduce false positives for -Wnonnull for VLA parameters [PR98541]
> |
> | This patch limits the warning about NULL arguments to VLA
> | parameters declared [static n].
> |
> | PR c/98541
> |
> | ... 8 lines of the commit log omitted.
>
> FAIL: 1 regressions
>
> regressions.sum:
> === gcc tests ===
>
> Running gcc:gcc.dg/dg.exp ...
> FAIL: gcc.dg/Wnonnull-4.c (test for excess errors)
>
> === Results Summary ===
>
> You can find the failure logs in *.log.1.xz files in
> - https://ci.linaro.org/job/tcwg_gcc_check--master-arm-
> build/1213/artifact/artifacts/00-sumfiles/ .
> The full lists of regressions and progressions are in
> - https://ci.linaro.org/job/tcwg_gcc_check--master-arm-
> build/1213/artifact/artifacts/notify/ .
> The list of [ignored] baseline and flaky failures are in
> - https://ci.linaro.org/job/tcwg_gcc_check--master-arm-
> build/1213/artifact/artifacts/sumfiles/xfails.xfail .
>
> The configuration of this build is:
> CI config tcwg_gcc_check/master-arm
>
> -----------------8<--------------------------8<--------------------------8<----------------
> ----------
> The information below can be used to reproduce a debug environment:
>
> Current build : https://ci.linaro.org/job/tcwg_gcc_check--master-arm-
> build/1213/artifact/artifacts
> Reference build : https://ci.linaro.org/job/tcwg_gcc_check--master-arm-
> build/1212/artifact/artifacts
>
> Reproduce last good and first bad builds: https://git-
> us.linaro.org/toolchain/ci/interesting-
> commits.git/plain/gcc/sha1/6e9ee44d96e5bda8808dd9d8ccf58d2525383f6
> b/tcwg_gcc_check/master-arm/reproduction_instructions.txt
>
> Full commit : https://github.com/gcc-
> mirror/gcc/commit/6e9ee44d96e5bda8808dd9d8ccf58d2525383f6b
>
> Latest bug report status : https://linaro.atlassian.net/browse/GNU-993
>
> List of configurations that regressed due to this commit :
> * tcwg_gcc_check
> ** master-arm
> *** FAIL: 1 regressions
> *** https://git-us.linaro.org/toolchain/ci/interesting-
> commits.git/plain/gcc/sha1/6e9ee44d96e5bda8808dd9d8ccf58d2525383f6
> b/tcwg_gcc_check/master-arm/details.txt
> *** https://ci.linaro.org/job/tcwg_gcc_check--master-arm-build/1213/
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