[libvirt] [PATCHv2 1/5] S390: Override QEMU_CAPS_NO_ACPI for s390x
Daniel Veillard
veillard at redhat.com
Mon Jun 25 08:33:37 UTC 2012
On Fri, Jun 22, 2012 at 01:50:10PM +0200, Viktor Mihajlovski wrote:
> Starting a KVM guest on s390 fails immediately. This is because
> "qemu --help" reports -no-acpi even for the s390(x) architecture but
> -no-acpi isn't supported there.
> Workaround is to remove QEMU_CAPS_NO_ACPI from the capability set
> after the version/capability extraction.
>
> Signed-off-by: Viktor Mihajlovski <mihajlov at linux.vnet.ibm.com>
> ---
> src/qemu/qemu_capabilities.c | 5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
> index 4308833..0c01cb0 100644
> --- a/src/qemu/qemu_capabilities.c
> +++ b/src/qemu/qemu_capabilities.c
> @@ -1511,6 +1511,11 @@ int qemuCapsExtractVersionInfo(const char *qemu, const char *arch,
> qemuCapsSet(flags, QEMU_CAPS_PCI_MULTIBUS);
> }
>
> + /* S390 and probably other archs do not support no-acpi -
> + maybe the qemu option parsing should be re-thought. */
> + if (STREQLEN(arch, "s390x", 5))
> + qemuCapsClear(flags, QEMU_CAPS_NO_ACPI);
> +
> /* qemuCapsExtractDeviceStr will only set additional flags if qemu
> * understands the 0.13.0+ notion of "-device driver,". */
> if (qemuCapsGet(flags, QEMU_CAPS_DEVICE) &&
Shouldn't the "s390" prefix be used instead to catch both s390 and s390x ?
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel at veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
More information about the libvir-list
mailing list