From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54375) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aijir-0007R0-VI for qemu-devel@nongnu.org; Wed, 23 Mar 2016 10:25:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aijiq-0000X0-Sf for qemu-devel@nongnu.org; Wed, 23 Mar 2016 10:25:21 -0400 Date: Wed, 23 Mar 2016 22:25:00 +0800 From: Peter Xu Message-ID: <20160323142500.GU28183@pxdev.xzpeter.org> References: <1458271654-23706-1-git-send-email-peterx@redhat.com> <1458271654-23706-2-git-send-email-peterx@redhat.com> <87zitqjrhj.fsf@blackfin.pond.sub.org> <87vb4eicdu.fsf@blackfin.pond.sub.org> <20160323025819.GD28183@pxdev.xzpeter.org> <87shzhbktm.fsf@blackfin.pond.sub.org> <20160323114830.GR28183@pxdev.xzpeter.org> <8737rh8jvj.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <8737rh8jvj.fsf@blackfin.pond.sub.org> Subject: Re: [Qemu-devel] [PATCH v5 1/5] arm: qmp: add GICCapability struct List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: wei@redhat.com, peter.maydell@linaro.org, drjones@redhat.com, mdroth@linux.vnet.ibm.com, qemu-devel@nongnu.org, abologna@redhat.com, qemu-arm@nongnu.org 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 ... If we specify other explicit GIC version rather than the above, QEMU will not be able to boot. """ Thanks. -- peterx