From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gleb Natapov Subject: Re: [PATCH v2 4/4] nested vmx: use a list to store the launched vmcs12 for L1 VMM Date: Wed, 28 Nov 2012 13:27:59 +0200 Message-ID: <20121128112757.GA928@redhat.com> References: <1353559919-29439-1-git-send-email-dongxiao.xu@intel.com> <1353559919-29439-5-git-send-email-dongxiao.xu@intel.com> <20121128002908.GD8295@amt.cnet> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Dongxiao Xu , kvm@vger.kernel.org To: Marcelo Tosatti Return-path: Received: from mx1.redhat.com ([209.132.183.28]:7083 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754236Ab2K1L2D (ORCPT ); Wed, 28 Nov 2012 06:28:03 -0500 Content-Disposition: inline In-Reply-To: <20121128002908.GD8295@amt.cnet> Sender: kvm-owner@vger.kernel.org List-ID: On Tue, Nov 27, 2012 at 10:29:08PM -0200, Marcelo Tosatti wrote: > On Thu, Nov 22, 2012 at 12:51:59PM +0800, Dongxiao Xu wrote: > > The launch state is not a member in the VMCS area, use a separate > > variable (list) to store it instead. > > > > Signed-off-by: Dongxiao Xu > > 1. What is the problem with keeping launched state in the VMCS? > Assuming there is a positive answer to the above: > > 2. Don't you have to change VMCS ID? > > 3. Can't it be kept somewhere else other than a list? Current scheme > allows guest to allocate unlimited amounts of host memory. > > 4. What is the state of migration / nested vmx again? If vmcs12 is > migrated, this means launched state is not migrated anymore. > > Patches 1-3 seem fine. According to Dongxiao they are slowing down nested guest by 4%. -- Gleb.