[Qemu-devel] [PATCH 0/5] tests/acceptance: Add bFLT loader linux-user test

Philippe Mathieu-Daudé posted 5 patches 4 years, 10 months ago
Test docker-clang@ubuntu passed
Test s390x failed
Test asan passed
Test docker-mingw@fedora passed
Test FreeBSD passed
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190625101524.13447-1-philmd@redhat.com
Maintainers: Aurelien Jarno <aurelien@aurel32.net>, "Philippe Mathieu-Daudé" <philmd@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, Fam Zheng <fam@euphon.net>, Aleksandar Rikalo <arikalo@wavecomp.com>
There is a newer version of this series
.travis.yml                               |  2 +-
docs/devel/testing.rst                    |  8 ++--
tests/acceptance/avocado_qemu/__init__.py | 26 +++++++++---
tests/acceptance/boot_linux_console.py    |  4 +-
tests/acceptance/cpu_queries.py           |  4 +-
tests/acceptance/empty_cpu_model.py       |  4 +-
tests/acceptance/linux_initrd.py          |  4 +-
tests/acceptance/linux_ssh_mips_malta.py  |  4 +-
tests/acceptance/load_bflt.py             | 52 +++++++++++++++++++++++
tests/acceptance/migration.py             |  4 +-
tests/acceptance/version.py               |  4 +-
tests/acceptance/virtio_version.py        |  4 +-
tests/acceptance/vnc.py                   |  4 +-
13 files changed, 95 insertions(+), 29 deletions(-)
create mode 100644 tests/acceptance/load_bflt.py
[Qemu-devel] [PATCH 0/5] tests/acceptance: Add bFLT loader linux-user test
Posted by Philippe Mathieu-Daudé 4 years, 10 months ago
Hi,

This series refactor the current avocado_qemu Test class to allow
MachineTest and LinuxUserTest classes.
We can then add linux-user tests.
A simple/quick one is added as example, it runs a STM32 busybox
binary (in the bFLT format) on Travis-CI.
The same test can be used to confirm LP bug 1833668 [*] is fixed.

Regards,

Phil.

[*] https://bugs.launchpad.net/qemu/+bug/1833668

Philippe Mathieu-Daudé (5):
  tests/acceptance: Rename avocado_qemu.Test as MachineTest
  tests/acceptance: Make pick_default_qemu_bin() more generic
  tests/acceptance: Introduce LinuxUserTest base class
  tests/acceptance: Add bFLT loader linux-user test
  .travis.yml: Let the avocado job run linux-user tests

 .travis.yml                               |  2 +-
 docs/devel/testing.rst                    |  8 ++--
 tests/acceptance/avocado_qemu/__init__.py | 26 +++++++++---
 tests/acceptance/boot_linux_console.py    |  4 +-
 tests/acceptance/cpu_queries.py           |  4 +-
 tests/acceptance/empty_cpu_model.py       |  4 +-
 tests/acceptance/linux_initrd.py          |  4 +-
 tests/acceptance/linux_ssh_mips_malta.py  |  4 +-
 tests/acceptance/load_bflt.py             | 52 +++++++++++++++++++++++
 tests/acceptance/migration.py             |  4 +-
 tests/acceptance/version.py               |  4 +-
 tests/acceptance/virtio_version.py        |  4 +-
 tests/acceptance/vnc.py                   |  4 +-
 13 files changed, 95 insertions(+), 29 deletions(-)
 create mode 100644 tests/acceptance/load_bflt.py

-- 
2.20.1


Re: [Qemu-devel] [PATCH 0/5] tests/acceptance: Add bFLT loader linux-user test
Posted by no-reply@patchew.org 4 years, 10 months ago
Patchew URL: https://patchew.org/QEMU/20190625101524.13447-1-philmd@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
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

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

echo
echo "=== PACKAGES ==="
rpm -qa
=== TEST SCRIPT END ===

  CC      qapi/qmp-registry.o
  CC      qapi/qmp-dispatch.o

Warning, treated as error:
/var/tmp/patchew-tester-tmp-63tbziko/src/docs/devel/testing.rst:627:Title underline too short.

The ``avocado_qemu.MachineTest`` base test class


The full log is available at
http://patchew.org/logs/20190625101524.13447-1-philmd@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] [PATCH 0/5] tests/acceptance: Add bFLT loader linux-user test
Posted by Philippe Mathieu-Daudé 4 years, 10 months ago
On 6/25/19 12:47 PM, no-reply@patchew.org wrote:
> Patchew URL: https://patchew.org/QEMU/20190625101524.13447-1-philmd@redhat.com/
> 
> This series failed build test on s390x host. Please find the details below.
> 
> === TEST SCRIPT END ===
> 
>   CC      qapi/qmp-registry.o
>   CC      qapi/qmp-dispatch.o
> 
> Warning, treated as error:
> /var/tmp/patchew-tester-tmp-63tbziko/src/docs/devel/testing.rst:627:Title underline too short.
>
> The ``avocado_qemu.MachineTest`` base test class

Ah, this warning was not obvious to figure out...

The fix is trivial:

-- >8 --

 The ``avocado_qemu.MachineTest`` base test class
------------------------------------------
+------------------------------------------------

---