From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=46413 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Osz29-0006Qv-Qa for qemu-devel@nongnu.org; Tue, 07 Sep 2010 10:20:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1Osz24-00010J-L4 for qemu-devel@nongnu.org; Tue, 07 Sep 2010 10:20:21 -0400 Received: from va3ehsobe006.messaging.microsoft.com ([216.32.180.16]:53296 helo=VA3EHSOBE008.bigfish.com) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1Osz24-00010F-Hu for qemu-devel@nongnu.org; Tue, 07 Sep 2010 10:20:20 -0400 Date: Tue, 7 Sep 2010 16:18:35 +0200 From: "Roedel, Joerg" Subject: Re: [Qemu-devel] [PATCH 1/3] qemu-kvm: Invert svm-flag setting logic Message-ID: <20100907141835.GD2177@amd.com> References: <1283858843-21330-1-git-send-email-joerg.roedel@amd.com> <1283858843-21330-2-git-send-email-joerg.roedel@amd.com> <4C864365.30301@redhat.com> <20100907141216.GA2177@amd.com> <0B0F0DB3-0948-4735-97CA-64C432E730C2@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <0B0F0DB3-0948-4735-97CA-64C432E730C2@suse.de> 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 Tue, Sep 07, 2010 at 10:14:46AM -0400, Alexander Graf wrote: > > On 07.09.2010, at 16:12, Roedel, Joerg wrote: > > > On Tue, Sep 07, 2010 at 09:51:33AM -0400, Avi Kivity wrote: > >> On 09/07/2010 03:23 PM, Alexander Graf wrote: > >>> > >>> I think we should get rid of kvm_nested and -enable-nesting. Instead, we should enable the SVM bit in the "host" and "qemu64" cpu types, but not in "kvm64". This way users are safe to not use nested svm, but can choose to do so if they like. > >>> > >>> Also, it should be possible to do something like -cpu kvm64,flags=+svm. Without having to mess with -enable-nesting. > >> > >> I agree, -enable-nesting is redundant with -cpu ,+svm. > > > > This patchset makes -enable-nesting pratically a synonym for -cpu ,+svm. > > So we can safely remove -enable-nesting in the future. > > Why not just not introduce it? :) It's always hard to get rid of legacy. Because its already there? The patches are against qemu-kvm. -- 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