lkml.org 
[lkml]   [2022]   [Apr]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    Subject[PATCH v2 7/8] KVM: Do not pin pages tracked by gfn=>pfn caches
    From
    Put the reference to any struct page mapped/tracked by a gfn=>pfn cache
    upon inserting the pfn into its associated cache, as opposed to putting
    the reference only when the cache is done using the pfn. In other words,
    don't pin pages while they're in the cache. One of the major roles of
    the gfn=>pfn cache is to play nicely with invalidation events, i.e. it
    exists in large part so that KVM doesn't rely on pinning pages.

    Signed-off-by: Sean Christopherson <seanjc@google.com>
    ---
    virt/kvm/pfncache.c | 36 ++++++++++++++++++++----------------
    1 file changed, 20 insertions(+), 16 deletions(-)

    diff --git a/virt/kvm/pfncache.c b/virt/kvm/pfncache.c
    index b1665d0e6c32..3cb439b505b4 100644
    --- a/virt/kvm/pfncache.c
    +++ b/virt/kvm/pfncache.c
    @@ -95,20 +95,16 @@ bool kvm_gfn_to_pfn_cache_check(struct kvm *kvm, struct gfn_to_pfn_cache *gpc,
    }
    EXPORT_SYMBOL_GPL(kvm_gfn_to_pfn_cache_check);

    -static void gpc_release_pfn_and_khva(struct kvm *kvm, kvm_pfn_t pfn, void *khva)
    +static void gpc_unmap_khva(struct kvm *kvm, kvm_pfn_t pfn, void *khva)
    {
    - /* Unmap the old page if it was mapped before, and release it */
    - if (!is_error_noslot_pfn(pfn)) {
    - if (khva) {
    - if (pfn_valid(pfn))
    - kunmap(pfn_to_page(pfn));
    + /* Unmap the old pfn/page if it was mapped before. */
    + if (!is_error_noslot_pfn(pfn) && khva) {
    + if (pfn_valid(pfn))
    + kunmap(pfn_to_page(pfn));
    #ifdef CONFIG_HAS_IOMEM
    - else
    - memunmap(khva);
    + else
    + memunmap(khva);
    #endif
    - }
    -
    - kvm_release_pfn(pfn, false);
    }
    }

    @@ -147,10 +143,10 @@ static kvm_pfn_t hva_to_pfn_retry(struct kvm *kvm, struct gfn_to_pfn_cache *gpc)
    * Keep the mapping if the previous iteration reused
    * the existing mapping and didn't create a new one.
    */
    - if (new_khva == old_khva)
    - new_khva = NULL;
    + if (new_khva != old_khva)
    + gpc_unmap_khva(kvm, new_pfn, new_khva);

    - gpc_release_pfn_and_khva(kvm, new_pfn, new_khva);
    + kvm_release_pfn_clean(new_pfn);

    cond_resched();
    }
    @@ -222,6 +218,14 @@ static kvm_pfn_t hva_to_pfn_retry(struct kvm *kvm, struct gfn_to_pfn_cache *gpc)
    gpc->valid = true;
    gpc->pfn = new_pfn;
    gpc->khva = new_khva + (gpc->gpa & ~PAGE_MASK);
    +
    + /*
    + * Put the reference to the _new_ pfn. The pfn is now tracked by the
    + * cache and can be safely migrated, swapped, etc... as the cache will
    + * invalidate any mappings in response to relevant mmu_notifier events.
    + */
    + kvm_release_pfn_clean(new_pfn);
    +
    return 0;

    out_error:
    @@ -315,7 +319,7 @@ int kvm_gfn_to_pfn_cache_refresh(struct kvm *kvm, struct gfn_to_pfn_cache *gpc,
    write_unlock_irq(&gpc->lock);

    if (old_pfn != new_pfn)
    - gpc_release_pfn_and_khva(kvm, old_pfn, old_khva);
    + gpc_unmap_khva(kvm, old_pfn, old_khva);

    return ret;
    }
    @@ -342,7 +346,7 @@ void kvm_gfn_to_pfn_cache_unmap(struct kvm *kvm, struct gfn_to_pfn_cache *gpc)

    write_unlock_irq(&gpc->lock);

    - gpc_release_pfn_and_khva(kvm, old_pfn, old_khva);
    + gpc_unmap_khva(kvm, old_pfn, old_khva);
    }
    EXPORT_SYMBOL_GPL(kvm_gfn_to_pfn_cache_unmap);

    --
    2.36.0.rc2.479.g8af0fa9b8e-goog
    \
     
     \ /
      Last update: 2022-04-27 03:41    [W:5.156 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site