From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lai Jiangshan Subject: Re: [PATCH] kvm,async_pf: add missing kvm_async_pf_hash_reset() Date: Mon, 21 Feb 2011 11:24:50 +0800 Message-ID: <4D61DB02.6060009@cn.fujitsu.com> References: <4D61DA3A.7040501@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit To: "kvm@vger.kernel.org" , Avi Kivity , Marcelo Tosatti , Gleb Natapov Return-path: Received: from cn.fujitsu.com ([222.73.24.84]:65466 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1750819Ab1BUDYZ (ORCPT ); Sun, 20 Feb 2011 22:24:25 -0500 In-Reply-To: <4D61DA3A.7040501@cn.fujitsu.com> Sender: kvm-owner@vger.kernel.org List-ID: On 02/21/2011 11:21 AM, Lai Jiangshan wrote: > > The hash array of async gfns may still contain some left gfns after > kvm_clear_async_pf_completion_queue() called, need to clear them. > sorry. Signed-off-by: Lai Jiangshan > diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c > index 17af71d..dae2d15 100644 > --- a/arch/x86/kvm/x86.c > +++ b/arch/x86/kvm/x86.c > @@ -525,8 +525,10 @@ int kvm_set_cr0(struct kvm_vcpu *vcpu, unsigned long cr0) > > kvm_x86_ops->set_cr0(vcpu, cr0); > > - if ((cr0 ^ old_cr0) & X86_CR0_PG) > + if ((cr0 ^ old_cr0) & X86_CR0_PG) { > kvm_clear_async_pf_completion_queue(vcpu); > + kvm_async_pf_hash_reset(vcpu); > + } > > if ((cr0 ^ old_cr0) & update_bits) > kvm_mmu_reset_context(vcpu); > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >