Re: [PATCH] kvm,async_pf: add missing kvm_async_pf_hash_reset()

[Gleb Natapov <gleb@redhat.com>]

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 <laijs@cn.fujitsu.com>
> 
Acked-by: Gleb Natapov <gleb@redhat.com>

> > 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.