From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Tosatti Subject: Re: [PATCH v4 4/6] KVM: MMU: prefetch ptes when intercepted guest #PF Date: Mon, 5 Jul 2010 09:01:18 -0300 Message-ID: <20100705120118.GB15826@amt.cnet> References: <4C2C9DC0.8050607@cn.fujitsu.com> <4C2C9E45.1040304@cn.fujitsu.com> <20100702165432.GB25969@amt.cnet> <4C2EF00A.2090406@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Avi Kivity , LKML , KVM list To: Xiao Guangrong Return-path: Content-Disposition: inline In-Reply-To: <4C2EF00A.2090406@cn.fujitsu.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On Sat, Jul 03, 2010 at 04:08:42PM +0800, Xiao Guangrong wrote: > > > Marcelo Tosatti wrote: > > >> + > >> + if (is_rsvd_bits_set(vcpu, gpte, PT_PAGE_TABLE_LEVEL)) > >> + break; > > > > BTW, doesnt sync_page also lack reserved bit checking? (unrelated to > > this patch). > > > > I think it's not since if EPT is enabled, no unsync page exist, the sync page > path can't be trigged. :-) What i mean is, why the sync_page path does not require reserved bit checking? (EPT disabled). Just to clarify, this has nothing to do with your patch.