From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45053) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yd2TU-0003Za-C5 for qemu-devel@nongnu.org; Tue, 31 Mar 2015 16:09:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yd2TP-0003Ou-8B for qemu-devel@nongnu.org; Tue, 31 Mar 2015 16:09:24 -0400 Received: from e06smtp15.uk.ibm.com ([195.75.94.111]:58063) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yd2TO-0003No-Un for qemu-devel@nongnu.org; Tue, 31 Mar 2015 16:09:19 -0400 Received: from /spool/local by e06smtp15.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 31 Mar 2015 21:09:15 +0100 Received: from b06cxnps3074.portsmouth.uk.ibm.com (d06relay09.portsmouth.uk.ibm.com [9.149.109.194]) by d06dlp03.portsmouth.uk.ibm.com (Postfix) with ESMTP id BF2891B08069 for ; Tue, 31 Mar 2015 21:09:40 +0100 (BST) Received: from d06av10.portsmouth.uk.ibm.com (d06av10.portsmouth.uk.ibm.com [9.149.37.251]) by b06cxnps3074.portsmouth.uk.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id t2VK9CK49568698 for ; Tue, 31 Mar 2015 20:09:12 GMT Received: from d06av10.portsmouth.uk.ibm.com (localhost [127.0.0.1]) by d06av10.portsmouth.uk.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id t2VK9BTv004743 for ; Tue, 31 Mar 2015 14:09:12 -0600 Date: Tue, 31 Mar 2015 22:09:09 +0200 From: Michael Mueller Message-ID: <20150331220909.4410e8c0@bee> In-Reply-To: <20150331183526.GJ7031@thinpad.lan.raisama.net> References: <1427725708-52100-1-git-send-email-mimu@linux.vnet.ibm.com> <1427725708-52100-12-git-send-email-mimu@linux.vnet.ibm.com> <20150331183526.GJ7031@thinpad.lan.raisama.net> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4 11/15] target-s390x: New QMP command query-cpu-model List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Habkost Cc: linux-s390@vger.kernel.org, kvm@vger.kernel.org, Gleb Natapov , linux-kernel@vger.kernel.org, Alexander Graf , qemu-devel@nongnu.org, Christian Borntraeger , Daniel Hansel , "Jason J. Herne" , Cornelia Huck , Paolo Bonzini , Andreas Faerber , Richard Henderson On Tue, 31 Mar 2015 15:35:26 -0300 Eduardo Habkost wrote: > On Mon, Mar 30, 2015 at 04:28:24PM +0200, Michael Mueller wrote: > > This patch implements a new QMP request named 'query-cpu-model'. > > It returns the cpu model of cpu 0 and its backing accelerator. > > > > request: > > {"execute" : "query-cpu-model" } > > > > answer: > > {"return" : {"name": "2827-ga2", "accel": "kvm" }} > > > > Alias names are resolved to their respective machine type and GA names > > already during cpu instantiation. Thus, also a cpu model like 'host' > > which is implemented as alias will return its normalized cpu model name. > > > > Furthermore the patch implements the following function: > > > > - s390_cpu_models_used(), returns true if S390 cpu models are in use > > > > Signed-off-by: Michael Mueller > > --- > [...] > > +static inline char *strdup_s390_cpu_name(S390CPUClass *cc) > > +{ > > + return g_strdup_printf("%04x-ga%u", cc->proc.type, cc->mach.ga); > > +} > > How exactly is this information going to be used by clients? If getting > the correct type and ga values is important for them, maybe you could > add them as integer fields, instead of requiring clients to parse the > CPU model name? The consumer don't need to parse the name, it is just important for them to have distinctive names that correlate with the names returned by query-cpu-definitions. Once the name of an active guest is known, e.g. ("2827-ga2", "kvm") a potential migration target can be verified, i.e. its query-cpu-definitions answer for "kvm" has to contain "2827-ga2" with the attribute runnable set to true. With that mechanism also the largest common denominator can be calculated. That model will be used then. I also changed the above mentioned routine to map the cpu model none case: static inline char *strdup_s390_cpu_name(S390CPUClass *cc) { if (cpuid(cc->proc)) { return g_strdup_printf("%04x-ga%u", cc->proc.type, cc->mach.ga); } else { return g_strdup("none"); } } This implicitly will fail a comparison for cpu model ("none", "kvm") as that will never be part of the query-cpu-definitions answer. I actually applied a couple of your suggestions like: - test for NULL skipped after strdup_s390_cpu_name() - strdup_s390_cpu_name() now also handles none cpu model case - omit runnable and is-default field from query-cpu-definitions answer when they are false - global variable cpu_models_used dropped - function s390_cpu_models_used() dropped - routine query-cpu-definitions has a single code path now Only the integration of the ACCEL_ID with the cpu state in cpu_generic_init() and the change for the query-cpus implementation is under construction. I hope to resend the patches by tomorrow evening. Thanks, Michael >