From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53872) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZCjNi-0006Hh-Ue for qemu-devel@nongnu.org; Wed, 08 Jul 2015 03:02:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZCjNf-0002kt-OE for qemu-devel@nongnu.org; Wed, 08 Jul 2015 03:02:58 -0400 Received: from mail-pd0-f172.google.com ([209.85.192.172]:36697) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZCjNf-0002km-IO for qemu-devel@nongnu.org; Wed, 08 Jul 2015 03:02:55 -0400 Received: by pddu5 with SMTP id u5so52944537pdd.3 for ; Wed, 08 Jul 2015 00:02:54 -0700 (PDT) References: <1436327021-14744-1-git-send-email-david@gibson.dropbear.id.au> <559CA2B4.3090303@ozlabs.ru> <20150708053708.GN17857@voom.redhat.com> <559CC5DB.2030407@ozlabs.ru> <20150708064518.GO17857@voom.redhat.com> From: Alexey Kardashevskiy Message-ID: <559CCB15.3070806@ozlabs.ru> Date: Wed, 8 Jul 2015 17:02:45 +1000 MIME-Version: 1.0 In-Reply-To: <20150708064518.GO17857@voom.redhat.com> Content-Type: text/plain; charset=koi8-r; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Gibson Cc: lvivier@redhat.com, thuth@redhat.com, qemu-devel@nongnu.org, agraf@suse.de, qemu-ppc@nongnu.org, abologna@redhat.com, mdroth@linuc.vnet.ibm.com, afaerber@suse.de On 07/08/2015 04:45 PM, David Gibson wrote: > On Wed, Jul 08, 2015 at 04:40:27PM +1000, Alexey Kardashevskiy wrote: >> On 07/08/2015 03:37 PM, David Gibson wrote: >>> On Wed, Jul 08, 2015 at 02:10:28PM +1000, Alexey Kardashevskiy wrote: >>>> On 07/08/2015 01:43 PM, David Gibson wrote: >>>>> From: Andrea Bolognani >>>>> >>>>> Add a missing PVR value for the POWER8E v2.1 CPU. Information taken >>>> >from the kernel cputable. >>>>> >>>>> Signed-off-by: Andrea Bolognani >>>>> Signed-off-by: David Gibson >>>>> --- >>>>> target-ppc/cpu-models.c | 4 +++- >>>>> target-ppc/cpu-models.h | 1 + >>>>> 2 files changed, 4 insertions(+), 1 deletion(-) >>>>> >>>>> Alex, >>>>> >>>>> Not sure if this counts as a bugfix which can be merged now we're in >>>>> the hard freeze. The lack of it does mean we can't work on one of our >>>>> dev machines which has this CPU. >>>> >>>> >>>> Why is this a stopper? We stopped bothering with exact PVRs some time ago >>>> and -cpu POWER8 or -cpu host still work. >>> >>> Andrea, can you clarify? >>> >>> I think it's because libvirt likes to specify a specific CPU - and if >>> it gets the new PVR from the host, qemu won't understand it. >> >> >> A specific CPU in this case is "POWER8", I added this specifically for >> libvirt (to allow migration between all versions of POWER8), it should not >> use versioned CPUs and it does not in powerkvm. > > Uh.. won't that make qemu attempt to set a specific PVR, though - > which will fail with recent KVM if it's not *exactly* the same as the > host PVR. The "POWER8" CPU class is created dynamically (like the "host" CPU class) and has the actual host PVR so setting it to KVM cannot fail. When TCG, the class is not registered and alias is used instead. >> There is actually a patch to make this PVR masking/subclassing nicer, I >> replied to it with adding you in cc:, please have a look. Andreas Faerber >> had objections which I did not really grasp then. >> >> > -- Alexey