[PULL 11/12] tests: increase timeout per instance of bios-tables-test

861-1-imamm...@redhat.com> Acked-by: Michael S. Tsirkin Signed-off-by: Thomas Huth --- tests/qtest/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build index 6508bfb1a2..ff9200f882 100644 --- a/tests/qtest/meson.build

[PULL 09/12] hw: Fix crash that happens when introspecting scsi-block on older machine types

y only changing the property on the devices that really have this property. Fixes: b4912afa5f ("scsi-disk: Fix crash for VM configured with USB CDROM after live migration") Message-ID: <20240703090904.909720-1-th...@redhat.com> Acked-by: Hyman Huang Reviewed-by: Philippe Mathieu-Daudé S

[PULL 08/12] tests/avocado/machine_aspeed.py: Increase timeout for TPM test

From: Cédric Le Goater On some runners, test_arm_ast2600_evb_buildroot_tpm can take longer than 90s to complete. Increase timeout for these. Reported-by: Thomas Huth Signed-off-by: Cédric Le Goater Message-ID: <20240722085547.90650-1-...@redhat.com> Signed-off-by: Thomas Huth ---

[PULL 12/12] target/s390x: filter deprecated properties based on model expansion type

eprecated properties that were never otherwise introduced for certain models. Acked-by: David Hildenbrand Suggested-by: Jiri Denemark Signed-off-by: Collin Walling Message-ID: <20240719181741.35146-1-wall...@linux.ibm.com> Signed-off-by: Thomas Huth --- qapi/machine-target.json

[PULL 07/12] tests/avocado: Remove the remainders of the virtiofs_submounts test

The virtiofs_submounts test has been removed in commit 5da7701e2a ("virtiofsd: Remove test"), so we don't need this files anymore. Message-ID: <20240718173125.489901-1-th...@redhat.com> Signed-off-by: Thomas Huth --- .../virtiofs_submounts.py.data/cleanup.sh | 46 -

[PULL 10/12] qtest/fuzz: make range overlap check more readable

t.f...@fujitsu.com> Signed-off-by: Thomas Huth --- tests/qtest/fuzz/generic_fuzz.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/qtest/fuzz/generic_fuzz.c b/tests/qtest/fuzz/generic_fuzz.c index ec842e03c5..d107a496da 100644 --- a/tests/qtest/fuzz/generic_fuzz.c

[PULL 06/12] tests/avocado/mem-addr-space-check: Remove unused "import signal"

The "signal" module is not used here, so we can remove this import statement. Message-ID: <20240719095408.33298-1-th...@redhat.com> Reviewed-by: Daniel P. Berrangé Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Thomas Huth --- tests/avocado/mem-addr-space-check.py | 1 - 1

[PULL 05/12] tests/avocado: Move LinuxTest related code into a separate file

athieu-Daudé Signed-off-by: Thomas Huth --- tests/avocado/avocado_qemu/__init__.py | 239 +- tests/avocado/avocado_qemu/linuxtest.py | 253 tests/avocado/boot_linux.py | 3 +- tests/avocado/hotplug_blk.py| 2 +- tests/a

[PULL 04/12] tests/avocado: Allow overwriting AVOCADO_SHOW env variable

-ID: <20240719180211.48073-1-phi...@linaro.org> Signed-off-by: Thomas Huth --- tests/Makefile.include | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/Makefile.include b/tests/Makefile.include index d39d5dd6a4..6618bfed70 100644 --- a/tests/Makefile.include +++ b

[PULL 02/12] tests/avocado/boot_xen.py: unify tags

From: Cleber Rosa Because all tests share the same tags, it's possible to have all of them at the class level. Signed-off-by: Cleber Rosa Reviewed-by: Philippe Mathieu-Daudé Message-ID: <20231208190911.102879-10-cr...@redhat.com> Signed-off-by: Thomas Huth --- tests/avocado/boot_xen.p

[PULL 03/12] tests/avocado/boot_xen.py: use class attribute

From: Cleber Rosa Rather than defining a single use variable, let's just use the class attribute directly. Signed-off-by: Cleber Rosa Reviewed-by: Philippe Mathieu-Daudé Message-ID: <20231208190911.102879-11-cr...@redhat.com> Signed-off-by: Thomas Huth --- tests/avocado/boot_xen.

[PULL 00/12] QTests, Avocado and s390x fixes for 9.1 softfreeze

Mammedov (1): tests: increase timeout per instance of bios-tables-test Philippe Mathieu-Daudé (1): tests/avocado: Allow overwriting AVOCADO_SHOW env variable Thomas Huth (4): tests/avocado: Move LinuxTest related code into a separate file tests/avocado/mem-addr-space-check

[PULL 01/12] tests/avocado/boot_xen.py: merge base classes

