Hi, On Tue, Feb 18, 2025 at 06:12:49PM +0000, Daniel P. Berrangé wrote: > This was requested by KubeVirt in > > https://gitlab.com/libvirt/libvirt/-/issues/748 > > I've not functionally tested this, since I lack any suitable guest > windows environment this is looking for MSDM tables, nor does my > machine have MSDM ACPI tables to pass to a guest. > > I'm blindly assuming that the QEMU CLI code is identical except for > s/SLIC/MSDM/. I think it is the right assumption from what I see people using with the qemu:arg from qemu:commandline option in the thread: https://gist.github.com/Informatic/49bd034d43e054bd1d8d4fec38c305ec#file-domain-xml > Also I'm pretty unhappy about the situation with the Xen driver > support. This is pre-existing, and IMHO should never have been added > as it exists today, as it allows arbitrary passthrough of *any* set > of ACPI tables, as opposed to a single type of the specific type > listed in the XML. This should have been handled with a different > XML syntax, but with stuck with this undesirable approach now, so > I've kept it as is. > > Daniel P. Berrangé (4): > conf: introduce support for multiple ACPI tables > src: validate permitted ACPI table types in libxl/qemu drivers > conf: support MSDM ACPI table type > qemu: support MSDM ACPI table type > > docs/formatdomain.rst | 4 +- > src/conf/domain_conf.c | 88 ++++++++++++++----- > src/conf/domain_conf.h | 22 ++++- > src/conf/schemas/domaincommon.rng | 5 +- > src/libvirt_private.syms | 2 + > src/libxl/libxl_conf.c | 8 +- > src/libxl/libxl_domain.c | 21 +++++ > src/libxl/xen_xl.c | 22 ++++- > src/qemu/qemu_command.c | 14 ++- > src/qemu/qemu_validate.c | 16 ++++ > src/security/security_dac.c | 18 ++-- > src/security/security_selinux.c | 16 ++-- > src/security/virt-aa-helper.c | 5 +- > .../acpi-table-many.x86_64-latest.args | 34 +++++++ > .../acpi-table-many.x86_64-latest.xml | 39 ++++++++ > tests/qemuxmlconfdata/acpi-table-many.xml | 31 +++++++ > tests/qemuxmlconftest.c | 1 + > 17 files changed, 296 insertions(+), 50 deletions(-) > create mode 100644 tests/qemuxmlconfdata/acpi-table-many.x86_64-latest.args > create mode 100644 tests/qemuxmlconfdata/acpi-table-many.x86_64-latest.xml > create mode 100644 tests/qemuxmlconfdata/acpi-table-many.xml Cheers, Victor
Attachment:
signature.asc
Description: PGP signature
