qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] d0068b: tests/docker: remove debian-armel-cro

From: Peter Maydell
Subject: [Qemu-commits] [qemu/qemu] d0068b: tests/docker: remove debian-armel-cross
Date: Wed, 11 Sep 2024 12:23:08 +0000 (UTC) 
  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: d0068b746a0a8cd4bb148527a0d199b130cd5288
      
https://github.com/qemu/qemu/commit/d0068b746a0a8cd4bb148527a0d199b130cd5288
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M .gitlab-ci.d/container-cross.yml
    M .gitlab-ci.d/crossbuilds.yml
    R tests/docker/dockerfiles/debian-armel-cross.docker
    M tests/lcitool/refresh

  Log Message:
  -----------
  tests/docker: remove debian-armel-cross

As debian-11 transitions to LTS we are starting to have problems
building the image. While we could update to a later Debian building a
32 bit QEMU without modern floating point is niche host amongst the
few remaining 32 bit hosts we regularly build for. For now we still
have armhf-debian-cross-container which is currently built from the
more recent debian-12.

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-2-alex.bennee@linaro.org>


  Commit: 19d2111059c87d3f58349f27b9be9dee81fc1681
      
https://github.com/qemu/qemu/commit/19d2111059c87d3f58349f27b9be9dee81fc1681
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M tests/docker/dockerfiles/debian-i686-cross.docker
    M tests/docker/dockerfiles/debian-mipsel-cross.docker
    M tests/lcitool/refresh

  Log Message:
  -----------
  tests/docker: update debian i686 and mipsel images to bookworm

Whatever issues there were which stopped these being updates when the
rest were have now been resolved. However mips64el continues to be
broken so don't update it here.

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-3-alex.bennee@linaro.org>


  Commit: 320d2a9d1346e53e5708a58265a7d0376b763e55
      
https://github.com/qemu/qemu/commit/320d2a9d1346e53e5708a58265a7d0376b763e55
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M configure

  Log Message:
  -----------
  tests/docker: use debian-all-test-cross for mips64el tests

While the mips64el cross compilation environment is busted in Debian
we can use the debian-all-test-cross image for building TCG tests like
we do in the CI.

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-4-alex.bennee@linaro.org>


  Commit: 613edf0c0dfe8d0e704eb1722fd6f1bbbd0856bb
      
https://github.com/qemu/qemu/commit/613edf0c0dfe8d0e704eb1722fd6f1bbbd0856bb
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M docs/devel/testing/main.rst

  Log Message:
  -----------
  docs/devel: fix duplicate line

I guess the same change came in via two patch series. Remove the
repetition.

Fixes: 2a851fca9f (docs/devel: remind developers to run CI container pipeline 
when updating images)
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-5-alex.bennee@linaro.org>


  Commit: 8c68c11cf3a940c14f23a599a78f0baf3c536d7a
      
https://github.com/qemu/qemu/commit/8c68c11cf3a940c14f23a599a78f0baf3c536d7a
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M scripts/ci/setup/gitlab-runner.yml

  Log Message:
  -----------
  scripts/ci: update the gitlab-runner playbook

The upstream install instructions:

  https://docs.gitlab.com/runner/install/linux-repository.html

Now refer to repositories and a setup script. Modernise the playbook
to use the preferred delivery method.

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-6-alex.bennee@linaro.org>


  Commit: f6110605313aaea66307f2470a85b7c053542442
      
https://github.com/qemu/qemu/commit/f6110605313aaea66307f2470a85b7c053542442
  Author: Gustavo Romero <gustavo.romero@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M target/arm/gdbstub64.c

  Log Message:
  -----------
  gdbstub: Use specific MMU index when probing MTE addresses

Use cpu_mmu_index() to determine the specific translation regime (MMU
index) before probing addresses using allocation_tag_mem_probe().

Currently, the MMU index is hardcoded to 0 and only works for user mode.
By obtaining the specific MMU index according to the translation regime,
future use of the stubs relying on allocation_tag_mem_probe in other
regimes will be possible, like in EL1.

This commit also changes the ptr_size value passed to
allocation_tag_mem_probe() from 8 to 1. The ptr_size parameter actually
represents the number of bytes in the memory access (which can be as
small as 1 byte), rather than the number of bits used in the address
space pointed to by ptr.

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20240906143316.657436-2-gustavo.romero@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-7-alex.bennee@linaro.org>


  Commit: 0298229ad6341898fb3bbacaf0eee4935e780fa4
      
