From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gleb Natapov Subject: Re: [PATCH] kvm,async_pf: add missing kvm_async_pf_hash_reset() Date: Wed, 23 Feb 2011 15:22:23 +0200 Message-ID: <20110223132223.GA25876@redhat.com> References: <4D61DA3A.7040501@cn.fujitsu.com> <4D61DB02.6060009@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "kvm@vger.kernel.org" , Avi Kivity , Marcelo Tosatti To: Lai Jiangshan Return-path: Received: from mx1.redhat.com ([209.132.183.28]:1771 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754101Ab1BWNW1 (ORCPT ); Wed, 23 Feb 2011 08:22:27 -0500 Content-Disposition: inline In-Reply-To: <4D61DB02.6060009@cn.fujitsu.com> Sender: kvm-owner@vger.kernel.org List-ID: On Mon, Feb 21, 2011 at 11:24:50AM +0800, Lai Jiangshan wrote: > 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 > Acked-by: Gleb Natapov > > 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 > > -- Gleb.