mbox

[PULL,0/9] qtest patches

Message ID 20190815175922.3475-1-thuth@redhat.com
State New
Headers show

Pull-request

https://gitlab.com/huth/qemu.git tags/pull-request-2019-08-15

Message

Thomas Huth Aug. 15, 2019, 5:59 p.m. UTC
Hi Peter,

the following changes since commit 9e06029aea3b2eca1d5261352e695edc1e7d7b8b:

  Update version for v4.1.0 release (2019-08-15 13:03:37 +0100)

are available in the Git repository at:

  https://gitlab.com/huth/qemu.git tags/pull-request-2019-08-15

for you to fetch changes up to 6fc9f3d347aee337421f8afc4d0984294f8ea6c3:

  tests/libqtest: Make qmp_assert_success() independent from global_qtest (2019-08-15 19:24:10 +0200)

----------------------------------------------------------------
- Fix for ctrl queue in the virtio-net QOS driver
- Improve Valgrind reports in the tests that use the null-co driver
- Get rid of global_qtest related code in libqtest and libqos
----------------------------------------------------------------

Andrey Shinkevich (1):
      tests: Set read-zeroes on for null-co driver

Oleinik, Alexander (2):
      qtest: Rename qtest.c:qtest_init()
      libqos: Account for the ctrl queue in virtio-net

Thomas Huth (6):
      tests/libqos: Make generic virtio code independent from global_qtest
      tests/libqos: Make virtio-pci code independent from global_qtest
      tests/libqtest: Remove unused function hmp()
      tests/libqtest: Clean up qtest_cb_for_every_machine() wrt global_qtest
      tests/libqtest: Make qtest_qmp_device_add/del independent from global_qtest
      tests/libqtest: Make qmp_assert_success() independent from global_qtest

 include/sysemu/qtest.h     |   2 +-
 qtest.c                    |   3 +-
 tests/cpu-plug-test.c      |  15 +++--
 tests/drive_del-test.c     |   3 +-
 tests/e1000e-test.c        |   2 +-
 tests/ivshmem-test.c       |   2 +-
 tests/libqos/usb.c         |   6 +-
 tests/libqos/usb.h         |   2 +-
 tests/libqos/virtio-net.c  |   1 +
 tests/libqos/virtio-net.h  |   2 +-
 tests/libqos/virtio-pci.c  |   8 +--
 tests/libqos/virtio-scsi.c |   3 +-
 tests/libqos/virtio.c      |  81 ++++++++++++-----------
 tests/libqos/virtio.h      |  30 +++++----
 tests/libqtest.c           |  37 ++++-------
 tests/libqtest.h           |  24 +++----
 tests/megasas-test.c       |   3 +-
 tests/nvme-test.c          |   3 +-
 tests/qmp-test.c           |   2 +-
 tests/test-blockjob-txn.c  |   5 +-
 tests/test-blockjob.c      |   5 +-
 tests/usb-hcd-ohci-test.c  |   2 +-
 tests/usb-hcd-uhci-test.c  |  11 ++--
 tests/usb-hcd-xhci-test.c  |  25 ++++---
 tests/virtio-9p-test.c     |  16 +++--
 tests/virtio-blk-test.c    | 159 ++++++++++++++++++++++++---------------------
 tests/virtio-ccw-test.c    |  25 ++++---
 tests/virtio-net-test.c    |  35 ++++++----
 tests/virtio-rng-test.c    |   2 +-
 tests/virtio-scsi-test.c   |  35 ++++++----
 tests/virtio-serial-test.c |   4 +-
 vl.c                       |   2 +-
 32 files changed, 300 insertions(+), 255 deletions(-)

Comments

no-reply@patchew.org Aug. 15, 2019, 6:17 p.m. UTC | #1
Patchew URL: https://patchew.org/QEMU/20190815175922.3475-1-thuth@redhat.com/



Hi,

This series failed build test on s390x host. Please find the details below.

=== TEST SCRIPT BEGIN ===
#!/bin/bash
# Testing script will be invoked under the git checkout with
# HEAD pointing to a commit that has the patches applied on top of "base"
# branch
set -e

echo
echo "=== ENV ==="
env

echo
echo "=== PACKAGES ==="
rpm -qa

echo
echo "=== UNAME ==="
uname -a

CC=$HOME/bin/cc
INSTALL=$PWD/install
BUILD=$PWD/build
mkdir -p $BUILD $INSTALL
SRC=$PWD
cd $BUILD
$SRC/configure --cc=$CC --prefix=$INSTALL
make -j4
# XXX: we need reliable clean up
# make check -j4 V=1
make install
=== TEST SCRIPT END ===




The full log is available at
http://patchew.org/logs/20190815175922.3475-1-thuth@redhat.com/testing.s390x/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
Thomas Huth Aug. 15, 2019, 6:34 p.m. UTC | #2
On 8/15/19 8:17 PM, no-reply@patchew.org wrote:
> Patchew URL: https://patchew.org/QEMU/20190815175922.3475-1-thuth@redhat.com/
> 
> Hi,
> 
> This series failed build test on s390x host. Please find the details below.
[...]
> The full log is available at
> http://patchew.org/logs/20190815175922.3475-1-thuth@redhat.com/testing.s390x/?type=message.

The error message is not really useful and seems unrelated:

make: *** [/var/tmp/patchew-tester-tmp-krxfzhwl/src/rules.mak:69:
qobject/block-qdict.o] Error 1

I did not touch anything in qobject/ at all in this series, so the
problem is likely something completely different? Could it be that the
machine simply ran out of disk space or something similar?

 Thomas
Peter Maydell Aug. 16, 2019, 1:53 p.m. UTC | #3
On Thu, 15 Aug 2019 at 18:59, Thomas Huth <thuth@redhat.com> wrote:
>
>  Hi Peter,
>
> the following changes since commit 9e06029aea3b2eca1d5261352e695edc1e7d7b8b:
>
>   Update version for v4.1.0 release (2019-08-15 13:03:37 +0100)
>
> are available in the Git repository at:
>
>   https://gitlab.com/huth/qemu.git tags/pull-request-2019-08-15
>
> for you to fetch changes up to 6fc9f3d347aee337421f8afc4d0984294f8ea6c3:
>
>   tests/libqtest: Make qmp_assert_success() independent from global_qtest (2019-08-15 19:24:10 +0200)
>
> ----------------------------------------------------------------
> - Fix for ctrl queue in the virtio-net QOS driver
> - Improve Valgrind reports in the tests that use the null-co driver
> - Get rid of global_qtest related code in libqtest and libqos
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/4.1
for any user-visible changes.

-- PMM