. Signed-off-by: Cleber Rosa Message-ID: <20231208190911.102879-9-cr...@redhat.com> Signed-off-by: Thomas Huth --- tests/avocado/boot_xen.py | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tests/avocado/boot_xen.py b/tests/avocado/boot_xen.py index fc2faeedb5..f80c

Re: [RFC PATCH v4 1/2] target/riscv: Add RISC-V CSR qtest support

On 03/07/2024 10.19, Ivan Klokov wrote: The RISC-V architecture supports the creation of custom CSR-mapped devices. It would be convenient to test them in the same way as MMIO-mapped devices. To do this, a new call has been added to read/write CSR registers. Signed-off-by: Ivan Klokov ---

Re: [RFC PATCH v4 2/2] tests/qtest: QTest example for RISC-V CSR register

On 03/07/2024 10.19, Ivan Klokov wrote: Added demo for reading CSR register from qtest environment. Signed-off-by: Ivan Klokov --- tests/qtest/meson.build | 2 + tests/qtest/riscv-csr-test.c | 86 2 files changed, 88 insertions(+) create mode

Re: [PATCH] tests/avocado/machine_aspeed.py: Increase timeout for TPM test

On 22/07/2024 10.55, Cédric Le Goater wrote: On some runners, test_arm_ast2600_evb_buildroot_tpm can take longer than 90s to complete. Increase timeout for these. Reported-by: Thomas Huth Signed-off-by: Cédric Le Goater --- tests/avocado/machine_aspeed.py | 2 +- 1 file changed, 1

Re: [PATCH] tests: increase timeout per instance of bios-tables-test

On 16/07/2024 14.59, Igor Mammedov wrote: CI often fails 'cross-i686-tci' job due to runner slowness Log shows that test almost complete, with a few remaining when bios-tables-test timeout hits: 19/270 qemu:qtest+qtest-aarch64 / qtest-aarch64/bios-tables-test TIMEOUT610.02s

Re: [PULL 06/27] iotests/{024, 271}: add testcases for qemu-img rebase

