From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=52830 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OwFPi-0004Au-CN for qemu-devel@nongnu.org; Thu, 16 Sep 2010 10:26:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OwFPc-0002Rv-E3 for qemu-devel@nongnu.org; Thu, 16 Sep 2010 10:26:14 -0400 Received: from va3ehsobe001.messaging.microsoft.com ([216.32.180.11]:50152 helo=VA3EHSOBE001.bigfish.com) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OwFPc-0002Rp-Bk for qemu-devel@nongnu.org; Thu, 16 Sep 2010 10:26:08 -0400 Date: Thu, 16 Sep 2010 16:26:29 +0200 From: "Roedel, Joerg" 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" Content-Disposition: inline In-Reply-To: <4C9226E5.4080906@suse.de> Subject: [Qemu-devel] Re: [PATCH 3/3] Add svm cpuid features List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: Marcelo Tosatti , Avi Kivity , "kvm@vger.kernel.org" , "qemu-devel@nongnu.org" 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