lkml.org 
[lkml]   [2022]   [Aug]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.18 0684/1095] KVM: Dont set Accessed/Dirty bits for ZERO_PAGE
    Date
    From: Sean Christopherson <seanjc@google.com>

    [ Upstream commit a1040b0d42acf69bb4f6dbdc54c2dcd78eea1de5 ]

    Don't set Accessed/Dirty bits for a struct page with PG_reserved set,
    i.e. don't set A/D bits for the ZERO_PAGE. The ZERO_PAGE (or pages
    depending on the architecture) should obviously never be written, and
    similarly there's no point in marking it accessed as the page will never
    be swapped out or reclaimed. The comment in page-flags.h is quite clear
    that PG_reserved pages should be managed only by their owner, and
    strictly following that mandate also simplifies KVM's logic.

    Fixes: 7df003c85218 ("KVM: fix overflow of zero page refcount with ksm running")
    Signed-off-by: Sean Christopherson <seanjc@google.com>
    Message-Id: <20220429010416.2788472-4-seanjc@google.com>
    Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    virt/kvm/kvm_main.c | 16 ++++++++++++++--
    1 file changed, 14 insertions(+), 2 deletions(-)

    diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
    index 6c8295e76548..843396ed4ad3 100644
    --- a/virt/kvm/kvm_main.c
    +++ b/virt/kvm/kvm_main.c
    @@ -2852,16 +2852,28 @@ void kvm_release_pfn_dirty(kvm_pfn_t pfn)
    }
    EXPORT_SYMBOL_GPL(kvm_release_pfn_dirty);

    +static bool kvm_is_ad_tracked_pfn(kvm_pfn_t pfn)
    +{
    + if (!pfn_valid(pfn))
    + return false;
    +
    + /*
    + * Per page-flags.h, pages tagged PG_reserved "should in general not be
    + * touched (e.g. set dirty) except by its owner".
    + */
    + return !PageReserved(pfn_to_page(pfn));
    +}
    +
    void kvm_set_pfn_dirty(kvm_pfn_t pfn)
    {
    - if (!kvm_is_reserved_pfn(pfn) && !kvm_is_zone_device_pfn(pfn))
    + if (kvm_is_ad_tracked_pfn(pfn))
    SetPageDirty(pfn_to_page(pfn));
    }
    EXPORT_SYMBOL_GPL(kvm_set_pfn_dirty);

    void kvm_set_pfn_accessed(kvm_pfn_t pfn)
    {
    - if (!kvm_is_reserved_pfn(pfn) && !kvm_is_zone_device_pfn(pfn))
    + if (kvm_is_ad_tracked_pfn(pfn))
    mark_page_accessed(pfn_to_page(pfn));
    }
    EXPORT_SYMBOL_GPL(kvm_set_pfn_accessed);
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-08-15 23:50    [W:2.323 / U:1.628 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site