On 31/10/2023 19.58, Kevin Wolf wrote: From: Andrey Drobyshev As the previous commit changes the logic of "qemu-img rebase" (it's using write alignment now), let's add a couple more test cases which would ensure it works correctly. In particular, the following scenarios: 024: add test case

Re: [PATCH] tests/avocado: Move common Avocado tags to class

On 19/07/2024 17.13, Philippe Mathieu-Daudé wrote: When Avocado tags apply to all tests in a class, we can define them once in the class: they will be applied to all test methods. Signed-off-by: Philippe Mathieu-Daudé --- tests/avocado/boot_linux.py | 2 +-

Re: [PATCH v2] target/s390x: filter deprecated properties based on model expansion type

On 18/07/2024 20.22, Collin Walling wrote: On 7/18/24 9:39 AM, Markus Armbruster wrote: Collin Walling writes: As s390 CPU models progress and deprecated properties are dropped outright, it will be cumbersome for management apps to query the host for a comprehensive list of deprecated

[PATCH] tests/avocado/mem-addr-space-check: Remove unused "import signal"

The "signal" module is not used here, so we can remove this import statement. Signed-off-by: Thomas Huth --- tests/avocado/mem-addr-space-check.py | 1 - 1 file changed, 1 deletion(-) diff --git a/tests/avocado/mem-addr-space-check.py b/tests/avocado/mem-addr-space-check.py index

[PATCH] tests/avocado: Move LinuxTest related code into a separate file

Only some few tests are using the LinuxTest class. Move the related code into a separate file so that this does not pollute the main namespace. Signed-off-by: Thomas Huth --- tests/avocado/avocado_qemu/__init__.py | 239 +- tests/avocado/avocado_qemu/linuxtest.py | 253

[PATCH] tests/avocado: Remove the remainders of the virtiofs_submounts test

The virtiofs_submounts test has been removed in commit 5da7701e2a ("virtiofsd: Remove test"), so we don't need this files anymore. Signed-off-by: Thomas Huth --- .../virtiofs_submounts.py.data/cleanup.sh | 46 -- .../guest-cleanup.sh

Re: [PATCH 1/2] ci: add gtk-vnc to the deps

+ - gtk-vnc - hostname - json-c - libaio Reviewed-by: Thomas Huth IIRC Alex has a patch in his queue already to refresh the docker images, maybe he could include this change there, too...?

Re: [PATCH 04/11] tests/functional: Add python-based tests to the meson build system

On 16/07/2024 17.15, Fabiano Rosas wrote: Thomas Huth writes: Integrate the new python-based test framework with the meson build system. Since these tests now require the pycotap module, make sure that it gets installed in the venv. The changes to the meson.build files are partly based

Re: [PATCH v1 00/11] Convert avocado tests to normal Python unittests

On 17/07/2024 10.37, Daniel P. Berrangé wrote: On Wed, Jul 17, 2024 at 10:04:19AM +0200, Thomas Huth wrote: On 16/07/2024 19.03, Thomas Huth wrote: On 16/07/2024 18.51, Daniel P. Berrangé wrote: On Tue, Jul 16, 2024 at 01:26:03PM +0200, Thomas Huth wrote: ... So instead of trying to update

Re: [PATCH v1 00/11] Convert avocado tests to normal Python unittests

On 16/07/2024 19.03, Thomas Huth wrote: On 16/07/2024 18.51, Daniel P. Berrangé wrote: On Tue, Jul 16, 2024 at 01:26:03PM +0200, Thomas Huth wrote: ... So instead of trying to update the python-based test suite in QEMU to a newer version of Avocado, we should maybe try to better integrate

Re: [RFC PATCH 0/8] Convert avocado tests to normal Python unittests

On 16/07/2024 20.03, Paolo Bonzini wrote: Il mar 16 lug 2024, 18:45 John Snow > ha scritto: My only ask is that we keep the tests running in the custom venv environment we set up at build time Yes, they do, however pytest should also be added to

Re: [RFC PATCH 0/8] Convert avocado tests to normal Python unittests

On 16/07/2024 18.45, John Snow wrote: On Thu, Jul 11, 2024, 7:55 AM Thomas Huth <mailto:th...@redhat.com>> wrote: ... - I haven't looked into logging yet ... this still needs some work   so that you could e.g. inspect the console output of the guests   somewhe

Re: [PATCH] tests: increase timeout per instance of bios-tables-test

On 16/07/2024 15.06, Michael S. Tsirkin wrote: On Tue, Jul 16, 2024 at 02:59:30PM +0200, Igor Mammedov wrote: CI often fails 'cross-i686-tci' job due to runner slowness Log shows that test almost complete, with a few remaining when bios-tables-test timeout hits: 19/270

Re: [PATCH v1 00/11] Convert avocado tests to normal Python unittests

On 16/07/2024 18.51, Daniel P. Berrangé wrote: On Tue, Jul 16, 2024 at 01:26:03PM +0200, Thomas Huth wrote: ... So instead of trying to update the python-based test suite in QEMU to a newer version of Avocado, we should maybe try to better integrate it with the meson test runner instead

[PATCH 08/11] tests/functional: Convert some avocado tests that needed avocado.utils.archive

Instead of using the "archive" module from avocado.utils, switch these tests to use the new wrapper function that is based on the "tarfile" module instead. Signed-off-by: Thomas Huth --- MAINTAINERS | 6 ++--- tests/functional/meson.build

[PATCH 05/11] tests/functional: Implement fetch_asset() method for downloading assets

In the new python test framework, we cannot use the fetch_asset() function from Avocado anymore, so we have to provide our own implementation now instead. Thus add such a function based on the urllib python module for this purpose. Signed-off-by: Thomas Huth --- tests/functional/qemu_test

[PATCH 01/11] tests/functional: Add base classes for the upcoming pytest-based tests

that will be set via meson.build later. Signed-off-by: Thomas Huth --- tests/functional/qemu_test/__init__.py | 326 + 1 file changed, 326 insertions(+) create mode 100644 tests/functional/qemu_test/__init__.py diff --git a/tests/functional/qemu_test/__init__.py b/tests

[PATCH 07/11] tests/functional: Add a function for extracting files from an archive

provide us with this functionality, so let's just add a nice wrapper function around that. Signed-off-by: Thomas Huth --- tests/functional/qemu_test/utils.py | 21 + 1 file changed, 21 insertions(+) create mode 100644 tests/functional/qemu_test/utils.py diff --git a/tests/functional/qemu_test/u

[PATCH 03/11] tests/functional: Convert avocado tests that just need a small adjustment

These simple tests can be converted to stand-alone tests quite easily, e.g. by just setting the machine to 'none' now manually or by adding "-cpu" command line parameters, since we don't support the corresponding avocado tags in the new python test framework. Signed-off-by: T

[PATCH 11/11] gitlab-ci: Add "check-functional" to the build tests

Now that we converted many tests from the "check-avocado" test suite to the "check-functional" test suite, we should make sure that these also get tested in the CI. Signed-off-by: Thomas Huth --- .gitlab-ci.d/buildtest-template.yml | 3 +- .gitlab-ci.d/buildte

[PATCH 04/11] tests/functional: Add python-based tests to the meson build system

Integrate the new python-based test framework with the meson build system. Since these tests now require the pycotap module, make sure that it gets installed in the venv. The changes to the meson.build files are partly based on an earlier patch by Ani Sinha (but heavily modified by Thomas Huth

[PATCH 10/11] tests/functional: Convert the s390x avocado tests into standalone tests

nowadays. Signed-off-by: Thomas Huth --- MAINTAINERS | 4 +- tests/functional/meson.build | 6 ++ tests/functional/qemu_test/utils.py | 7 ++ .../test_s390x_ccw_virtio.py} | 32 - .../test_s390x_topology.py

[PATCH 09/11] tests/functional: Set up logging

Create log files for each test separately, one file that contains the basic logging and one that contains the console output. Signed-off-by: Thomas Huth --- tests/functional/qemu_test/__init__.py | 24 +++- 1 file changed, 23 insertions(+), 1 deletion(-) diff --git a/tests

[PATCH 02/11] tests/functional: Convert simple avocado tests into standalone python tests

Signed-off-by: Thomas Huth --- .../test_cpu_queries.py} | 7 ++- .../test_empty_cpu_model.py} | 7 ++- .../test_mem_addr_space.py} | 53 +++ .../test_pc_cpu_hotplug_props.py} | 11 ++-- .../test_virtio_

