qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 03d9e4: block/blkio: Add virtio-blk-vfio-pci


From: Paolo Bonzini
Subject: [Qemu-commits] [qemu/qemu] 03d9e4: block/blkio: Add virtio-blk-vfio-pci BlockDriver
Date: Tue, 01 Nov 2022 10:50:22 -0700

  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: 03d9e4c0dba9d1b5d0c532ac3518415466ebdf8f
      
https://github.com/qemu/qemu/commit/03d9e4c0dba9d1b5d0c532ac3518415466ebdf8f
  Author: Alberto Faria <afaria@redhat.com>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M block/blkio.c
    M qapi/block-core.json

  Log Message:
  -----------
  block/blkio: Add virtio-blk-vfio-pci BlockDriver

libblkio 1.1.0 [1] introduces a virtio-blk-vfio-pci driver, which
accesses a virtio-blk PCI device using VFIO. Add a corresponding
BlockDriver.

[1] https://gitlab.com/libblkio/libblkio/-/tree/v1.1.0

Signed-off-by: Alberto Faria <afaria@redhat.com>
Message-id: 20221028131635.710267-1-afaria@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>


  Commit: 4c8f4fda0504564580f5c0a37e2d4b32ff17d2a1
      
https://github.com/qemu/qemu/commit/4c8f4fda0504564580f5c0a37e2d4b32ff17d2a1
  Author: Alberto Faria <afaria@redhat.com>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M block/blkio.c

  Log Message:
  -----------
  block/blkio: Tolerate device size changes

Some libblkio drivers may be able to work with regular files (e.g.,
io_uring) or otherwise resizable devices. Conservatively set
BlockDriver::has_variable_length to true to ensure bdrv_nb_sectors()
always gives up-to-date results.

Also implement BlockDriver::bdrv_co_truncate for the case where no
preallocation is needed and the device already has a size compatible
with what was requested.

Signed-off-by: Alberto Faria <afaria@redhat.com>
Message-id: 20221029122031.975273-1-afaria@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>


  Commit: 6c32fc0df9cd901add75618c831fb26a9eb742cb
      
https://github.com/qemu/qemu/commit/6c32fc0df9cd901add75618c831fb26a9eb742cb
  Author: Alberto Faria <afaria@redhat.com>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M block/blkio.c
    M qapi/block-core.json

  Log Message:
  -----------
  block/blkio: Make driver nvme-io_uring take a "path" instead of a "filename"

The nvme-io_uring driver expects a character special file such as
/dev/ng0n1. Follow the convention of having a "filename" option when a
regular file is expected, and a "path" option otherwise.

This makes io_uring the only libblkio-based driver with a "filename"
option, as it accepts a regular file (even though it can also take a
block special file).

Signed-off-by: Alberto Faria <afaria@redhat.com>
Message-id: 20221028233854.839933-1-afaria@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>


  Commit: 6532426aa056673b1a20e4c1efa3fc26f0567077
      
https://github.com/qemu/qemu/commit/6532426aa056673b1a20e4c1efa3fc26f0567077
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/lcitool/refresh

  Log Message:
  -----------
  tests/lcitool: Rename non-Debian specific helper

This helper is not Debian specific, rename it to cross_build().

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Acked-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220929114231.583801-10-alex.bennee@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221028095659.48734-2-philmd@linaro.org>


  Commit: 9e243b7669054574e5cfbc2a7282dbb62827627b
      
https://github.com/qemu/qemu/commit/9e243b7669054574e5cfbc2a7282dbb62827627b
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/docker/dockerfiles/fedora-win32-cross.docker
    M tests/docker/dockerfiles/fedora-win64-cross.docker
    M tests/lcitool/refresh

  Log Message:
  -----------
  tests/docker: update fedora-win[32|64]-cross with lcitool

Convert another two dockerfiles to lcitool and update.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Acked-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220929114231.583801-10-alex.bennee@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221028095659.48734-4-philmd@linaro.org>


  Commit: 5b1229fa2dfde161ee322faaaeb35a99765c8712
      
