[Qemu-commits] [qemu/qemu] 4d8698: tests/lcitool/refresh: Treat the output of lcitool...

2024-05-18 Thread Richard Henderson via Qemu-commits
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 4d8698e692cca96bbf20505a90e42d10141d2b94
  
https://github.com/qemu/qemu/commit/4d8698e692cca96bbf20505a90e42d10141d2b94
  Author: Thomas Huth 
  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é 
Message-ID: <20240516084059.511463-2-th...@redhat.com>
Reviewed-by: Daniel P. Berrangé 
Signed-off-by: Thomas Huth 


  Commit: 9ebe09e633773bebd2dc52b151fb48c1b9b0542a
  
https://github.com/qemu/qemu/commit/9ebe09e633773bebd2dc52b151fb48c1b9b0542a
  Author: Philippe Mathieu-Daudé 
  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é 
[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é 
Signed-off-by: Thomas Huth 


  Commit: fd77b25bbde962395b93335ade09b6d3127a6537
  
https://github.com/qemu/qemu/commit/fd77b25bbde962395b93335ade09b6d3127a6537
  Author: Thomas Huth 
  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é 
Signed-off-by: Thomas Huth 


  Commit: d2f213cc069d3013b3204993d957a6eff59c220c
  
https://github.com/qemu/qemu/commit/d2f213cc069d3013b3204993d957a6eff59c220c
  Author: Thomas Huth 
  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é 
Signed-off-by: Thomas Huth 


  Commit: e4b6adae7aa7ab440e4dd98de3c1c532f48eda60
  
https://github.com/qemu/qemu/commit/e4b6adae7aa7ab440e4dd98de3c1c532f48eda60
  Author: Thomas Huth 
  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é 
Signed-off-by: Thomas Huth 


  Commit: bebe9603fcb072dcdb7fb22005781b3582a4d701
  
https://github.com/qemu/qemu/commit/bebe9603fcb072dcdb7fb22005781b3582a4d701
  Author: Thomas Huth 
  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 

[Qemu-commits] [qemu/qemu] 4d8698: tests/lcitool/refresh: Treat the output of lcitool...

2024-05-18 Thread Richard Henderson via Qemu-commits
  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: 4d8698e692cca96bbf20505a90e42d10141d2b94
  
https://github.com/qemu/qemu/commit/4d8698e692cca96bbf20505a90e42d10141d2b94
  Author: Thomas Huth 
  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é 
Message-ID: <20240516084059.511463-2-th...@redhat.com>
Reviewed-by: Daniel P. Berrangé 
Signed-off-by: Thomas Huth 


  Commit: 9ebe09e633773bebd2dc52b151fb48c1b9b0542a
  
https://github.com/qemu/qemu/commit/9ebe09e633773bebd2dc52b151fb48c1b9b0542a
  Author: Philippe Mathieu-Daudé 
  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é 
[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é 
Signed-off-by: Thomas Huth 


  Commit: fd77b25bbde962395b93335ade09b6d3127a6537
  
https://github.com/qemu/qemu/commit/fd77b25bbde962395b93335ade09b6d3127a6537
  Author: Thomas Huth 
  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é 
Signed-off-by: Thomas Huth 


  Commit: d2f213cc069d3013b3204993d957a6eff59c220c
  
https://github.com/qemu/qemu/commit/d2f213cc069d3013b3204993d957a6eff59c220c
  Author: Thomas Huth 
  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é 
Signed-off-by: Thomas Huth 


  Commit: e4b6adae7aa7ab440e4dd98de3c1c532f48eda60
  
https://github.com/qemu/qemu/commit/e4b6adae7aa7ab440e4dd98de3c1c532f48eda60
  Author: Thomas Huth 
  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é 
Signed-off-by: Thomas Huth 


  Commit: bebe9603fcb072dcdb7fb22005781b3582a4d701
  
https://github.com/qemu/qemu/commit/bebe9603fcb072dcdb7fb22005781b3582a4d701
  Author: Thomas Huth 
  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