From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH v3 06/11] KVM: MMU: cleanup FNAME(invlpg) Date: Wed, 14 Sep 2011 12:57:12 +0300 Message-ID: <4E707A78.4050603@redhat.com> References: <4E5C4C20.3000403@cn.fujitsu.com> <4E5C4CA3.6030808@cn.fujitsu.com> <4E6F37B4.2080405@redhat.com> <4E6FA189.7080607@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Marcelo Tosatti , LKML , KVM To: Xiao Guangrong Return-path: In-Reply-To: <4E6FA189.7080607@cn.fujitsu.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 09/13/2011 09:31 PM, Xiao Guangrong wrote: > >> @@ -675,36 +684,20 @@ static void FNAME(invlpg)(struct kvm_vcpu *vcpu, gva_t gva) > >> > >> sp = page_header(__pa(sptep)); > >> if (is_last_spte(*sptep, level)) { > >> - int offset, shift; > >> - > >> if (!sp->unsync) > >> break; > >> > >> - shift = PAGE_SHIFT - > >> - (PT_LEVEL_BITS - PT64_LEVEL_BITS) * level; > >> - offset = sp->role.quadrant<< shift; > >> - > >> - pte_gpa = (sp->gfn<< PAGE_SHIFT) + offset; > >> + pte_gpa = FNAME(get_first_pte_gpa)(sp); > > > > > > Here is can be used for L2 - I think we can use 2MB host pages to back 4MB guest mappings. > > > > Only unsync shadow page is fetched here, and its level is always 1. > Right. -- error compiling committee.c: too many arguments to function