https://github.com/qemu/qemu/commit/5b1229fa2dfde161ee322faaaeb35a99765c8712
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/lcitool/libvirt-ci

  Log Message:
  -----------
  tests/lcitool: Refresh to latest libvirt-ci module

We need an updated lcitool for this to deal with the weirdness
of a 32bit nsis tool for both 32 and 64 bit builds.

Acked-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220929114231.583801-10-alex.bennee@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221028095659.48734-3-philmd@linaro.org>
[AJB: no longer triggers whitespace changes due to rebase]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>


  Commit: b1314192524a8a43f3d4fb1cae8c447a2b255308
      
https://github.com/qemu/qemu/commit/b1314192524a8a43f3d4fb1cae8c447a2b255308
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/docker/test-mingw

  Log Message:
  -----------
  tests/docker: update test-mingw to run single build

While the test-build test happily run for mingw the test-mingw case
runs more of the packaging inline with what our CI does. It however
fails if we don't find both compilers and expects to be run on a
docker image with both.

Remove that distinction and make it work more like the other build
test scripts.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-3-alex.bennee@linaro.org>


  Commit: ed77c37ac8500d750d5858d02dc78b761dc998be
      
https://github.com/qemu/qemu/commit/ed77c37ac8500d750d5858d02dc78b761dc998be
  Author: Anton Johansson <anjo@rev.ng>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/docker/dockerfiles/debian-all-test-cross.docker

  Log Message:
  -----------
  tests/docker: Add flex/bison to `debian-all-test`

Adds flex/bison to the debian-all-test-cross container which was missed
in the previous CI patch. These dependencies are required by the
idef-parser patchset for target/hexagon.

Signed-off-by: Anton Johansson <anjo@rev.ng>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20221011173229.57909-1-anjo@rev.ng>
Message-Id: <20221027183637.2772968-4-alex.bennee@linaro.org>


  Commit: c3b570b5a9a24d25ab522def7c928c6a3be3d842
      
https://github.com/qemu/qemu/commit/c3b570b5a9a24d25ab522def7c928c6a3be3d842
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: don't enable cross compilers unless in target_list

This avoids the unfortunate effect of always builds the pc-bios blobs
for targets the user isn't interested in.

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-5-alex.bennee@linaro.org>


  Commit: 977cccb8451ecf0fd60388aec95dc9c1ab35afc8
      
https://github.com/qemu/qemu/commit/977cccb8451ecf0fd60388aec95dc9c1ab35afc8
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: fix the --enable-static --disable-pie case

The previous tweak was incomplete as it missed a leg.

Fixes: abafb64b6d (configure: explicitly set cflags for --disable-pie)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-6-alex.bennee@linaro.org>


  Commit: e6025635db6a6fb406a6910fc1b45ea597ea9f9e
      
https://github.com/qemu/qemu/commit/e6025635db6a6fb406a6910fc1b45ea597ea9f9e
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/avocado/boot_linux.py

  Log Message:
  -----------
  tests/avocado: extend the timeout for x86_64 tcg tests

These are timing out on gitlab.

Acked-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-7-alex.bennee@linaro.org>


  Commit: b4c82b1b4d030b0b3edbab7173b6888aa12e7e0a
      
https://github.com/qemu/qemu/commit/b4c82b1b4d030b0b3edbab7173b6888aa12e7e0a
  Author: Anton Johansson <anjo@rev.ng>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/docker/dockerfiles/debian-hexagon-cross.docker

  Log Message:
  -----------
  tests/docker: Add flex/bison to `debian-hexagon-cross`

debian-hexagon-cross contains two images, one to build the toolchain
used for building the Hexagon tests themselves, and one image to build
QEMU and run the tests.

This commit adds flex/bison to the final image that builds QEMU so that
it can also build idef-parser.

Note: This container is not built by the CI and needs to be rebuilt and
updated manually.

Signed-off-by: Anton Johansson <anjo@rev.ng>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20221014223642.147845-1-anjo@rev.ng>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-8-alex.bennee@linaro.org>


  Commit: 991e9051238e15207ca3fbfc630fd98f0e2b86cd
      
