Re: [PATCH] KVM: Fix srcu struct leakage

[Avi Kivity <avi@redhat.com>]

On 11/09/2010 03:53 PM, Marcelo Tosatti wrote:
> On Tue, Nov 09, 2010 at 12:41:26PM +0100, Jan Kiszka wrote:
> >  Clean up the srcu struct on vm destruction.
> >
> >  Signed-off-by: Jan Kiszka<jan.kiszka@siemens.com>
> >  ---
> >   virt/kvm/kvm_main.c |    1 +
> >   1 files changed, 1 insertions(+), 0 deletions(-)
> >
> >  diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
> >  index 4111a4b..6ec58d1 100644
> >  --- a/virt/kvm/kvm_main.c
> >  +++ b/virt/kvm/kvm_main.c
> >  @@ -513,6 +513,7 @@ static void kvm_destroy_vm(struct kvm *kvm)
> >   #else
> >   	kvm_arch_flush_shadow(kvm);
> >   #endif
> >  +	cleanup_srcu_struct(&kvm->srcu);
> >   	kvm_arch_destroy_vm(kvm);
> >   	hardware_disable_all();
> >   	mmdrop(mm);
> >  -- 
> >  1.7.1
>
> kvm_arch_destroy_vm does it.

True.  Dropped.

-- 
error compiling committee.c: too many arguments to function