[PATCH 06/11] tests/functional: Convert some tests that download files via fetch_asset()

Now that we've got a working fetch_asset() function, we can convert some Avocado tests that use this function for downloading their required files. Signed-off-by: Thomas Huth --- MAINTAINERS | 12 +++ tests/functional/meson.build | 25

[PATCH v1 00/11] Convert avocado tests to normal Python unittests

ab-CI - ... lots of other changes I forgot about ... in fact, I changed so many things that I also did not dare to pick up the Reviewed-bys from the RFC Thomas Huth (11): tests/functional: Add base classes for the upcoming pytest-based tests tests/functional: Convert simple avocado tests into

Re: FreeBSD update required for CI?

On 16/07/2024 01.00, Richard Henderson wrote: Hi guys, CI currently failing FreeBSD: https://gitlab.com/qemu-project/qemu/-/jobs/7347517439 pkg: No packages available to install matching 'py39-pillow' have been found in the repositories pkg: No packages available to install matching

Re: [RFC PATCH 4/8] tests/pytest: add pytest to the meson build system

On 12/07/2024 13.47, Daniel P. Berrangé wrote: On Fri, Jul 12, 2024 at 12:14:45PM +0200, Thomas Huth wrote: On 12/07/2024 11.01, Daniel P. Berrangé wrote: On Thu, Jul 11, 2024 at 01:55:42PM +0200, Thomas Huth wrote: From: Ani Sinha Integrate the pytest framework with the meson build system

Re: [RFC PATCH 4/8] tests/pytest: add pytest to the meson build system

On 12/07/2024 12.26, Daniel P. Berrangé wrote: On Fri, Jul 12, 2024 at 12:14:45PM +0200, Thomas Huth wrote: On 12/07/2024 11.01, Daniel P. Berrangé wrote: On Thu, Jul 11, 2024 at 01:55:42PM +0200, Thomas Huth wrote: From: Ani Sinha Integrate the pytest framework with the meson build system

Re: [RFC PATCH 7/8] tests/pytest: Add a function for extracting files from an archive

On 12/07/2024 11.14, Daniel P. Berrangé wrote: On Thu, Jul 11, 2024 at 01:55:45PM +0200, Thomas Huth wrote: Some Avocado-based tests use the "archive" module from avocado.utils to extract files from an archive. To be able to use these tests without Avocado, we have to provide our ow

Re: [RFC PATCH 4/8] tests/pytest: add pytest to the meson build system

On 12/07/2024 11.01, Daniel P. Berrangé wrote: On Thu, Jul 11, 2024 at 01:55:42PM +0200, Thomas Huth wrote: From: Ani Sinha Integrate the pytest framework with the meson build system. This will make meson run all the pytests under the pytest directory. Lets add a note about the compelling

Re: [RFC PATCH 5/8] tests_pytest: Implement fetch_asset() method for downloading assets

On 12/07/2024 11.09, Daniel P. Berrangé wrote: On Thu, Jul 11, 2024 at 01:55:43PM +0200, Thomas Huth wrote: In the pytests, we cannot use the fetch_asset() function from Avocado anymore, so we have to provide our own implementation now instead. Thus add such a function based

Re: [RFC PATCH 5/8] tests_pytest: Implement fetch_asset() method for downloading assets

