]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
x86, vmi: put a missing paravirt_release_pmd in pgd_dtor
authorAlok Kataria <akataria@vmware.com>
Fri, 6 Feb 2009 18:29:35 +0000 (10:29 -0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 17 Feb 2009 17:46:18 +0000 (09:46 -0800)
commit2368d2cc93bf8e4a050f78b4c1d90e2a3746b177
tree3f3fe9251608c36b28e4b34f7fd7292152c8b710
parent3c5badb6ff029a53c281a2bab62ff9d12a04725b
x86, vmi: put a missing paravirt_release_pmd in pgd_dtor

commit 55a8ba4b7f76bebd7e8ce3f74c04b140627a1bad upstream.

Commit 6194ba6ff6ccf8d5c54c857600843c67aa82c407 ("x86: don't special-case
pmd allocations as much") made changes to the way we handle pmd allocations,
and while doing that it dropped a call to  paravirt_release_pd on the
pgd page from the pgd_dtor code path.

As a result of this missing release, the hypervisor is now unaware of the
pgd page being freed, and as a result it ends up tracking this page as a
page table page.

After this the guest may start using the same page for other purposes, and
depending on what use the page is put to, it may result in various performance
and/or functional issues ( hangs, reboots).

Since this release is only required for VMI, I now release the pgd page from
the (vmi)_pgd_free hook.

Signed-off-by: Alok N Kataria <akataria@vmware.com>
Acked-by: Jeremy Fitzhardinge <jeremy@goop.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
arch/x86/kernel/vmi_32.c