From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Roedel, Joerg" Subject: Re: drop -enable-nesting Date: Tue, 31 May 2011 11:06:17 +0200 Message-ID: <20110531090617.GC7855@amd.com> References: <4DE0B4B1.1000407@web.de> <20110530081847.GC27062@amd.com> <4DE3A3EC.80201@siemens.com> <20110530143846.GA2957@fermat.math.technion.ac.il> <4DE3B1E2.7020006@siemens.com> <20110530151020.GB7855@amd.com> <4DE3B4A5.6050509@siemens.com> <4DE3B572.6030105@redhat.com> <20110531084429.GA2671@redhat.com> <4DE4ADA8.5000503@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: "Daniel P. Berrange" , Jan Kiszka , Nadav Har'El , john cooper , "qemu-devel@nongnu.org" , Anthony Liguori , kvm To: Avi Kivity Return-path: Received: from ch1ehsobe002.messaging.microsoft.com ([216.32.181.182]:37466 "EHLO CH1EHSOBE014.bigfish.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1758452Ab1EaJGI (ORCPT ); Tue, 31 May 2011 05:06:08 -0400 Content-Disposition: inline In-Reply-To: <4DE4ADA8.5000503@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Tue, May 31, 2011 at 04:58:16AM -0400, Avi Kivity wrote: > On 05/31/2011 11:44 AM, Daniel P. Berrange wrote: > > > I think it's safe to drop -enable-nesting immediately. Dan, does > > > libvirt make use of it? > > > > Yes, but it should be safe to drop it. Currently, if the user specifies > > a CPU with the 'svm' flag present in libvirt guest XML, then we will > > pass args '-cpu ....+svm -enable-nesting'. So if we drop --enable-nesting, > > then libvirt will simply omit it and everything should still work because > > we have still got +svm set. > > But qemu will complain about an option it can't parse. The best choice is probably to keep the option and make it a nop for the lifetime of qemu-kvm. Optionally qemu-kvm can print a warning about the deprecated option. 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