On 11/07/2024 23.35, Richard Henderson wrote: On 7/11/24 12:23, Alex Bennée wrote: Richard Henderson writes: On 7/11/24 09:45, Richard Henderson wrote: On 7/11/24 04:55, Thomas Huth wrote: +    def fetch_asset(self, url, asset_hash): +    cache_dir = os.path.expanduser("~/.cache

Re: [RFC PATCH 5/8] tests_pytest: Implement fetch_asset() method for downloading assets

On 11/07/2024 20.49, Richard Henderson wrote: On 7/11/24 09:45, Richard Henderson wrote: On 7/11/24 04:55, Thomas Huth wrote: +    def fetch_asset(self, url, asset_hash): +    cache_dir = os.path.expanduser("~/.cache/qemu/download") +    if not os.path.exists

Re: [RFC PATCH 5/8] tests_pytest: Implement fetch_asset() method for downloading assets

On 11/07/2024 18.45, Richard Henderson wrote: On 7/11/24 04:55, Thomas Huth wrote: +    def fetch_asset(self, url, asset_hash): +    cache_dir = os.path.expanduser("~/.cache/qemu/download") +    if not os.path.exists(cache_dir): +    os.makedirs(cache_dir) +   

Re: [RFC PATCH 0/8] Convert avocado tests to normal Python unittests

On 11/07/2024 16.39, Fabiano Rosas wrote: Thomas Huth writes: ... Things that need further attention though: - All tests that use the LinuxTest / LinuxDistro classes (e.g. based on cloud-init images) really depend on the Avocado framework, thus we'd need a solution for those if we want

Re: Avocado 88.1 vs Python 3.12

On 10/07/2024 01.45, Richard Henderson wrote: On 7/9/24 09:26, Philippe Mathieu-Daudé wrote: On 9/7/24 17:41, Richard Henderson wrote: Hi guys, I have reinstalled my development box to ubuntu 24 (because the Rust support is better than my previous install; ho hum).  I thought I had tested

[RFC PATCH 3/8] tests/pytest: Convert info_usernet and version test with small adjustments

These two simple tests can be converted to a pytest quite easily, we just have to set the machine to 'none' now manually since we don't support the avocado tags here yet. Signed-off-by: Thomas Huth --- .../info_usernet.py => pytest/test_info_usernet.py} | 6 ++ tests/{avoc

[RFC PATCH 5/8] tests_pytest: Implement fetch_asset() method for downloading assets

In the pytests, we cannot use the fetch_asset() function from Avocado anymore, so we have to provide our own implementation now instead. Thus add such a function based on the _download_with_cache() function from tests/vm/basevm.py for this purpose. Signed-off-by: Thomas Huth --- tests/pytest

[RFC PATCH 4/8] tests/pytest: add pytest to the meson build system

From: Ani Sinha Integrate the pytest framework with the meson build system. This will make meson run all the pytests under the pytest directory. Signed-off-by: Ani Sinha [thuth: Removed the acpi-bits and adjusted for converted avocado tests instead] Signed-off-by: Thomas Huth --- tests

[RFC PATCH 7/8] tests/pytest: Add a function for extracting files from an archive

provide us with this functionality, so let's just add a nice wrapper function around that. Signed-off-by: Thomas Huth --- tests/pytest/qemu_pytest/utils.py | 21 + 1 file changed, 21 insertions(+) create mode 100644 tests/pytest/qemu_pytest/utils.py diff --git a/tests/pytest/qemu_pytest/utils.p

[RFC PATCH 8/8] tests/pytest: Convert avocado test that needed avocado.utils.archive

Instead of using the "archive" module from avocado.utils, switch these tests to use the new wrapper function that is based on the "tarfile" module instead. Signed-off-by: Thomas Huth --- tests/pytest/meson.build | 5 + .../te

[RFC PATCH 6/8] tests/pytest: Convert some tests that download files via fetch_asset()

Now that we've got a working fetch_asset() function, we can convert some Avocado tests that use this function for downloading their required files. Signed-off-by: Thomas Huth --- tests/pytest/meson.build | 16 +++ .../test_machine_arm_n8x0.py

[RFC PATCH 2/8] tests/pytest: Convert some simple avocado tests into pytests

e PYTEST_QEMU_BINARY environment variable, and the source and build directories via the PYTEST_SOURCE_ROOTand PYTEST_BUILD_ROOT environment variables. Signed-off-by: Thomas Huth --- tests/{avocado/cpu_queries.py => pytest/test_cpu_queries.py} | 2 +- .../empty_cpu_model.py => pyte

[RFC PATCH 1/8] tests/pytest: Add base classes for the upcoming pytest-based tests

that will be set via meson.build later. Signed-off-by: Thomas Huth --- tests/pytest/qemu_pytest/__init__.py | 344 +++ 1 file changed, 344 insertions(+) create mode 100644 tests/pytest/qemu_pytest/__init__.py diff --git a/tests/pytest/qemu_pytest/__init__.py b/tests/pytest

[RFC PATCH 0/8] Convert avocado tests to normal Python unittests

rk on documentation updates yet (will do that if we agree to continue with this patch series) What's your thoughts? Is it worth to continue with this approach? Or shall I rather forget about it and wait for the Avocado version update? Thomas Ani Sinha (1): tests/pytest: add pytest t

[PATCH] tests/avocado: Remove the non-working virtio_check_params test

The test has been marked as broken more than 4 years ago, and so far nobody ever cared to fix it. Thus let's simply remove it now ... if somebody ever needs it again, they can restore the file from an older version of QEMU. Signed-off-by: Thomas Huth --- tests/avocado/virtio_check_params.py

[PATCH] tests/avocado: Remove non-working sparc leon3 test

The test has been marked as broken more than 4 years ago, and so far nobody ever cared to fix it. Thus let's simply remove it now ... if somebody ever needs it again, they can restore the file from an older version of QEMU. Signed-off-by: Thomas Huth --- tests/avocado/machine_sparc_leon3.py

Re: [PATCH v3 9/9] tests/qtest: Delete previous boot file

insertions(+), 7 deletions(-) Acked-by: Thomas Huth

Re: [PATCH v2 20/40] gitlab: don't bother with KVM for TCI builds

On 05/07/2024 23.44, Philippe Mathieu-Daudé wrote: On 5/7/24 18:49, Thomas Huth wrote: On 05/07/2024 18.34, Philippe Mathieu-Daudé wrote: On 5/7/24 10:40, Alex Bennée wrote: In fact any other accelerator would be pointless as the point is to exercise the TCI accelerator anyway. Signed-off

Re: [PATCH v2 20/40] gitlab: don't bother with KVM for TCI builds

On 05/07/2024 18.34, Philippe Mathieu-Daudé wrote: On 5/7/24 10:40, Alex Bennée wrote: In fact any other accelerator would be pointless as the point is to exercise the TCI accelerator anyway. Signed-off-by: Alex Bennée ---   .gitlab-ci.d/buildtest.yml   | 2 +-   .gitlab-ci.d/crossbuilds.yml |

[PATCH] system: Enable the device aliases for or1k, too

o the list. Signed-off-by: Thomas Huth --- system/qdev-monitor.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/system/qdev-monitor.c b/system/qdev-monitor.c index 6af6ef7d66..60c3b6ad96 100644 --- a/system/qdev-monitor.c +++ b/system/qdev-monitor.c @@ -57,10 +57,10 @@ type

Re: [PATCH v2 20/40] gitlab: don't bother with KVM for TCI builds

deletions(-) Reviewed-by: Thomas Huth

Re: [PULL 02/12] tests/qtest/migration-test: enable on s390x with TCG

On 04/07/2024 13.20, Nicholas Piggin wrote: On Tue Jul 2, 2024 at 8:33 PM AEST, Thomas Huth wrote: From: Nicholas Piggin s390x with TCG is more stable now. Enable it. Ah, you did a more complete version of my flic fix that migrates all the state. I didn't see that go by but yeah I suspect

Re: [PULL v2 00/88] virtio: features,fixes

On 03/07/2024 21.45, Michael S. Tsirkin wrote: On Wed, Jul 03, 2024 at 08:46:38PM +0200, Thomas Huth wrote: On 03/07/2024 19.01, Richard Henderson wrote: On 7/3/24 09:51, Michael S. Tsirkin wrote: On Wed, Jul 03, 2024 at 09:31:45AM -0700, Richard Henderson wrote: On 7/2/24 13:15, Michael S

Re: [PULL v2 00/88] virtio: features,fixes

On 03/07/2024 22.26, Michael S. Tsirkin wrote: On Wed, Jul 03, 2024 at 08:46:38PM +0200, Thomas Huth wrote: On 03/07/2024 19.01, Richard Henderson wrote: On 7/3/24 09:51, Michael S. Tsirkin wrote: On Wed, Jul 03, 2024 at 09:31:45AM -0700, Richard Henderson wrote: On 7/2/24 13:15, Michael S

Re: [PULL v2 00/88] virtio: features,fixes

On 03/07/2024 19.01, Richard Henderson wrote: On 7/3/24 09:51, Michael S. Tsirkin wrote: On Wed, Jul 03, 2024 at 09:31:45AM -0700, Richard Henderson wrote: On 7/2/24 13:15, Michael S. Tsirkin wrote: The following changes since commit 1152a0414944f03231f3177207d379d58125890e:     Merge tag

Re: [PATCH v2 2/2] docs: remove Sphinx 1.x compatibility code

-- docs/sphinx/kernellog.py | 28 docs/sphinx/qapidoc.py | 29 +++-- 4 files changed, 19 insertions(+), 97 deletions(-) delete mode 100644 docs/sphinx/kernellog.py Reviewed-by: Thomas Huth

Re: [PATCH v2 1/2] Python: bump minimum sphinx version to 3.4.3

Signed-off-by: John Snow Acked-by: Paolo Bonzini Acked-by: Markus Armbruster --- docs/conf.py| 7 +++ pythondeps.toml | 2 +- 2 files changed, 4 insertions(+), 5 deletions(-) Reviewed-by: Thomas Huth

[PATCH] hw: Fix crash that happens when introspecting scsi-block on older machine types

y only changing the property on the devices that really have this property. Fixes: b4912afa5f ("scsi-disk: Fix crash for VM configured with USB CDROM after live migration") Signed-off-by: Thomas Huth --- hw/core/machine.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/

[PULL 03/12] tests/qtest: Use qtest_add_data_func_full()

627-san-v2-10-750bb0946...@daynix.com> Reviewed-by: Thomas Huth Signed-off-by: Thomas Huth --- tests/qtest/device-introspect-test.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/tests/qtest/device-introspect-test.c b/tests/qtest/device-introspect-test.c index 5b0f

[PULL 05/12] tests/qtest: Free old machine variable name

From: Akihiko Odaki This fixes LeakSanitizer warnings. Signed-off-by: Akihiko Odaki Reviewed-by: Peter Maydell Reviewed-by: Michael S. Tsirkin Message-ID: <20240627-san-v2-12-750bb0946...@daynix.com> Signed-off-by: Thomas Huth --- tests/qtest/libqtest.c | 1 + 1 file changed, 1 ins

[PULL 04/12] tests/qtest: Free unused QMP response

From: Akihiko Odaki This fixes LeakSanitizer warnings. Signed-off-by: Akihiko Odaki Reviewed-by: Michael S. Tsirkin Message-ID: <20240627-san-v2-11-750bb0946...@daynix.com> Signed-off-by: Thomas Huth --- tests/qtest/libqtest.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a

[PULL 10/12] tests/avocado: add hotplug_blk test

From: Vladimir Sementsov-Ogievskiy Introduce a test, that checks that plug/unplug of virtio-blk device works. (the test is developed by copying hotplug_cpu.py, so keep original copyright) Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Thomas Huth Message-ID: <20240409065854.366

[PULL 09/12] hw/s390x: Attach default virtio-net devices to the /machine/virtual-css-bridge

viewed-by: Cédric Le Goater Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Thomas Huth --- hw/s390x/s390-virtio-ccw.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/hw/s390x/s390-virtio-ccw.c b/hw/s390x/s390-virtio-ccw.c index f87ca36264..c1edbd9131 100644 --- a/hw/s390x/s390-virtio-ccw.c

[PULL 11/12] .travis.yml: Install python3-tomli in all build jobs

Since commit 1f97715c83 ('Revert "python: use vendored tomli"') this package is a hard requirement for compiling QEMU, so install it now in all Travis jobs, too. Message-ID: <20240624094807.182313-1-th...@redhat.com> Acked-by: Alex Bennée Signed-off-by: Thomas Huth ---

[PULL 02/12] tests/qtest/migration-test: enable on s390x with TCG

From: Nicholas Piggin s390x with TCG is more stable now. Enable it. Signed-off-by: Nicholas Piggin Message-Id: <20240525131241.378473-3-npig...@gmail.com> Reviewed-by: Prasad Pandit [thuth: Added "with TCG" to the commit message] Signed-off-by: Thomas Huth --- tests/qtest/

[PULL 12/12] pc-bios/s390-ccw: Remove duplicated LDFLAGS

The -Wl,-pie and -nostdlib flags are added to LDFLAGS twice. Merge the two lines to get rid of the duplicates. Message-ID: <20240621082422.136217-2-th...@redhat.com> Signed-off-by: Thomas Huth --- pc-bios/s390-ccw/Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff

[PULL 07/12] tests/qtest: Free GThread

From: Akihiko Odaki These GThreads are never referenced. Signed-off-by: Akihiko Odaki Reviewed-by: Peter Maydell Reviewed-by: Michael S. Tsirkin Message-ID: <20240627-san-v2-15-750bb0946...@daynix.com> Signed-off-by: Thomas Huth --- tests/qtest/vhost-user-test.c | 6 +++--- 1 file c

[PULL 08/12] docs: add precision about capstone for execlog plugin

: <20240620135731.977377-1-erdn...@crans.org> Signed-off-by: Thomas Huth --- docs/devel/tcg-plugins.rst | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/docs/devel/tcg-plugins.rst b/docs/devel/tcg-plugins.rst index 9cc09d8c3d..f7d7b9e3a4 100644 --- a/docs/devel/tcg-plugins.rst +++ b/docs

[PULL 06/12] tests/qtest: Free paths

From: Akihiko Odaki This fixes LeakSanitizer warnings. Signed-off-by: Akihiko Odaki Reviewed-by: Michael S. Tsirkin Message-ID: <20240627-san-v2-14-750bb0946...@daynix.com> Signed-off-by: Thomas Huth --- tests/qtest/qos-test.c | 16 1 file changed, 12 insertions

[PULL 01/12] hw/intc/s390_flic: Fix interrupt controller migration on s390x with TCG

e, so s390_cpu_exec_interrupt falls through to halting again." Thus let's finally migrate the pending state, and to be on the safe side, also the other state variables of the QEMUS390FLICState structure. Message-ID: <20240619144421.261342-1-th...@redhat.com> Signed-off-by: Thomas Huth ---

[PULL 00/12] qtest, s390x, avocado and doc patches

): tests/qtest/migration-test: enable on s390x with TCG Thomas Huth (4): hw/intc/s390_flic: Fix interrupt controller migration on s390x with TCG hw/s390x: Attach default virtio-net devices to the /machine/virtual-css-bridge .travis.yml: Install python3-tomli in all build jobs pc

