On Fri, Dec 21, 2012 at 10:20 PM, Ján Tomko <jtomko@redhat.com> wrote:
On 12/21/12 15:07, Li Zhang wrote:
>
>
> On Fri, Dec 21, 2012 at 9:38 PM, Ján Tomko <jtomko@redhat.com
> <mailto:jtomko@redhat.com>> wrote:
>
>     In commit c4bbaaf8, caps->arch was checked uninitialized, rendering the
>     whole check useless.
>
>     This patch moves the conditional setting of QEMU_CAPS_NO_ACPI to
>     qemuCapsInitQMP, and removes the no longer needed exception for S390.
>
>
> Will it get QEMU_CAPS_NO_ACPI capability by QMP if S390 doesn't support
> ACPI?
> I test it on PPC, it won't get this capability by QMP.
>
> Thanks.
>

No, QEMU_CAPS_NO_ACPI capability will only be set on i686 and x86_64.


So I think there shouldn't be the conditional setting in qemuCapsInitQMP.
It doesn't need to clear this capability in this place.


Jan



--

Best Regards
-Li