On Thu, Aug 07, 2025 at 15:48:39 -0500, Praveen K Paladugu wrote:
> "[PATCH v3 00/26] Implementing a MSHV (Microsoft Hypervisor) accelerator"

[...]

> Known issue:
> 
> ```
> abs_builddir=/home/user/QEMU_MSHV/libvirt/build/tests 
> abs_top_srcdir=/home/user/QEMU_MSHV/libvirt 
> LD_LIBRARY_PATH=/home/user/QEMU_MSHV/libvirt/build/tests:/home/user/QEMU_MSHV/libvirt/build/src
>  abs_srcdir=/home/user/QEMU_MSHV/libvirt/tests VIR_TEST_EXPENSIVE=0 
> ASAN_OPTIONS=halt_on_error=1:abort_on_error=1:print_summary=1 
> UBSAN_OPTIONS=halt_on_error=1:abort_on_error=1:print_summary=1:print_stacktrace=1
>  LC_ALL=C LIBVIRT_AUTOSTART=0 G_DEBUG=fatal-warnings MALLOC_PERTURB_=255 
> abs_top_builddir=/home/user/QEMU_MSHV/libvirt/build 
> /home/user/QEMU_MSHV/libvirt/build/tests/qemucapabilitiestest
> TEST: qemucapabilitiestest
> 
> wrong expected command in 
> /home/user/QEMU_MSHV/libvirt/tests/qemucapabilitiesdata/caps_9.2.0_s390x.replies:23538:
>  : {"execute":"query-mshv","id":"libvirt-6"} expected 
> {"execute":"qom-list-types","id":"libvirt-6"}
> ```
> Could someone point me to the steps to regenerate the replies files to fix 
> above
> issue?

There are two possibilities how to approach this.

First one is that you move this probe slightly further which will allow
you to probe available commands first. You then can avoid the probe in
cases when qemu doesn't yet support that command preventing you from the
need to modify the old replies files.

The test files can't really be re-generated as in some cases the old
qemu versions no longer compile and parts of the file depend on the host
setup.

Nevertheless, we do have a tool to programatically modify those.
The scripts/qemu-replies-tool.py has provisions which allow you to write
code to add/remove sections in the .replies files. look for the
'modify_replies' method which already has an example adding some device
data based on the version of qemu. You modify that code to put the json
at the appropriate place and then either re-run the testsuite with
VIR_TEST_REGENERATE_OUTPUT or run the tool with:

  ./scripts/qemu-replies-tool.py --regenerate --repliesdir
  tests/qemucapabilitiesdata/


Beware that you must run it exactly once, and then commit the changes
and disable the modification code again, otherwise it'll likely try to
re-add the stuff.

Reply via email to