Re: [PATCH v2 13/15] tests/qtest: Delete previous boot file

On 27/06/2024 15.37, Akihiko Odaki wrote: A test run may create boot files several times. Delete the previous boot file before creating a new one. Signed-off-by: Akihiko Odaki --- tests/qtest/migration-test.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff

Re: [PATCH v2 00/15] Fix check-qtest-ppc64 sanitizer errors

On 02/07/2024 00.23, BALATON Zoltan wrote: On Mon, 1 Jul 2024, Michael S. Tsirkin wrote: On Thu, Jun 27, 2024 at 10:37:43PM +0900, Akihiko Odaki wrote: Based-on: <3ad18bc590ef28e1526e8053568086b453e7ffde.1718211878.git.quic_mathb...@quicinc.com> ("[PATCH] cpu: fix memleak of 'halt_cond' and

Re: [PATCH v2 10/15] tests/qtest: Use qtest_add_data_func_full()

-introspect-test.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) Reviewed-by: Thomas Huth

[PATCH] hw/s390x: Attach default virtio-net devices to the /machine/virtual-css-bridge

The initial virtio-net-ccw devices currently do not have a proper parent in the QOM tree, so they show up under /machine/unattached - which is somewhat ugly. Let's attach them to /machine/virtual-css-bridge/virtual-css instead. Signed-off-by: Thomas Huth --- hw/s390x/s390-virtio-ccw.c | 3