https://github.com/qemu/qemu/commit/991e9051238e15207ca3fbfc630fd98f0e2b86cd
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/tcg/nios2/Makefile.softmmu-target

  Log Message:
  -----------
  tests/tcg: use regular semihosting for nios2-softmmu

The nios2 code now plugs into the common semihosting code so we can
use the same redirect invocation as the other boards. There is however
a bug raised for the fact the tests don't seem to be completing
properly and silently passing anyway:

  https://gitlab.com/qemu-project/qemu/-/issues/1258

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


  Commit: 25916dd65d7e72f7bb5b9727bb346a9d17993072
      
https://github.com/qemu/qemu/commit/25916dd65d7e72f7bb5b9727bb346a9d17993072
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/tcg/nios2/10m50-ghrd.ld

  Log Message:
  -----------
  tests/tcg/nios2: Tweak 10m50-ghrd.ld

More closely follow the default linker script for nios2.
This magically fixes a problem resolving .got relocs from
the toolchain's libgcc.a.

Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1258
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221024035341.2971123-1-richard.henderson@linaro.org>
Message-Id: <20221027183637.2772968-10-alex.bennee@linaro.org>


  Commit: 3878d0c7d7de9fe201513b8ee31e38e53361a97d
      
https://github.com/qemu/qemu/commit/3878d0c7d7de9fe201513b8ee31e38e53361a97d
  Author: Bin Meng <bin.meng@windriver.com>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M semihosting/arm-compat-semi.c

  Log Message:
  -----------
  semihosting/arm-compat-semi: Avoid using hardcoded /tmp

Use g_get_tmp_dir() to get the directory to use for temporary files.

Signed-off-by: Bin Meng <bin.meng@windriver.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20221006151927.2079583-2-bmeng.cn@gmail.com>
Message-Id: <20221027183637.2772968-11-alex.bennee@linaro.org>


  Commit: eb6b2edf8e1a5845c81f3c4da905f1f8bbc0dec3
      
https://github.com/qemu/qemu/commit/eb6b2edf8e1a5845c81f3c4da905f1f8bbc0dec3
  Author: Bin Meng <bin.meng@windriver.com>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tcg/tcg.c

  Log Message:
  -----------
  tcg: Avoid using hardcoded /tmp

Use g_get_tmp_dir() to get the directory to use for temporary files.

Signed-off-by: Bin Meng <bin.meng@windriver.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221006151927.2079583-3-bmeng.cn@gmail.com>
Message-Id: <20221027183637.2772968-12-alex.bennee@linaro.org>


  Commit: c2632994e93eee2f165113f1c78775b113f99926
      
https://github.com/qemu/qemu/commit/c2632994e93eee2f165113f1c78775b113f99926
  Author: Bin Meng <bin.meng@windriver.com>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M block/vvfat.c

  Log Message:
  -----------
  block/vvfat: Unify the mkdir() call

There is a difference in the mkdir() call for win32 and non-win32
platforms, and currently is handled in the codes with #ifdefs.

glib provides a portable g_mkdir() API and we can use it to unify
the codes without #ifdefs.

Signed-off-by: Bin Meng <bin.meng@windriver.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221006151927.2079583-6-bmeng.cn@gmail.com>
Message-Id: <20221027183637.2772968-14-alex.bennee@linaro.org>


  Commit: 34b55848a15bca120d9b9381881c40b045409ee9
      
https://github.com/qemu/qemu/commit/34b55848a15bca120d9b9381881c40b045409ee9
  Author: Bin Meng <bin.meng@windriver.com>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M hw/usb/dev-mtp.c

  Log Message:
  -----------
  hw/usb: dev-mtp: Use g_mkdir()

Use g_mkdir() to create a directory on all platforms.

Signed-off-by: Bin Meng <bin.meng@windriver.com>
Acked-by: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221006151927.2079583-8-bmeng.cn@gmail.com>
Message-Id: <20221027183637.2772968-15-alex.bennee@linaro.org>


  Commit: 48fad83ff49bd47368223cf1121351f51cf3565f
      
