Re: [PATCHv2 1/5] S390: Override QEMU_CAPS_NO_ACPI for s390x

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 22.06.2012 13:50, 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@xxxxxxxxxxxxxxxxxx>
> ---
>  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) &&
> 

Pushed with this change:

diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index dda9422..1e12a39 100644
--- a/src/qemu/qemu_capabilities.c
+++ b/src/qemu/qemu_capabilities.c
@@ -1516,7 +1516,7 @@ int qemuCapsExtractVersionInfo(const char *qemu,
const char *arch,

     /* S390 and probably other archs do not support no-acpi -
        maybe the qemu option parsing should be re-thought. */
-    if (STREQLEN(arch, "s390x", 5))
+    if (STRPREFIX(arch, "s390"))
         qemuCapsClear(flags, QEMU_CAPS_NO_ACPI);

     /* qemuCapsExtractDeviceStr will only set additional flags if qemu


Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]