From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Yang, Sheng" Subject: Re: [PATCH 6/6] kvm: qemu: Enable EPT support for real mode Date: Thu, 24 Apr 2008 15:57:51 +0800 Message-ID: <200804241557.51359.sheng.yang@intel.com> References: <200804181730.16318.sheng.yang@intel.com> <200804241529.57878.sheng.yang@intel.com> <481038D1.4030809@qumranet.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel@lists.sourceforge.net To: Avi Kivity Return-path: In-Reply-To: <481038D1.4030809@qumranet.com> Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces@lists.sourceforge.net Errors-To: kvm-devel-bounces@lists.sourceforge.net List-Id: kvm.vger.kernel.org On Thursday 24 April 2008 15:37:53 Avi Kivity wrote: > Yang, Sheng wrote: > >> Why not use ept_identity_pagetable != NULL to encode > >> ept_identity_pagetable_done? > > > > ept_identity_pagetable_done was used to indicate if the pagetable was > > setted up, and ept_identity_pagetable was used to indicate if the page > > used for pagetable was allocated... I don't want to run > > alloc_identity_pagetable() again and again and again... Another method is > > read several bits at the front of page to tell if the pagetable was > > setted up, but somehow tricky... > > No, better to avoid tricks. But rmode_tss is only allocated once, so if > you unify the allocations, the identity table will also only be > allocated once. But set_tss_addr() is a x86_ops, if the identity mapping allocation using the same approach, another null function should be added into SVM side, but this thing is VMX specific one... -- Thanks Yang, Sheng ------------------------------------------------------------------------- This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone