diff options
author | Avi Kivity <avi@qumranet.com> | 2007-01-06 01:36:52 +0100 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.osdl.org> | 2007-01-06 08:55:27 +0100 |
commit | f51234c2cd3ab8bed836e09686e27877e1b55f2a (patch) | |
tree | fbdfbfaf5d7b9c80a105d15b28d600653661572b /drivers/kvm/mmu.c | |
parent | [PATCH] KVM: MMU: Treat user-mode faults as a hint that a page is no longer a... (diff) | |
download | linux-f51234c2cd3ab8bed836e09686e27877e1b55f2a.tar.xz linux-f51234c2cd3ab8bed836e09686e27877e1b55f2a.zip |
[PATCH] KVM: MMU: Free pages on kvm destruction
Because mmu pages have attached rmap and parent pte chain structures, we need
to zap them before freeing so the attached structures are freed.
Signed-off-by: Avi Kivity <avi@qumranet.com>
Acked-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/kvm/mmu.c')
-rw-r--r-- | drivers/kvm/mmu.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/kvm/mmu.c b/drivers/kvm/mmu.c index f16321498093..0bd2a19709ce 100644 --- a/drivers/kvm/mmu.c +++ b/drivers/kvm/mmu.c @@ -1065,9 +1065,14 @@ EXPORT_SYMBOL_GPL(kvm_mmu_free_some_pages); static void free_mmu_pages(struct kvm_vcpu *vcpu) { - while (!list_empty(&vcpu->free_pages)) { - struct kvm_mmu_page *page; + struct kvm_mmu_page *page; + while (!list_empty(&vcpu->kvm->active_mmu_pages)) { + page = container_of(vcpu->kvm->active_mmu_pages.next, + struct kvm_mmu_page, link); + kvm_mmu_zap_page(vcpu, page); + } + while (!list_empty(&vcpu->free_pages)) { page = list_entry(vcpu->free_pages.next, struct kvm_mmu_page, link); list_del(&page->link); |