From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Roedel, Joerg" Subject: Re: [PATCH 3/3] Add svm cpuid features Date: Thu, 16 Sep 2010 16:26:29 +0200 Message-ID: <20100916142628.GC7972@amd.com> References: <1284479530-4748-1-git-send-email-joerg.roedel@amd.com> <1284479530-4748-4-git-send-email-joerg.roedel@amd.com> <4C922478.7010703@redhat.com> <20100916141259.GA7972@amd.com> <4C9226E5.4080906@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: Avi Kivity , Marcelo Tosatti , "kvm@vger.kernel.org" , "qemu-devel@nongnu.org" To: Alexander Graf Return-path: Received: from va3ehsobe001.messaging.microsoft.com ([216.32.180.11]:50164 "EHLO VA3EHSOBE001.bigfish.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754455Ab0IPO0J (ORCPT ); Thu, 16 Sep 2010 10:26:09 -0400 Content-Disposition: inline In-Reply-To: <4C9226E5.4080906@suse.de> Sender: kvm-owner@vger.kernel.org List-ID: On Thu, Sep 16, 2010 at 10:17:09AM -0400, Alexander Graf wrote: > Roedel, Joerg wrote: > > On Thu, Sep 16, 2010 at 10:06:48AM -0400, Avi Kivity wrote: > > > >> On 09/14/2010 05:52 PM, Joerg Roedel wrote: > >> > >>> This patch adds the svm cpuid feature flags to the qemu > >>> intialization path. It also adds the svm features available > >>> on phenom to its cpu-definition and extends the host cpu > >>> type to support all svm features KVM can provide. > >>> > >>> > >>> > >> Does phenom really have vmcbclean? I thought it was a really new feature. > >> > > > > No, but we could emulate it on almost all hardware that has SVM. Its > > basically the same as with x2apic. > > > > -cpu is not about what we could emulate, but what the > hardware really is. Features not supported by the particular CPU do not > belong there. I'll remove it then. It would have been nice to have it there because it will improve performance of svm emulation. But if it is about emulating the real cpu then I'll just drop it. Joerg -- AMD Operating System Research Center Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach General Managers: Alberto Bozzo, Andrew Bowd Registration: Dornach, Landkr. Muenchen; Registerger. Muenchen, HRB Nr. 43632