https://github.com/qemu/qemu/commit/48fad83ff49bd47368223cf1121351f51cf3565f
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: add entries for the key build bits

Changes to the build files are a bit special in that they usually go
through other maintainer trees. However considering the build system
is the root of everything a developer is likely to do we should at
least set it out in MAINTAINERS.

I'm going to nominate Paolo for meson stuff given the conversion was
his passion project. I'm happy to cast an eye over configure stuff
considering a lot of the cross compile logic is in there anyway.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Acked-by: Thomas Huth <thuth@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20221027183637.2772968-16-alex.bennee@linaro.org>


  Commit: 97cfba13996067a572db168214e3ccbc22f7adaa
      
https://github.com/qemu/qemu/commit/97cfba13996067a572db168214e3ccbc22f7adaa
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: add features_to_c.sh to gdbstub files

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: <20221027183637.2772968-17-alex.bennee@linaro.org>


  Commit: 162f916453fb9e2d1d1c45cd081a8f8c66f29b9b
      
https://github.com/qemu/qemu/commit/162f916453fb9e2d1d1c45cd081a8f8c66f29b9b
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: fix-up for check-tcg Makefile changes

Fixes: fc76c56d3f ("tests/tcg: cleanup Makefile inclusions")
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-18-alex.bennee@linaro.org>


  Commit: 5104b73824670316b1ae1736054e4ba8ab77f345
      
https://github.com/qemu/qemu/commit/5104b73824670316b1ae1736054e4ba8ab77f345
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/avocado/info_usernet.py
    M tests/avocado/vnc.py

  Log Message:
  -----------
  tests/avocado: set -machine none for userfwd and vnc tests

These are exercising core QEMU features and don't actually run code.
Not specifying a machine will fail when avocado chooses the native
arch binary to run. Be explicit.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-19-alex.bennee@linaro.org>


  Commit: cc45d25c65cf69b54fda0794ef74f53daaa272c2
      
https://github.com/qemu/qemu/commit/cc45d25c65cf69b54fda0794ef74f53daaa272c2
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/avocado/boot_linux_console.py

  Log Message:
  -----------
  tests/avocado: raspi2_initrd: Wait for guest shutdown message before stopping

The avocado test
 tests/avocado/boot_linux_console.py:BootLinuxConsole.test_arm_raspi2_initrd
finishes with

    exec_command(self, 'halt')
    # Wait for VM to shut down gracefully
    self.vm.wait()

In theory this should be fine. In practice it runs into two bugs:

 * when the test calls self.vm.wait() Avocado closes the socket
   connection to the guest serial console immediately, so the
   avocado logs don't have the last part of the guest output:
   https://gitlab.com/qemu-project/qemu/-/issues/1265
 * when the socket is closed, a bug in the QEMU socket chardev
   means that it loses any data that the guest UART has not
   yet consumed. This means that the guest doesn't always read
   the full 'halt' command string, so the test intermittently
   fails with a timeout:
   https://gitlab.com/qemu-project/qemu/-/issues/1264

Work around both of these by waiting for the guest to print the
string that means it has completed the shutdown process.  This fixes
a very long standing intermittent failure in this test.

Resolves: https://gitlab.com/qemu-project/qemu/-/issues/636
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20221020102012.3015662-1-peter.maydell@linaro.org>
Message-Id: <20221027183637.2772968-20-alex.bennee@linaro.org>


  Commit: 500f73b1aa24b10278a6bf496ad3263b24964388
      
https://github.com/qemu/qemu/commit/500f73b1aa24b10278a6bf496ad3263b24964388
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/avocado/boot_linux_console.py

  Log Message:
  -----------
  tests/avocado: disable sh4 rd2 tests on Gitlab

Running repeated invocations on a number of test boxes show a fairly
high error rate:

  $ retry.py -n 100 -c -- ./tests/venv/bin/avocado run 
