Branch: refs/heads/master Home: https://github.com/qemu/qemu Commit: 4d8698e692cca96bbf20505a90e42d10141d2b94 https://github.com/qemu/qemu/commit/4d8698e692cca96bbf20505a90e42d10141d2b94 Author: Thomas Huth <th...@redhat.com> Date: 2024-05-17 (Fri, 17 May 2024)
Changed paths: M tests/lcitool/refresh Log Message: ----------- tests/lcitool/refresh: Treat the output of lcitool as text, not as bytes In case lcitool fails (e.g. with a python backtrace), this makes the output of lcitool much more readable. Suggested-by: Daniel P. Berrangé <berra...@redhat.com> Message-ID: <20240516084059.511463-2-th...@redhat.com> Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> Signed-off-by: Thomas Huth <th...@redhat.com> Commit: 9ebe09e633773bebd2dc52b151fb48c1b9b0542a https://github.com/qemu/qemu/commit/9ebe09e633773bebd2dc52b151fb48c1b9b0542a Author: Philippe Mathieu-Daudé <phi...@linaro.org> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M tests/lcitool/projects/qemu.yml Log Message: ----------- tests/lcitool: Remove 'xfsprogs' from QEMU QEMU's commit a5730b8bd3 ("block/file-posix: Simplify the XFS_IOC_DIOINFO handling") removed the need for the 'xfsprogs' package. Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> [thuth: Adjusted the patch from the lcitools repo to QEMU's repo] Message-ID: <20240516084059.511463-3-th...@redhat.com> Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> Signed-off-by: Thomas Huth <th...@redhat.com> Commit: fd77b25bbde962395b93335ade09b6d3127a6537 https://github.com/qemu/qemu/commit/fd77b25bbde962395b93335ade09b6d3127a6537 Author: Thomas Huth <th...@redhat.com> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M tests/lcitool/projects/qemu-minimal.yml A tests/lcitool/projects/qemu-win-installer.yml M tests/lcitool/projects/qemu.yml M tests/lcitool/refresh Log Message: ----------- tests/lcitool: Remove g++ from the containers (except for the MinGW one) We don't need C++ for the normal QEMU builds anymore, so installing g++ in each and every container seems to be a waste of time and disk space. The only container that still needs it is the Fedora MinGW container that builds the only remaining C++ code in ./qga/vss-win32/ and we can install it there with an extra project yml file instead. Message-ID: <20240516084059.511463-4-th...@redhat.com> Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> Signed-off-by: Thomas Huth <th...@redhat.com> Commit: d2f213cc069d3013b3204993d957a6eff59c220c https://github.com/qemu/qemu/commit/d2f213cc069d3013b3204993d957a6eff59c220c Author: Thomas Huth <th...@redhat.com> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M tests/lcitool/projects/qemu.yml Log Message: ----------- tests/lcitool/projects/qemu.yml: Sort entries alphabetically again Let's try to keep the entries in alphabetical order here! Message-ID: <20240516084059.511463-5-th...@redhat.com> Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> Signed-off-by: Thomas Huth <th...@redhat.com> Commit: e4b6adae7aa7ab440e4dd98de3c1c532f48eda60 https://github.com/qemu/qemu/commit/e4b6adae7aa7ab440e4dd98de3c1c532f48eda60 Author: Thomas Huth <th...@redhat.com> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M tests/docker/dockerfiles/alpine.docker M tests/docker/dockerfiles/centos9.docker M tests/docker/dockerfiles/debian-amd64-cross.docker M tests/docker/dockerfiles/debian-arm64-cross.docker M tests/docker/dockerfiles/debian-armel-cross.docker M tests/docker/dockerfiles/debian-armhf-cross.docker M tests/docker/dockerfiles/debian-i686-cross.docker M tests/docker/dockerfiles/debian-mips64el-cross.docker M tests/docker/dockerfiles/debian-mipsel-cross.docker M tests/docker/dockerfiles/debian-ppc64el-cross.docker M tests/docker/dockerfiles/debian-riscv64-cross.docker M tests/docker/dockerfiles/debian-s390x-cross.docker M tests/docker/dockerfiles/debian.docker M tests/docker/dockerfiles/fedora-win64-cross.docker M tests/docker/dockerfiles/fedora.docker M tests/docker/dockerfiles/opensuse-leap.docker M tests/docker/dockerfiles/ubuntu2204.docker Log Message: ----------- tests/docker/dockerfiles: Update container files with "lcitool-refresh" Run "make lcitool-refresh" after the previous changes to the lcitool files. This removes the g++ and xfslibs-dev packages from the dockerfiles (except for the fedora-win64-cross dockerfile where we keep the C++ compiler). Message-ID: <20240516084059.511463-6-th...@redhat.com> Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> Signed-off-by: Thomas Huth <th...@redhat.com> Commit: bebe9603fcb072dcdb7fb22005781b3582a4d701 https://github.com/qemu/qemu/commit/bebe9603fcb072dcdb7fb22005781b3582a4d701 Author: Thomas Huth <th...@redhat.com> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M hw/intc/s390_flic.c Log Message: ----------- hw/intc/s390_flic: Fix crash that occurs when saving the machine state adapter_info_so_needed() treats its "opaque" parameter as a S390FLICState, but the function belongs to a VMStateDescription that is attached to a TYPE_VIRTIO_CCW_BUS device. This is currently causing a crash when the user tries to save or migrate the VM state. Fix it by using s390_get_flic() to get the correct device here instead. Reported-by: Marc Hartmayer <mhart...@linux.ibm.com> Fixes: 9d1b0f5bf5 ("s390_flic: add migration-enabled property") Message-ID: <20240517061553.564529-1-th...@redhat.com> Reviewed-by: Cédric Le Goater <c...@redhat.com> Tested-by: Marc Hartmayer <mhart...@linux.ibm.com> Signed-off-by: Thomas Huth <th...@redhat.com> Commit: 3dba3c0b2501e6afa9f722608be6a343465895da https://github.com/qemu/qemu/commit/3dba3c0b2501e6afa9f722608be6a343465895da Author: Cédric Le Goater <c...@redhat.com> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M ui/gtk-egl.c Log Message: ----------- ui/console: Only declare variable fence_fd when CONFIG_GBM is defined This to avoid a build breakage : ../ui/gtk-egl.c: In function ‘gd_egl_draw’: ../ui/gtk-egl.c:73:9: error: unused variable ‘fence_fd’ [-Werror=unused-variable] 73 | int fence_fd; | ^~~~~~~~ Fixes: fa6426805b12 ("ui/console: Use qemu_dmabuf_set_..() helpers instead") Cc: Dongwon Kim <dongwon....@intel.com> Cc: Marc-André Lureau <marcandre.lur...@redhat.com> Signed-off-by: Cédric Le Goater <c...@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org> Message-ID: <20240515100520.574383-1-...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> Commit: 2563be6317fa9b5e661d79581538c704ecb90a1a https://github.com/qemu/qemu/commit/2563be6317fa9b5e661d79581538c704ecb90a1a Author: Gerd Hoffmann <kra...@redhat.com> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M hw/block/pflash_cfi01.c Log Message: ----------- hw/pflash: fix block write start Move the pflash_blk_write_start() call. We need the offset of the first data write, not the offset for the setup (number-of-bytes) write. Without this fix u-boot can do block writes to the first flash block only. While being at it drop a leftover FIXME. Cc: qemu-sta...@nongnu.org Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2343 Fixes: 284a7ee2e290 ("hw/pflash: implement update buffer for block writes") Signed-off-by: Gerd Hoffmann <kra...@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org> Message-ID: <20240516121237.534875-1-kra...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> Commit: 9d7950edb0cdf8f4e5746e220e6e8a9e713bad16 https://github.com/qemu/qemu/commit/9d7950edb0cdf8f4e5746e220e6e8a9e713bad16 Author: Daniel P. Berrangé <berra...@redhat.com> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M hw/core/machine-smp.c M tests/unit/test-smp-parse.c Log Message: ----------- hw/core: allow parameter=1 for SMP topology on any machine This effectively reverts commit 54c4ea8f3ae614054079395842128a856a73dbf9 Author: Zhao Liu <zhao1....@intel.com> Date: Sat Mar 9 00:01:37 2024 +0800 hw/core/machine-smp: Deprecate unsupported "parameter=1" SMP configurations but is not done as a 'git revert' since the part of the changes to the file hw/core/machine-smp.c which add 'has_XXX' checks remain desirable. Furthermore, we have to tweak the subsequently added unit test to account for differing warning message. The rationale for the original deprecation was: "Currently, it was allowed for users to specify the unsupported topology parameter as "1". For example, x86 PC machine doesn't support drawer/book/cluster topology levels, but user could specify "-smp drawers=1,books=1,clusters=1". This is meaningless and confusing, so that the support for this kind of configurations is marked deprecated since 9.0." There are varying POVs on the topic of 'unsupported' topology levels. It is common to say that on a system without hyperthreading, that there is always 1 thread. Likewise when new CPUs introduced a concept of multiple "dies', it was reasonable to say that all historical CPUs before that implicitly had 1 'die'. Likewise for the more recently introduced 'modules' and 'clusters' parameter'. From this POV, it is valid to set 'parameter=1' on the -smp command line for any machine, only a value > 1 is strictly an error condition. It doesn't cause any functional difficulty for QEMU, because internally the QEMU code is itself assuming that all "unsupported" parameters implicitly have a value of '1'. At the libvirt level, we've allowed applications to set 'parameter=1' when configuring a guest, and pass that through to QEMU. Deprecating this creates extra difficulty for because there's no info exposed from QEMU about which machine types "support" which parameters. Thus, libvirt can't know whether it is valid to pass 'parameter=1' for a given machine type, or whether it will trigger deprecation messages. Since there's no apparent functional benefit to deleting this deprecated behaviour from QEMU, and it creates problems for consumers of QEMU, remove this deprecation. Signed-off-by: Daniel P. Berrangé <berra...@redhat.com> Reviewed-by: Zhao Liu <zhao1....@intel.com> Reviewed-by: Ján Tomko <jto...@redhat.com> Message-ID: <20240513123358.612355-2-berra...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> Commit: e68dcbb07923df0886802727edc3b21a10b0d342 https://github.com/qemu/qemu/commit/e68dcbb07923df0886802727edc3b21a10b0d342 Author: Daniel P. Berrangé <berra...@redhat.com> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M tests/unit/test-smp-parse.c Log Message: ----------- tests: add testing of parameter=1 for SMP topology Validate that it is possible to pass 'parameter=1' for any SMP topology parameter, since unsupported parameters are implicitly considered to always have a value of 1. Signed-off-by: Daniel P. Berrangé <berra...@redhat.com> Reviewed-by: Zhao Liu <zhao1....@intel.com> Reviewed-by: Ján Tomko <jto...@redhat.com> Message-ID: <20240513123358.612355-3-berra...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> Commit: 93a3048dcf4565c73f2aa1d751f7197e296f1f1f https://github.com/qemu/qemu/commit/93a3048dcf4565c73f2aa1d751f7197e296f1f1f Author: Gustavo Romero <gustavo.rom...@linaro.org> Date: 2024-05-17 (Fri, 17 May 2024) Changed paths: M tests/guest-debug/test_gdbstub.py Log Message: ----------- tests: Gently exit from GDB when tests complete GDB commit a207f6b3a38 ('Rewrite "python" command exception handling') changed how exit() called from Python scripts loaded by GDB behave, turning it into an exception instead of a generic error code that is returned. This change caused several QEMU tests to crash with the following exception: Python Exception <class 'SystemExit'>: 0 Error occurred in Python: 0 This happens because in tests/guest-debug/test_gdbstub.py exit is called after the tests have completed. This commit fixes it by politely asking GDB to exit via gdb.execute, passing the proper fail_count to be reported to 'make', instead of abruptly calling exit() from the Python script. Signed-off-by: Gustavo Romero <gustavo.rom...@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org> Message-ID: <20240515173132.2462201-4-gustavo.rom...@linaro.org> Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> Commit: 1f952d8cfdbc3caf169777ce416abbf1433f705c https://github.com/qemu/qemu/commit/1f952d8cfdbc3caf169777ce416abbf1433f705c Author: Richard Henderson <richard.hender...@linaro.org> Date: 2024-05-18 (Sat, 18 May 2024) Changed paths: M hw/intc/s390_flic.c M tests/docker/dockerfiles/alpine.docker M tests/docker/dockerfiles/centos9.docker M tests/docker/dockerfiles/debian-amd64-cross.docker M tests/docker/dockerfiles/debian-arm64-cross.docker M tests/docker/dockerfiles/debian-armel-cross.docker M tests/docker/dockerfiles/debian-armhf-cross.docker M tests/docker/dockerfiles/debian-i686-cross.docker M tests/docker/dockerfiles/debian-mips64el-cross.docker M tests/docker/dockerfiles/debian-mipsel-cross.docker M tests/docker/dockerfiles/debian-ppc64el-cross.docker M tests/docker/dockerfiles/debian-riscv64-cross.docker M tests/docker/dockerfiles/debian-s390x-cross.docker M tests/docker/dockerfiles/debian.docker M tests/docker/dockerfiles/fedora-win64-cross.docker M tests/docker/dockerfiles/fedora.docker M tests/docker/dockerfiles/opensuse-leap.docker M tests/docker/dockerfiles/ubuntu2204.docker M tests/lcitool/projects/qemu-minimal.yml A tests/lcitool/projects/qemu-win-installer.yml M tests/lcitool/projects/qemu.yml M tests/lcitool/refresh Log Message: ----------- Merge tag 'pull-request-2024-05-17' of https://gitlab.com/thuth/qemu into staging * Fix s390x crash when doing migration / savevm * Decrease size of CI containers by removing unnecessary packages # -----BEGIN PGP SIGNATURE----- # # iQJFBAABCAAvFiEEJ7iIR+7gJQEY8+q5LtnXdP5wLbUFAmZHTHgRHHRodXRoQHJl # ZGhhdC5jb20ACgkQLtnXdP5wLbXZrxAAsh6zHycGpaQdfcoy3bDmd8gTbuLiME/h # JyJxZ/+GQc+8v8WPwB+HuF7IijtopYCfyO6Vu2y/5wj8i1gHbNulxlS5SjusJp6i # Xxlvuw74xo8Z2oJ6D8Ayk2KHcld5M0m9T77CgP8WcGKmBQU42XWm89fKvviPtn+K # DtLNEpvTlcdEj0uhxhHldHKQnqNryxSHM1MSsmVIKibkQHgG7GBYnw922lZ2x27A # AqSzgzNXAbhmSn75oQfkGUk+vUmlXukfBAHi48BLnAs28sSUue3Su+zw9r8sxhKw # jdvzIB1kyF01AYiKWmhB3voXNduswT9I/cNiQorgOBEJ4lKEzrhsTI92GpvNG3gR # J0CRBUmnGC2k/4GRa+GhFEpFn9FyWeOjPj2oGv03LO4AgTWzi1zNcO++OIWsk0Ge # rO2n2PEEz8RaI/49CTLGi3Eu0Rh0yZnrgZRjcji5ZZ3omQ/OrwXGyr3FMDNFNuXs # vWr9p4K1vz2P/L+RC+TCM0U46gykQuBPseRsdVvbJxAoNP4HwmdE9jDy1Wl1mG1u # Iac63/+srr/871UFzp7ft8ukKTVKy4elQJ78tDCsmRhkVNjLFWwf4SNY6RaneeYM # IbsLcjWpZPl4I9KR6Of5p+aAHAUg6xKIIaIR01fMyQL44ELomfbpH2rKp4tObJHj # WIEKnOWuclo= # =vgc1 # -----END PGP SIGNATURE----- # gpg: Signature made Fri 17 May 2024 02:24:24 PM CEST # gpg: using RSA key 27B88847EEE0250118F3EAB92ED9D774FE702DB5 # gpg: issuer "th...@redhat.com" # gpg: Good signature from "Thomas Huth <th.h...@gmx.de>" [full] # gpg: aka "Thomas Huth <th...@redhat.com>" [full] # gpg: aka "Thomas Huth <th.h...@posteo.de>" [unknown] # gpg: aka "Thomas Huth <h...@tuxfamily.org>" [full] * tag 'pull-request-2024-05-17' of https://gitlab.com/thuth/qemu: hw/intc/s390_flic: Fix crash that occurs when saving the machine state tests/docker/dockerfiles: Update container files with "lcitool-refresh" tests/lcitool/projects/qemu.yml: Sort entries alphabetically again tests/lcitool: Remove g++ from the containers (except for the MinGW one) tests/lcitool: Remove 'xfsprogs' from QEMU tests/lcitool/refresh: Treat the output of lcitool as text, not as bytes Signed-off-by: Richard Henderson <richard.hender...@linaro.org> Commit: 01782d6b294f95bcde334386f0aaac593cd28c0d https://github.com/qemu/qemu/commit/01782d6b294f95bcde334386f0aaac593cd28c0d Author: Richard Henderson <richard.hender...@linaro.org> Date: 2024-05-18 (Sat, 18 May 2024) Changed paths: M hw/block/pflash_cfi01.c M hw/core/machine-smp.c M tests/guest-debug/test_gdbstub.py M tests/unit/test-smp-parse.c M ui/gtk-egl.c Log Message: ----------- Merge tag 'hw-misc-20240517' of https://github.com/philmd/qemu into staging Misc HW patches queue - Fix build when GBM buffer management library is detected (Cédric) - Fix PFlash block write (Gerd) - Allow 'parameter=1' for SMP topology on any machine (Daniel) - Allow guest-debug tests to run with recent GDB (Gustavo) # -----BEGIN PGP SIGNATURE----- # # iQIzBAABCAAdFiEE+qvnXhKRciHc/Wuy4+MsLN6twN4FAmZHcOsACgkQ4+MsLN6t # wN4CqxAA15Ow9ubxipORpM+XJgJ5isyPjD1s/6bR6lj7joBS6CYQbMaaskXuDQK8 # FpeoWw2DI2Fh/61NcUMAk7XBFF59DLrtngDhfLZJYdwBh0S8RFs1wp6sKyaBA9K6 # wDy39plxt/abKGzj3EcJUGDvhBLPJNnqy5OF9fZtWGrQg+A1i9uLMu/ac6srfX+K # zau/CxQaHYRYLYFmRcQCOhFVAtp2TQHw14CiiLYMCxF3GvUCN0xmtg8lzj9/y4ke # Yt0VN6jC3opfmQuDtPJNNkp8beaHbwMARFmXepDVB2cHp8DY5Gm4Ij2WiR0K985G # fqDknHEXDPI+RislV9+EN3p2c05m7ihPKLiDLYCulD4TIRDz+eUf71Onus9uecj9 # zCDdPYjU1ly9pyt7EVG2Bla9D/F51ZvbrzJQrHbvqhxWuZGOPSzHdpSsHZBIOXk6 # OhxTtUPeWDYW5K+wdNpxYPy5dqIR3jSEbDwLh2Wts2iPKxCGC8ly6CbZJPgA5lQE # hwYbiSKNcxAMV3V9qBfKLRSGadnnfPwG/zrGOHBni9ejz+m7foA13mJ4H6VFBn7Q # GGe9f00MCKcWTTlzRty1oIzAKcpupCOanX0MpVNcTYUqVtODhlQpDdH63ZVuiyRU # kux9xz71I+mwkjQiTHTki1qcAbLNj9+jgwbcc74Zz1BngIauqtc= # =Octv # -----END PGP SIGNATURE----- # gpg: Signature made Fri 17 May 2024 04:59:55 PM CEST # gpg: using RSA key FAABE75E12917221DCFD6BB2E3E32C2CDEADC0DE # gpg: Good signature from "Philippe Mathieu-Daudé (F4BUG) <f4...@amsat.org>" [full] * tag 'hw-misc-20240517' of https://github.com/philmd/qemu: tests: Gently exit from GDB when tests complete tests: add testing of parameter=1 for SMP topology hw/core: allow parameter=1 for SMP topology on any machine hw/pflash: fix block write start ui/console: Only declare variable fence_fd when CONFIG_GBM is defined Signed-off-by: Richard Henderson <richard.hender...@linaro.org> Compare: https://github.com/qemu/qemu/compare/85ef20f1673f...01782d6b294f To unsubscribe from these emails, change your notification settings at https://github.com/qemu/qemu/settings/notifications