[Qemu-devel] [PULL 0/9] qtest patches

Thomas Huth posted 9 patches 4 years, 8 months ago
Test FreeBSD passed
Test docker-mingw@fedora passed
Test asan passed
Test docker-clang@ubuntu passed
Test checkpatch passed
Test s390x failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190815175922.3475-1-thuth@redhat.com
Maintainers: Fam Zheng <fam@euphon.net>, Laurent Vivier <lvivier@redhat.com>, Greg Kurz <groug@kaod.org>, Thomas Huth <thuth@redhat.com>, Keith Busch <keith.busch@intel.com>, Stefan Hajnoczi <stefanha@redhat.com>, Amit Shah <amit@kernel.org>, Markus Armbruster <armbru@redhat.com>, Hannes Reinecke <hare@suse.com>, Gerd Hoffmann <kraxel@redhat.com>, Jason Wang <jasowang@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
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(-)
[Qemu-devel] [PULL 0/9] qtest patches
Posted by Thomas Huth 4 years, 8 months ago
 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(-)

Re: [Qemu-devel] [PULL 0/9] qtest patches
Posted by Peter Maydell 4 years, 8 months ago
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

Re: [Qemu-devel] [PULL 0/9] qtest patches
Posted by no-reply@patchew.org 4 years, 8 months ago
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
Re: [Qemu-devel] [PULL 0/9] qtest patches
Posted by Thomas Huth 4 years, 8 months ago
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