On 8/27/24 17:08, Andrea Bolognani wrote:
> Back when pvpanic-pci support was introduced 1.5 years ago (!), we
> required the user to manually provide the model name. This is
> incovenient and doesn't match the behavior seen on other
> architectures. Make things more user friendly.
>
> Andrea Bolognani (4):
> tests: Add coverage for panic on riscv64
> qemu: Refactor default panic model
> qemu: Sometimes the default panic model doesn't exist
> qemu: Use pvpanic by default on aarch64
>
> src/qemu/qemu_domain.c | 36 ++++++++++++-------
> src/qemu/qemu_validate.c | 6 +++-
> .../aarch64-panic-no-model.aarch64-latest.err | 1 -
> ...ault-models.aarch64-latest.abi-update.args | 1 +
> ...fault-models.aarch64-latest.abi-update.xml | 3 ++
> ...64-virt-default-models.aarch64-latest.args | 1 +
> ...h64-virt-default-models.aarch64-latest.xml | 3 ++
> .../aarch64-virt-default-models.xml | 2 +-
> .../riscv64-panic-no-model.riscv64-latest.err | 1 +
> ...o-model.xml => riscv64-panic-no-model.xml} | 4 +--
> tests/qemuxmlconftest.c | 2 +-
> 11 files changed, 42 insertions(+), 18 deletions(-)
> delete mode 100644 tests/qemuxmlconfdata/aarch64-panic-no-model.aarch64-latest.err
> create mode 100644 tests/qemuxmlconfdata/riscv64-panic-no-model.riscv64-latest.err
> rename tests/qemuxmlconfdata/{aarch64-panic-no-model.xml => riscv64-panic-no-model.xml} (65%)
>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
Michal