[Qemu-commits] [qemu/qemu] cb241d: tests/tcg: Stop using exit() in the gdbstub testcases

[Peter Maydell]

  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: cb241df412bce92440b08443f2f2ffa798e32086
      
https://github.com/qemu/qemu/commit/cb241df412bce92440b08443f2f2ffa798e32086
  Author: Ilya Leoshkevich <iii@linux.ibm.com>
  Date:   2024-11-18 (Mon, 18 Nov 2024)

  Changed paths:
    M tests/guest-debug/test_gdbstub.py
    M tests/tcg/multiarch/gdbstub/interrupt.py
    M tests/tcg/multiarch/gdbstub/prot-none.py
    M tests/tcg/multiarch/gdbstub/test-proc-mappings.py

  Log Message:
  -----------
  tests/tcg: Stop using exit() in the gdbstub testcases

GDB 15 does not like exit() anymore:

    (gdb) python exit(0)
    Python Exception <class 'SystemExit'>: 0
    Error occurred in Python: 0

Use the GDB's own exit command, like it's already done in a couple
places, everywhere. This is the same fix as commit 93a3048dcf45
("tests: Gently exit from GDB when tests complete"), but applied to
more places.

Acked-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
Message-Id: <20241022113939.19989-1-iii@linux.ibm.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>


  Commit: cca4fc6743d84db4383a64548ada1edc3d80c537
      
https://github.com/qemu/qemu/commit/cca4fc6743d84db4383a64548ada1edc3d80c537
  Author: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
  Date:   2024-11-18 (Mon, 18 Nov 2024)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: CC rust/ patches to qemu-rust list

As of Nov 2024 [1], we have a new topical mailing list for Rust
related patches. Add a new MAINTAINERS entry to touch all files under
rust/ subdirectory and additionally add it to previous rust related
entries.

[1] 
https://lore.kernel.org/qemu-devel/CAFEAcA-ZQuyGTtG-vhRTEwpz0L4cpimNxkKix45Yw6gVHXozRQ@mail.gmail.com

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
Message-Id: <20241109-update-maintainers-file-rust-v1-1-f4daba6f782f@linaro.org>
[AJB: tweak commit msg]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>


  Commit: c873a6569ff08317578c9810ca049f2c70e3ad99
      
https://github.com/qemu/qemu/commit/c873a6569ff08317578c9810ca049f2c70e3ad99
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2024-11-18 (Mon, 18 Nov 2024)

  Changed paths:
    M hw/display/virtio-gpu-virgl.c
    M hw/display/virtio-gpu.c
    M include/hw/virtio/virtio-gpu.h

  Log Message:
  -----------
  hw/display: factor out the scanout blob to fb conversion

There are two identical sequences of a code doing the same thing that
raise warnings with Coverity. Before fixing those issues lets factor
out the common code into a helper function we can share.

Reviewed-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
Tested-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20241111230040.68470-2-alex.bennee@linaro.org>


  Commit: d6902d7022ba1405a991c94a99b37259d0a6d3a7
      
https://github.com/qemu/qemu/commit/d6902d7022ba1405a991c94a99b37259d0a6d3a7
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2024-11-18 (Mon, 18 Nov 2024)

  Changed paths:
    M hw/display/virtio-gpu.c
    M include/hw/virtio/virtio-gpu.h

  Log Message:
  -----------
  hw/display: check frame buffer can hold blob

Coverity reports (CID 1564769, 1564770) that we potentially overflow
by doing some 32x32 multiplies for something that ends up in a 64 bit
value. Fix this by first using stride for all lines and casting input
to uint64_t to ensure a 64 bit multiply is used.

Reviewed-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
Tested-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20241111230040.68470-3-alex.bennee@linaro.org>


  Commit: 70e651279541424404788d55d5b3c40846f04215
      
https://github.com/qemu/qemu/commit/70e651279541424404788d55d5b3c40846f04215
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2024-11-19 (Tue, 19 Nov 2024)

  Changed paths:
    M MAINTAINERS
    M tests/guest-debug/test_gdbstub.py
    M tests/tcg/multiarch/gdbstub/interrupt.py
    M tests/tcg/multiarch/gdbstub/prot-none.py
    M tests/tcg/multiarch/gdbstub/test-proc-mappings.py

  Log Message:
  -----------
  Merge tag 'pull-9.2-rc-updates-181124-1' of https://gitlab.com/stsquad/qemu 
into staging

Misc fixes for 9.2

  - fix remaining gdbstub test cases to exit cleanly
  - update MAINTAINERS with qemu-rust mailing list details
  - re-factor virtio-gpu and fix coverity warnings

# -----BEGIN PGP SIGNATURE-----
#
# iQEzBAABCgAdFiEEZoWumedRZ7yvyN81+9DbCVqeKkQFAmc7aQwACgkQ+9DbCVqe
# KkQufwf/XN5dcHxddhUHCLQUNA/5STiO5P10vHOGpknqwT/ZKADAbY8gQsoJovSp
# qxczZPFpHlnGkf+AV3wrj2IcFUWlvlhgTw5njsMTOSSjK0jtI2MZrNn2v+6hmelg
# y8ACB+LHkj34Ood9y5H6augCiKsMC7ad2mpXEMQgcZzpp4KlAutu8nbsYpYlAF40
# 1d2Wdtkrt++jPHQGgp0pPqAxq5WyvR3uSdidUFGfmpAqkCRs0ExosqEgj/ODzdHF
# WrvLy9ISdCjZk+5XOxfadmz1vJ31QcZ6pcmkZRXJAJR4p7EST5BTfS10IBNIipgs
# C25TuT+IH9+It0n/Sohf3NlxTnbTDw==
# =TYAI
# -----END PGP SIGNATURE-----
# gpg: Signature made Mon 18 Nov 2024 16:19:24 GMT
# 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-9.2-rc-updates-181124-1' of https://gitlab.com/stsquad/qemu:
  hw/display: check frame buffer can hold blob
  hw/display: factor out the scanout blob to fb conversion
  MAINTAINERS: CC rust/ patches to qemu-rust list
  tests/tcg: Stop using exit() in the gdbstub testcases

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


Compare: https://github.com/qemu/qemu/compare/af4c4fd128d3...70e651279541

To unsubscribe from these emails, change your notification settings at 
https://github.com/qemu/qemu/settings/notifications