lkml.org 
[lkml]   [2009]   [Feb]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Subject[PARAVIRT/x86] BUGFIX: Put a missing paravirt_release_pmd in pgd_dtor
    From
    Date
    [PARAVIRT/x86] Put a missing paravirt_release_pmd in pgd_dtor.

    From: Alok N Kataria <akataria@vmware.com>

    The commit...
    -----------------------------
    commit 6194ba6ff6ccf8d5c54c857600843c67aa82c407
    Author: Jeremy Fitzhardinge <jeremy@goop.org>
    Date: Wed Jan 30 13:34:11 2008 +0100

    x86: don't special-case pmd allocations as much
    ------------------------------
    ...made changes to the way we handle pmd allocations, and while doing that
    it (accidently ?) dropped a call to paravirt_release_pd 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).

    The patch below adds a paravirt_release_pmd call for the PGD page.
    Patch on top of 2.6.29-rc3 (mainline-git).

    Signed-off-by: Alok N Kataria <akataria@vmware.com>
    Signed-off-by: Rohit Jain <rjain@vmware.com>
    Cc: Zachary Amsden <zach@vmware.com>
    Cc: Jeremy Fitzhardinge <jeremy@goop.org>
    Cc: stable@kernel.org
    ---

    arch/x86/mm/pgtable.c | 6 ++++++
    1 files changed, 6 insertions(+), 0 deletions(-)


    diff --git a/arch/x86/mm/pgtable.c b/arch/x86/mm/pgtable.c
    index 86f2ffc..c23cd7e 100644
    --- a/arch/x86/mm/pgtable.c
    +++ b/arch/x86/mm/pgtable.c
    @@ -89,6 +89,12 @@ static void pgd_dtor(pgd_t *pgd)
    {
    unsigned long flags; /* can be called from interrupt context */

    + if (PAGETABLE_LEVELS == 2 ||
    + (PAGETABLE_LEVELS == 3 && SHARED_KERNEL_PMD) ||
    + PAGETABLE_LEVELS == 4) {
    + paravirt_release_pmd(__pa(pgd) >> PAGE_SHIFT);
    + }
    +
    if (SHARED_KERNEL_PMD)
    return;





    \
     
     \ /
      Last update: 2009-02-06 03:05    [W:3.103 / U:0.648 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site