Re: [PATCH] hw/virtio: Fix the de-initialization of vhost-user devices

On 01/07/2024 17.06, Michael S. Tsirkin wrote: On Mon, Jul 01, 2024 at 04:07:56PM +0200, Thomas Huth wrote: On 18/06/2024 14.19, Thomas Huth wrote: The unrealize functions of the various vhost-user devices are calling the corresponding vhost_*_set_status() functions with a status of 0 to shut

Re: [PATCH] hw/virtio: Fix the de-initialization of vhost-user devices

On 18/06/2024 14.19, Thomas Huth wrote: The unrealize functions of the various vhost-user devices are calling the corresponding vhost_*_set_status() functions with a status of 0 to shut down the device correctly. Now these vhost_*_set_status() functions all follow this scheme: bool

[PATCH] Remove inclusion of hw/hw.h from files that don't need it

hw/hw.h only contains the prototype of hw_error() nowadays, so files that don't use this function don't need to include this header. Signed-off-by: Thomas Huth --- include/hw/misc/xlnx-cfi-if.h | 1 - hw/misc/edu.c | 1 - hw/vfio/container.c | 1 - 3 files changed, 3

Re: [PATCH v2 07/14] hw/i386: convert 'q35' machine definitions to use new macros

6/pc_q35.c | 215 --- 1 file changed, 90 insertions(+), 125 deletions(-) Reviewed-by: Thomas Huth

Re: [PATCH v2 06/14] hw/i386: convert 'i440fx' machine definitions to use new macros

thrice in three different formats in the calls to DEFINE_I440FX_MACHINE. Signed-off-by: Daniel P. Berrangé --- hw/i386/pc_piix.c| 219 +++ include/hw/i386/pc.h | 26 + 2 files changed, 122 insertions(+), 123 deletions(-) Reviewed-by: Thomas Huth

Re: [PATCH 0/7] pc-bios/s390-ccw: Merge the netboot loader into s390-ccw.img

On 28/06/2024 20.01, Jared Rossi wrote: On 6/24/24 1:55 AM, Thomas Huth wrote: [...] I think it should be fine, both functions are basically just a wrapper around the write() function in sclp.c, with sclp_print() being rather dumb while printf() is doing the usual string formatting before

  1   2   3   4   5   6   7   8   9   10   >