[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct |
Date: |
Wed, 23 Mar 2016 16:17:25 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Peter Xu <address@hidden> writes:
> On Wed, Mar 23, 2016 at 01:21:52PM +0100, Markus Armbruster wrote:
>> Can you tell me what a management application is supposed to do with the
>> information returned by query-gic-capabilities? Not just in general
>> terms, like "using this information, libvirt can warn the user during
>> configuration of guests when specified GIC device type is not supported,
>> but specifics. Something like "-frobnicate mutter=mumble won't work
>> unless query-gic-capabilities reports emulated version 2 is supported"
>> for every piece of configuration that should be vetted against
>> query-gic-capabilities.
>
> I suppose that won't be a very big problem since possibly only
> libvirt will use it... I agree that it's better to explain it more
> clearly though. How about adding these lines into patch 1 commit
> message:
>
> """
> For example, if we got the query result:
>
> {"return": {"capabilities":
> [ {"emulated": false, "version": 3, "kernel": true},
> {"emulated": true, "version": 2, "kernel": false} ] } }
>
> Then it means that we support emulated GIC version 2 using:
>
> qemu-system-aarch64 -M virt,gic-version=2 ...
>
> or kvm-accelerated GIC version 3 using:
>
> qemu-system-aarch64 -enable-kvm -M virt,gic-version=3 ...
I'd say "qemu-system-aarch64 -M virt,accel=kvm,gic=version=3"
> If we specify other explicit GIC version rather than the above, QEMU
> will not be able to boot.
> """
Works for me.
- [Qemu-devel] [PATCH v5 0/5] ARM: add query-gic-capabilities SMP command, Peter Xu, 2016/03/17
- [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Peter Xu, 2016/03/17
- Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Markus Armbruster, 2016/03/22
- Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Markus Armbruster, 2016/03/22
- Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Peter Xu, 2016/03/22
- Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Markus Armbruster, 2016/03/23
- Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Peter Xu, 2016/03/23
- Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Markus Armbruster, 2016/03/23
- Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Peter Xu, 2016/03/23
- Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Peter Xu, 2016/03/23
Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct, Eric Blake, 2016/03/22
[Qemu-devel] [PATCH v5 2/5] arm: qmp: add query-gic-capabilities interface, Peter Xu, 2016/03/17