https://github.com/qemu/qemu/commit/0298229ad6341898fb3bbacaf0eee4935e780fa4
  Author: Gustavo Romero <gustavo.romero@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M target/arm/gdbstub64.c

  Log Message:
  -----------
  gdbstub: Add support for MTE in system mode

This commit makes handle_q_memtag, handle_q_isaddresstagged, and
handle_Q_memtag stubs build for system mode, allowing all GDB
'memory-tag' subcommands to work with QEMU gdbstub on aarch64 system
mode.

Resolves: https://gitlab.com/qemu-project/qemu/-/issues/620
Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20240906143316.657436-3-gustavo.romero@linaro.org>
[AJB: add #ifdef CONFIG_TCG guards]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-8-alex.bennee@linaro.org>


  Commit: 3848409eb0f83fda5eb47e31d7deec5ae5ca46a1
      
https://github.com/qemu/qemu/commit/3848409eb0f83fda5eb47e31d7deec5ae5ca46a1
  Author: Gustavo Romero <gustavo.romero@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M tests/guest-debug/run-test.py
    M tests/guest-debug/test_gdbstub.py

  Log Message:
  -----------
  tests/guest-debug: Support passing arguments to the GDB test script

This commit adds support for passing arguments to the GDB test scripts
so it's possible to parse the args in an "argparse way" in the test
scripts launched by the runner. The arguments should be preceded by --
when passed to the runner. For example, passing "--help" arg to the
GDB_TEST_SCRIPT:

run-test.py [...] --test <GDB_TEST_SCRIPT> -- --help

The test script should not use the argparse module directly but import
arg_parser from test_gdbstub module. arg_parser then can be used just
like the argparse.ArgumentParser class:

from test_gdbstub import arg_parser

p = arg_parser(prog="test-mytest.py", description="My test.")
p.add_argument("--vowel", help="Select vowel",
               required=True, choices=['a','e','i','o','u'])
[...]

The arg_parser allows a smooth and informative exit if, for instance,
the caller of the runner script passes an invalid argument or misses a
required argument by the test script.

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240906143316.657436-4-gustavo.romero@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-9-alex.bennee@linaro.org>


  Commit: f0ef6c35acc0439ef081f7ef18643ba56888ddae
      
https://github.com/qemu/qemu/commit/f0ef6c35acc0439ef081f7ef18643ba56888ddae
  Author: Gustavo Romero <gustavo.romero@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M tests/tcg/aarch64/system/kernel.ld

  Log Message:
  -----------
  tests/tcg/aarch64: Improve linker script organization

Improve kernel.ld linker script organization by using MEMORY command.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
Message-Id: <20240906143316.657436-5-gustavo.romero@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-10-alex.bennee@linaro.org>


  Commit: 06dd94e84336d2c5a86c99049dc9abb4a1d6b2e5
      
https://github.com/qemu/qemu/commit/06dd94e84336d2c5a86c99049dc9abb4a1d6b2e5
  Author: Gustavo Romero <gustavo.romero@linaro.org>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M configure
    M tests/tcg/aarch64/Makefile.softmmu-target
    M tests/tcg/aarch64/Makefile.target
    M tests/tcg/aarch64/gdbstub/test-mte.py
    M tests/tcg/aarch64/system/boot.S
    M tests/tcg/aarch64/system/kernel.ld
    A tests/tcg/aarch64/system/mte.S

  Log Message:
  -----------
  tests/tcg/aarch64: Extend MTE gdbstub tests to system mode

Extend MTE gdbstub tests to also run in system mode (share tests between
user mode and system mode). The tests will only run if a version of GDB
that supports MTE on baremetal is available in the test environment and
if available compiler supports the 'memtag' flag
(-march=armv8.5-a+memtag).

For the tests running in system mode, a page that supports MTE ops. is
necessary. Therefore, an MTE-enabled page is made available (mapped) in
the third 2 MB chunk of the second 1 GB space in the flat mapping set in
boot.S. A new binary, mte.S, is also introduced for the tests. It links
against boot.S and is executed by QEMU in system mode.

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
Message-Id: <20240906143316.657436-6-gustavo.romero@linaro.org>
[AJB: fix stray _]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240910173900.4154726-11-alex.bennee@linaro.org>


  Commit: a4eb31c678400472de0b4915b9154a7c20d8332f
      
https://github.com/qemu/qemu/commit/a4eb31c678400472de0b4915b9154a7c20d8332f
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2024-09-11 (Wed, 11 Sep 2024)

  Changed paths:
    M .gitlab-ci.d/container-cross.yml
    M .gitlab-ci.d/crossbuilds.yml
    M configure
    M docs/devel/testing/main.rst
    M scripts/ci/setup/gitlab-runner.yml
    M target/arm/gdbstub64.c
    R tests/docker/dockerfiles/debian-armel-cross.docker
    M tests/docker/dockerfiles/debian-i686-cross.docker
    M tests/docker/dockerfiles/debian-mipsel-cross.docker
    M tests/guest-debug/run-test.py
    M tests/guest-debug/test_gdbstub.py
    M tests/lcitool/refresh
    M tests/tcg/aarch64/Makefile.softmmu-target
    M tests/tcg/aarch64/Makefile.target
    M tests/tcg/aarch64/gdbstub/test-mte.py
    M tests/tcg/aarch64/system/boot.S
    M tests/tcg/aarch64/system/kernel.ld
    A tests/tcg/aarch64/system/mte.S

  Log Message:
  -----------
  Merge tag 'pull-testing-gdbstub-oct-100924-1' of 
https://gitlab.com/stsquad/qemu into staging

testing and gdbstub updates:

  - remove docker-armel-cross
  - update i686 and mipsel images to bookworm
  - use docker-all-test-cross for mips64le tests
  - fix duplicated line in docs
  - update gitlab-runner ansible script
  - support MTE in gdbstub for system mode

# -----BEGIN PGP SIGNATURE-----
#
# iQEzBAABCgAdFiEEZoWumedRZ7yvyN81+9DbCVqeKkQFAmbgye8ACgkQ+9DbCVqe
# KkTesQf/WSTYAelzJWlEo0EPg5agokephfza4vdmweDujOT8MYPF9qxfsxoiTVA8
# GTtTOod9iqmY/4/EPKIqUtZH38oaX5h9on2FhSssOMy+N4lUADJ+CcHHMSj4BuUt
# jTXDSa9e5aj0m/yqg2PjF8U12Sygf7dKJturGLOWoWR5qa3xpQ2a6c3CkfxO3RQK
# yTBfIZk47iOrVvEX8chsRzpkpiXY6/S5hkZZwcqbXcUMKH2s0po9Yg031vE3yN+g
# kxJ7/mFNl49E/fqYdRahhyBDORlltCglCHsacxxa/4a216wOsNKyV3QLCJMjq8yO
# 3/SPu0p+UouSFcASwTUt5XIo0G0TcA==
# =7W1s
# -----END PGP SIGNATURE-----
# gpg: Signature made Tue 10 Sep 2024 23:36:31 BST
# gpg:                using RSA key 6685AE99E75167BCAFC8DF35FBD0DB095A9E2A44
# gpg: Good signature from "Alex Bennée (Master Work Key) 
<alex.bennee@linaro.org>" [full]
# Primary key fingerprint: 6685 AE99 E751 67BC AFC8  DF35 FBD0 DB09 5A9E 2A44

* tag 'pull-testing-gdbstub-oct-100924-1' of https://gitlab.com/stsquad/qemu:
  tests/tcg/aarch64: Extend MTE gdbstub tests to system mode
  tests/tcg/aarch64: Improve linker script organization
  tests/guest-debug: Support passing arguments to the GDB test script
  gdbstub: Add support for MTE in system mode
  gdbstub: Use specific MMU index when probing MTE addresses
  scripts/ci: update the gitlab-runner playbook
  docs/devel: fix duplicate line
  tests/docker: use debian-all-test-cross for mips64el tests
  tests/docker: update debian i686 and mipsel images to bookworm
  tests/docker: remove debian-armel-cross

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>


Compare: https://github.com/qemu/qemu/compare/b8eada54b2ad...a4eb31c67840

To unsubscribe from these emails, change your notification settings at 
https://github.com/qemu/qemu/settings/notifications
reply via email to
[Prev in Thread] Current Thread [Next in Thread]