From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 4/8] KVM: MMU: Add EPT support Date: Sun, 27 Apr 2008 12:35:00 +0300 Message-ID: <481448C4.6000104@qumranet.com> References: <200804251625.57245.sheng.yang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel To: "Yang, Sheng" Return-path: In-Reply-To: <200804251625.57245.sheng.yang@intel.com> 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 Yang, Sheng wrote: > From 239f38236196c2321989c64d7c61ff28490b3f00 Mon Sep 17 00:00:00 2001 > From: Sheng Yang > Date: Fri, 25 Apr 2008 21:13:50 +0800 > Subject: [PATCH 4/8] KVM: MMU: Add EPT support > > Enable kvm_set_spte() to generate EPT entries. > > > @@ -1155,7 +1178,7 @@ static int __direct_map(struct kvm_vcpu *vcpu, gpa_t v, > int write, > } > > table[index] = __pa(new_table->spt) | PT_PRESENT_MASK > - | PT_WRITABLE_MASK | PT_USER_MASK; > + | PT_WRITABLE_MASK | shadow_user_mask; > } > Shouldn't we have shadow_x_mask here as well? [and for non-EPT, shadow_accessed_mask and shadow_dirty_mask, but that's a different story]. -- error compiling committee.c: too many arguments to function ------------------------------------------------------------------------- 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