tests/avocado/boot_linux_console.py:BootLinuxConsole.test_sh4_r2d
  retry.py called with ['./tests/venv/bin/avocado', 'run', 
'tests/avocado/boot_linux_console.py:BootLinuxConsole.test_sh4_r2d']
  Results:
  Run, Ret, Pass/Fail, Time, Total Pass, Total Run
  ...
  Results summary:
  0: 94 times (94.00%), avg time 2.254 (0.00 varience/0.04 deviation)
  1: 3 times (3.00%), avg time 1.837 (0.02 varience/0.14 deviation)
  8: 3 times (3.00%), avg time 91.288 (0.02 varience/0.15 deviation)

Examining the logs they fall into various categories of un-handled
unaligned access by user space and unexpected FPU usage by the kernel
which ultimately lead to the failure to reach the login prompt. This
could be bugs in the translator that only get hit occasionally or just
a flaky kernel - its hard to tell. To avoid these failures gating CI
lets skip on GitLab.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Thomas Huth <thuth@redhat.com>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Cc: Magnus Damm <magnus.damm@gmail.com>
Message-Id: <20221027183637.2772968-21-alex.bennee@linaro.org>


  Commit: bc483a91d5a2182387e82913a7e22c86b6b1c85c
      
https://github.com/qemu/qemu/commit/bc483a91d5a2182387e82913a7e22c86b6b1c85c
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/tcg/sh4/Makefile.target

  Log Message:
  -----------
  tests/tcg: re-enable linux-test for sh4

This test was marked as broken due to bug #704 which was fixed by
aee14c77f4 (linux-user: Rewrite do_getdents, do_getdents64). Local
testing shows this is solid now so lets re-enable the test.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Message-Id: <20221027183637.2772968-22-alex.bennee@linaro.org>


  Commit: 839866fcbd4f2ce554f3c16bf68a520ebc2da310
      
https://github.com/qemu/qemu/commit/839866fcbd4f2ce554f3c16bf68a520ebc2da310
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/tcg/sh4/Makefile.target

  Log Message:
  -----------
  tests/tcg: re-enable threadcount for sh4

This test was marked as broken due to bug #856 which was fixed by
ab419fd8a0 (target/sh4: Fix TB_FLAG_UNALIGN). Local testing shows this
is solid now so lets re-enable the test.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Message-Id: <20221027183637.2772968-23-alex.bennee@linaro.org>


  Commit: efe7c4f08d543cd4181c672bedba5575390db2a8
      
https://github.com/qemu/qemu/commit/efe7c4f08d543cd4181c672bedba5575390db2a8
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M target/s390x/tcg/translate.c

  Log Message:
  -----------
  target/s390x: don't use ld_code2 to probe next pc

This isn't an translator picking up an instruction so we shouldn't use
the translator_lduw function which has side effects for plugins.

Reviewed-by: David Hildenbrand <david@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Ilya Leoshkevich <iii@linux.ibm.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-24-alex.bennee@linaro.org>


  Commit: 621aab6c7dffaa37a9d9b89d246894e516eb3c46
      
https://github.com/qemu/qemu/commit/621aab6c7dffaa37a9d9b89d246894e516eb3c46
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M target/s390x/tcg/translate.c

  Log Message:
  -----------
  target/s390x: don't probe next pc for EXecuted insns

We have finished the TB anyway so we can shortcut the other tests by
checking dc->ex_value first.

Reviewed-by: David Hildenbrand <david@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Ilya Leoshkevich <iii@linux.ibm.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-25-alex.bennee@linaro.org>


  Commit: 9fa97e04ae5677cc383b674f8efbf17950b07622
      
https://github.com/qemu/qemu/commit/9fa97e04ae5677cc383b674f8efbf17950b07622
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M include/exec/translator.h
    M target/s390x/tcg/translate.c

  Log Message:
  -----------
  target/s390x: fake instruction loading when handling 'ex'

The s390x EXecute instruction is a bit weird as we synthesis the
executed instruction from what we have stored in memory. This missed
the plugin instrumentation.

Work around this with a special helper to inform the rest of the
translator about the instruction so things stay consistent.

Reviewed-by: David Hildenbrand <david@redhat.com>
Acked-by: Ilya Leoshkevich <iii@linux.ibm.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20221027183637.2772968-26-alex.bennee@linaro.org>


  Commit: d1d94d968d30ce57674b7e3f2752657e1cd2e4b6
      
https://github.com/qemu/qemu/commit/d1d94d968d30ce57674b7e3f2752657e1cd2e4b6
  Author: Paolo Bonzini <pbonzini@redhat.com>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M configure

  Log Message:
  -----------
  tests/tcg: include CONFIG_PLUGIN in config-host.mak

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20221013131304.623740-1-pbonzini@redhat.com>
Message-Id: <20221027183637.2772968-27-alex.bennee@linaro.org>


  Commit: a4cc0b7dd7dc27f6afb5d4e0338d953e36319c65
      
https://github.com/qemu/qemu/commit/a4cc0b7dd7dc27f6afb5d4e0338d953e36319c65
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M contrib/plugins/Makefile

  Log Message:
  -----------
  contrib/plugins: enable debug on CONFIG_DEBUG_TCG

We used to rely on QEMU_CFLAGS to expose the debug flags but now this
is synthesised by meson and only available to the main build. Add our
own flags if we detect the build has been enabled with
CONFIG_DEBUG_TCG (which is the default for --enable-debug anyway).

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20221027183637.2772968-28-alex.bennee@linaro.org>


  Commit: 14fd492b89b512735ac3abf8f5300db865129378
      
https://github.com/qemu/qemu/commit/14fd492b89b512735ac3abf8f5300db865129378
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M contrib/plugins/execlog.c

  Log Message:
  -----------
  contrib/plugins: protect execlog's last_exec expansion

We originally naively treated expansion as safe because we expected
each new CPU/thread to appear in order. However the -M raspi2 model
triggered a case where a new high cpu_index thread started executing
just before a smaller one.

Clean this up by converting the GArray into the simpler GPtrArray and
then holding a lock for the expansion.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Alexandre Iooss <erdnaxe@crans.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20221027183637.2772968-29-alex.bennee@linaro.org>


  Commit: 68406d10859385c88da73d0106254a7f47e6652e
      
https://github.com/qemu/qemu/commit/68406d10859385c88da73d0106254a7f47e6652e
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/unit/test-io-channel-command.c

  Log Message:
  -----------
  tests/unit: cleanups for test-io-channel-command

This test is hanging under heavy load when the two socats race while
trying to create the socket. I've tried various approaches to avoid
the race but it seems "creat=0" won't stop socat trying to create a
pipe if it executes first. In the end I just use a small sleep which
seems to be reliable enough on the load situations I've tried.

While I was there I also properly created a tmpdir for the socket to
live in which is cleaned up at the end of the test.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Cc: Thomas Huth <thuth@redhat.com>
Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
Cc: Juan Quintela <quintela@redhat.com>
Message-Id: <20221027183637.2772968-30-alex.bennee@linaro.org>


  Commit: 339bf0c071eff5e6ff1d9ddb3ad5cd02e4cd9ca3
      
https://github.com/qemu/qemu/commit/339bf0c071eff5e6ff1d9ddb3ad5cd02e4cd9ca3
  Author: Ilya Leoshkevich <iii@linux.ibm.com>
  Date:   2022-10-31 (Mon, 31 Oct 2022)

  Changed paths:
    M tests/vm/basevm.py

  Log Message:
  -----------
  tests/vm: use -o IdentitiesOnly=yes for ssh

When one has a lot of keys in ~/.ssh directory, the ssh command will
try all of them before the one specified on the command line, and this
may cause the remote ssh server to reject the connection due to too
many failed authentication attempts.

Fix by adding -o IdentitiesOnly=yes, which makes the ssh client
consider only the keys specified on the command line.

Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20221027113026.2280863-1-iii@linux.ibm.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221027183637.2772968-31-alex.bennee@linaro.org>


  Commit: 0d37413c639d8f467fc92c86a1ed54ecdba5220d
      
