From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Graf Subject: Re: [PATCH 3/3] Add svm cpuid features Date: Thu, 16 Sep 2010 16:28:01 +0200 Message-ID: <4C922971.2020408@suse.de> 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> <20100916142628.GC7972@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Avi Kivity , Marcelo Tosatti , "kvm@vger.kernel.org" , "qemu-devel@nongnu.org" To: "Roedel, Joerg" Return-path: Received: from cantor2.suse.de ([195.135.220.15]:43837 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755287Ab0IPO2D (ORCPT ); Thu, 16 Sep 2010 10:28:03 -0400 In-Reply-To: <20100916142628.GC7972@amd.com> Sender: kvm-owner@vger.kernel.org List-ID: Roedel, Joerg wrote: > 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. > speed == -cpu host :). If the performance increase is significant, we can always add a new cpu type for a cpu that does support those flags so people can use it there and still be migration safe. In fact, that would even benefit you guys as you'd sell more new machines for speed ;). Alex From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=53417 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OwFRb-0005pu-3e for qemu-devel@nongnu.org; Thu, 16 Sep 2010 10:28:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OwFRU-0002tq-7s for qemu-devel@nongnu.org; Thu, 16 Sep 2010 10:28:10 -0400 Received: from cantor2.suse.de ([195.135.220.15]:43836 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OwFRU-0002tX-0R for qemu-devel@nongnu.org; Thu, 16 Sep 2010 10:28:04 -0400 Message-ID: <4C922971.2020408@suse.de> Date: Thu, 16 Sep 2010 16:28:01 +0200 From: Alexander Graf MIME-Version: 1.0 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> <20100916142628.GC7972@amd.com> In-Reply-To: <20100916142628.GC7972@amd.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit 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: "Roedel, Joerg" Cc: Marcelo Tosatti , Avi Kivity , "kvm@vger.kernel.org" , "qemu-devel@nongnu.org" Roedel, Joerg wrote: > 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. > speed == -cpu host :). If the performance increase is significant, we can always add a new cpu type for a cpu that does support those flags so people can use it there and still be migration safe. In fact, that would even benefit you guys as you'd sell more new machines for speed ;). Alex