https://github.com/qemu/qemu/commit/0d37413c639d8f467fc92c86a1ed54ecdba5220d
  Author: Stefan Hajnoczi <stefanha@redhat.com>
  Date:   2022-11-01 (Tue, 01 Nov 2022)

  Changed paths:
    M MAINTAINERS
    M block/vvfat.c
    M configure
    M contrib/plugins/Makefile
    M contrib/plugins/execlog.c
    M hw/usb/dev-mtp.c
    M include/exec/translator.h
    M semihosting/arm-compat-semi.c
    M target/s390x/tcg/translate.c
    M tcg/tcg.c
    M tests/avocado/boot_linux.py
    M tests/avocado/boot_linux_console.py
    M tests/avocado/info_usernet.py
    M tests/avocado/vnc.py
    M tests/docker/dockerfiles/debian-all-test-cross.docker
    M tests/docker/dockerfiles/debian-hexagon-cross.docker
    M tests/docker/dockerfiles/fedora-win32-cross.docker
    M tests/docker/dockerfiles/fedora-win64-cross.docker
    M tests/docker/test-mingw
    M tests/lcitool/libvirt-ci
    M tests/lcitool/refresh
    M tests/tcg/nios2/10m50-ghrd.ld
    M tests/tcg/nios2/Makefile.softmmu-target
    M tests/tcg/sh4/Makefile.target
    M tests/unit/test-io-channel-command.c
    M tests/vm/basevm.py

  Log Message:
  -----------
  Merge tag 'pull-testing-for-7.2-011122-3' of https://github.com/stsquad/qemu 
into staging

testing and plugin updates for 7.2:

  - cleanup win32/64 docker files
  - update test-mingw test
  - add flex/bison to debian-all-test
  - handle --enable-static/--disable-pie in config
  - extend timeouts on x86_64 avocado tests
  - add flex/bison to debian-hexagon-cross
  - use regular semihosting for nios2 check-tcg
  - fix obscure linker error to nios2 softmmu tests
  - various windows portability fixes for tests
  - clean-up of MAINTAINERS
  - use -machine none when appropriate in avocado
  - make raspi2_initrd test detect shutdown
  - disable sh4 rd2 tests on gitlab
  - re-enable threadcount/linux-test for sh4
  - clean-up s390x handling of "ex" instruction
  - better handle new CPUs in execlog plugin
  - pass CONFIG_DEBUG_TCG to plugin builds
  - try and avoid races in test-io-channel-command
  - speed up ssh key checking for tests/vm

# -----BEGIN PGP SIGNATURE-----
#
# iQEzBAABCgAdFiEEZoWumedRZ7yvyN81+9DbCVqeKkQFAmNhI/MACgkQ+9DbCVqe
# KkSFXggAg0HIpBDcNz0V5Mh5p69F14pwbDSygKqGDFBebdOHeL7f+WCvQPUGEWxp
# 814zjvRY3SC4Mo4mtzguRvNu0styaUpemvRw5FDYK48GpEjg2eVxTnAFD4nr7ud0
# dhw3iaHP+RjA6s3EpPUqQ5nlZEgFJ+Tvkckk3wKSpksBYA4tJra6Uey5kpZ27x0T
# KOzB2P6w+9B/B11n/aeSxvRPZdnXt2MyfS/3pwwfoFYioEyaEQ3Ie6ooachtdSL3
# PEvnJVK0VVYbZQwBXJlycNLlK/D++s4AEwmnZ5GmvDFuXlkRO9YMy9Wa5TKJl7gz
# 76Aw1KHsE03SyAPvH4bE7eGkIwhJOQ==
# =6hXE
# -----END PGP SIGNATURE-----
# gpg: Signature made Tue 01 Nov 2022 09:49:39 EDT
# gpg:                using RSA key 6685AE99E75167BCAFC8DF35FBD0DB095A9E2A44
# gpg: Good signature from "Alex Bennée (Master Work Key) 
<alex.bennee@linaro.org>" [unknown]
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg:          There is no indication that the signature belongs to the owner.
# Primary key fingerprint: 6685 AE99 E751 67BC AFC8  DF35 FBD0 DB09 5A9E 2A44

* tag 'pull-testing-for-7.2-011122-3' of https://github.com/stsquad/qemu: (31 
commits)
  tests/vm: use -o IdentitiesOnly=yes for ssh
  tests/unit: cleanups for test-io-channel-command
  contrib/plugins: protect execlog's last_exec expansion
  contrib/plugins: enable debug on CONFIG_DEBUG_TCG
  tests/tcg: include CONFIG_PLUGIN in config-host.mak
  target/s390x: fake instruction loading when handling 'ex'
  target/s390x: don't probe next pc for EXecuted insns
  target/s390x: don't use ld_code2 to probe next pc
  tests/tcg: re-enable threadcount for sh4
  tests/tcg: re-enable linux-test for sh4
  tests/avocado: disable sh4 rd2 tests on Gitlab
  tests/avocado: raspi2_initrd: Wait for guest shutdown message before stopping
  tests/avocado: set -machine none for userfwd and vnc tests
  MAINTAINERS: fix-up for check-tcg Makefile changes
  MAINTAINERS: add features_to_c.sh to gdbstub files
  MAINTAINERS: add entries for the key build bits
  hw/usb: dev-mtp: Use g_mkdir()
  block/vvfat: Unify the mkdir() call
  tcg: Avoid using hardcoded /tmp
  semihosting/arm-compat-semi: Avoid using hardcoded /tmp
  ...

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>


  Commit: a11f65ec1b8adcb012b89c92819cbda4dc25aaf1
      
https://github.com/qemu/qemu/commit/a11f65ec1b8adcb012b89c92819cbda4dc25aaf1
  Author: Stefan Hajnoczi <stefanha@redhat.com>
  Date:   2022-11-01 (Tue, 01 Nov 2022)

  Changed paths:
    M block/blkio.c
    M qapi/block-core.json

  Log Message:
  -----------
  Merge tag 'block-pull-request' of https://gitlab.com/stefanha/qemu into 
staging

Pull request

Note that we're still discussing "block/blkio: Make driver nvme-io_uring take a
"path" instead of a "filename"". I have sent the pull request now so everything
is ready for the soft freeze tomorrow if we decide to go ahead with the patch.

# -----BEGIN PGP SIGNATURE-----
#
# iQEzBAABCAAdFiEEhpWov9P5fNqsNXdanKSrs4Grc8gFAmNgGQkACgkQnKSrs4Gr
# c8hLFgf/dnszoHO02hjoJCN2LPAxDalyYKzog+ZU8U5VdzJn2gione1jVlf3Xb0l
# mhTgrioSbKLKXavGZTSwWUki/xRgCJMtG3m07EFmMsLX0QiSOIyzLr0DslQawYdZ
# FlXyCCyAVTUILz7oUXBqORlfTKsGPHms6nlXQYhitTOsDbPyqbT9nNPKAlfGkqfj
# Pwn+oWJmjLC0aARpcrB1bXCMbqQrtZGh4bBgfIXRUJmprWqk227bkFvXNCuXU16x
# PC4oH552+6nyQyRxGpHc3o1W/8gqlxU9DTBb5arDUQaDvsDTKVkuGe2HdDI7knAT
# /m57/BFVUnA35SYOxX+0piiEbawI6Q==
# =UWL7
# -----END PGP SIGNATURE-----
# gpg: Signature made Mon 31 Oct 2022 14:50:49 EDT
# gpg:                using RSA key 8695A8BFD3F97CDAAC35775A9CA4ABB381AB73C8
# gpg: Good signature from "Stefan Hajnoczi <stefanha@redhat.com>" [ultimate]
# gpg:                 aka "Stefan Hajnoczi <stefanha@gmail.com>" [ultimate]
# Primary key fingerprint: 8695 A8BF D3F9 7CDA AC35  775A 9CA4 ABB3 81AB 73C8

* tag 'block-pull-request' of https://gitlab.com/stefanha/qemu:
  block/blkio: Make driver nvme-io_uring take a "path" instead of a "filename"
  block/blkio: Tolerate device size changes
  block/blkio: Add virtio-blk-vfio-pci BlockDriver

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>


Compare: https://github.com/qemu/qemu/compare/18cd31ff3088...a11f65ec1b8a



reply via email to

[Prev in Thread] Current Thread [Next in Thread]