Commit fe2567eb authored by Peter Xu's avatar Peter Xu Committed by Andrew Morton

mm/shmem: allow uffd wr-protect none pte for file-backed mem

File-backed memory differs from anonymous memory in that even if the pte
is missing, the data could still resides either in the file or in
page/swap cache.  So when wr-protect a pte, we need to consider none ptes
too.

We do that by installing the uffd-wp pte markers when necessary.  So when
there's a future write to the pte, the fault handler will go the special
path to first fault-in the page as read-only, then report to userfaultfd
server with the wr-protect message.

On the other hand, when unprotecting a page, it's also possible that the
pte got unmapped but replaced by the special uffd-wp marker.  Then we'll
need to be able to recover from a uffd-wp pte marker into a none pte, so
that the next access to the page will fault in correctly as usual when
accessed the next time.

Special care needs to be taken throughout the change_protection_range()
process.  Since now we allow user to wr-protect a none pte, we need to be
able to pre-populate the page table entries if we see (!anonymous &&
MM_CP_UFFD_WP) requests, otherwise change_protection_range() will always
skip when the pgtable entry does not exist.

For example, the pgtable can be missing for a whole chunk of 2M pmd, but
the page cache can exist for the 2M range.  When we want to wr-protect one
4K page within the 2M pmd range, we need to pre-populate the pgtable and
install the pte marker showing that we want to get a message and block the
thread when the page cache of that 4K page is written.  Without
pre-populating the pmd, change_protection() will simply skip that whole
pmd.

Note that this patch only covers the small pages (pte level) but not
covering any of the transparent huge pages yet.  That will be done later,
and this patch will be a preparation for it too.

Link: https://lkml.kernel.org/r/20220405014850.14352-1-peterx@redhat.comSigned-off-by: default avatarPeter Xu <peterx@redhat.com>
Cc: Alistair Popple <apopple@nvidia.com>
Cc: Andrea Arcangeli <aarcange@redhat.com>
Cc: Axel Rasmussen <axelrasmussen@google.com>
Cc: David Hildenbrand <david@redhat.com>
Cc: Hugh Dickins <hughd@google.com>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: "Kirill A . Shutemov" <kirill@shutemov.name>
Cc: Matthew Wilcox <willy@infradead.org>
Cc: Mike Kravetz <mike.kravetz@oracle.com>
Cc: Mike Rapoport <rppt@linux.vnet.ibm.com>
Cc: Nadav Amit <nadav.amit@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 999dad82
...@@ -30,6 +30,7 @@ ...@@ -30,6 +30,7 @@
#include <linux/mm_inline.h> #include <linux/mm_inline.h>
#include <linux/pgtable.h> #include <linux/pgtable.h>
#include <linux/sched/sysctl.h> #include <linux/sched/sysctl.h>
#include <linux/userfaultfd_k.h>
#include <asm/cacheflush.h> #include <asm/cacheflush.h>
#include <asm/mmu_context.h> #include <asm/mmu_context.h>
#include <asm/tlbflush.h> #include <asm/tlbflush.h>
...@@ -193,8 +194,16 @@ static unsigned long change_pte_range(struct mmu_gather *tlb, ...@@ -193,8 +194,16 @@ static unsigned long change_pte_range(struct mmu_gather *tlb,
newpte = pte_swp_mksoft_dirty(newpte); newpte = pte_swp_mksoft_dirty(newpte);
if (pte_swp_uffd_wp(oldpte)) if (pte_swp_uffd_wp(oldpte))
newpte = pte_swp_mkuffd_wp(newpte); newpte = pte_swp_mkuffd_wp(newpte);
} else if (is_pte_marker_entry(entry)) { } else if (pte_marker_entry_uffd_wp(entry)) {
/* Skip it, the same as none pte */ /*
* If this is uffd-wp pte marker and we'd like
* to unprotect it, drop it; the next page
* fault will trigger without uffd trapping.
*/
if (uffd_wp_resolve) {
pte_clear(vma->vm_mm, addr, pte);
pages++;
}
continue; continue;
} else { } else {
newpte = oldpte; newpte = oldpte;
...@@ -209,6 +218,20 @@ static unsigned long change_pte_range(struct mmu_gather *tlb, ...@@ -209,6 +218,20 @@ static unsigned long change_pte_range(struct mmu_gather *tlb,
set_pte_at(vma->vm_mm, addr, pte, newpte); set_pte_at(vma->vm_mm, addr, pte, newpte);
pages++; pages++;
} }
} else {
/* It must be an none page, or what else?.. */
WARN_ON_ONCE(!pte_none(oldpte));
if (unlikely(uffd_wp && !vma_is_anonymous(vma))) {
/*
* For file-backed mem, we need to be able to
* wr-protect a none pte, because even if the
* pte is none, the page/swap cache could
* exist. Doing that by install a marker.
*/
set_pte_at(vma->vm_mm, addr, pte,
make_pte_marker(PTE_MARKER_UFFD_WP));
pages++;
}
} }
} while (pte++, addr += PAGE_SIZE, addr != end); } while (pte++, addr += PAGE_SIZE, addr != end);
arch_leave_lazy_mmu_mode(); arch_leave_lazy_mmu_mode();
...@@ -242,6 +265,39 @@ static inline int pmd_none_or_clear_bad_unless_trans_huge(pmd_t *pmd) ...@@ -242,6 +265,39 @@ static inline int pmd_none_or_clear_bad_unless_trans_huge(pmd_t *pmd)
return 0; return 0;
} }
/* Return true if we're uffd wr-protecting file-backed memory, or false */
static inline bool
uffd_wp_protect_file(struct vm_area_struct *vma, unsigned long cp_flags)
{
return (cp_flags & MM_CP_UFFD_WP) && !vma_is_anonymous(vma);
}
/*
* If wr-protecting the range for file-backed, populate pgtable for the case
* when pgtable is empty but page cache exists. When {pte|pmd|...}_alloc()
* failed it means no memory, we don't have a better option but stop.
*/
#define change_pmd_prepare(vma, pmd, cp_flags) \
do { \
if (unlikely(uffd_wp_protect_file(vma, cp_flags))) { \
if (WARN_ON_ONCE(pte_alloc(vma->vm_mm, pmd))) \
break; \
} \
} while (0)
/*
* This is the general pud/p4d/pgd version of change_pmd_prepare(). We need to
* have separate change_pmd_prepare() because pte_alloc() returns 0 on success,
* while {pmd|pud|p4d}_alloc() returns the valid pointer on success.
*/
#define change_prepare(vma, high, low, addr, cp_flags) \
do { \
if (unlikely(uffd_wp_protect_file(vma, cp_flags))) { \
low##_t *p = low##_alloc(vma->vm_mm, high, addr); \
if (WARN_ON_ONCE(p == NULL)) \
break; \
} \
} while (0)
static inline unsigned long change_pmd_range(struct mmu_gather *tlb, static inline unsigned long change_pmd_range(struct mmu_gather *tlb,
struct vm_area_struct *vma, pud_t *pud, unsigned long addr, struct vm_area_struct *vma, pud_t *pud, unsigned long addr,
unsigned long end, pgprot_t newprot, unsigned long cp_flags) unsigned long end, pgprot_t newprot, unsigned long cp_flags)
...@@ -260,6 +316,7 @@ static inline unsigned long change_pmd_range(struct mmu_gather *tlb, ...@@ -260,6 +316,7 @@ static inline unsigned long change_pmd_range(struct mmu_gather *tlb,
next = pmd_addr_end(addr, end); next = pmd_addr_end(addr, end);
change_pmd_prepare(vma, pmd, cp_flags);
/* /*
* Automatic NUMA balancing walks the tables with mmap_lock * Automatic NUMA balancing walks the tables with mmap_lock
* held for read. It's possible a parallel update to occur * held for read. It's possible a parallel update to occur
...@@ -329,6 +386,7 @@ static inline unsigned long change_pud_range(struct mmu_gather *tlb, ...@@ -329,6 +386,7 @@ static inline unsigned long change_pud_range(struct mmu_gather *tlb,
pud = pud_offset(p4d, addr); pud = pud_offset(p4d, addr);
do { do {
next = pud_addr_end(addr, end); next = pud_addr_end(addr, end);
change_prepare(vma, pud, pmd, addr, cp_flags);
if (pud_none_or_clear_bad(pud)) if (pud_none_or_clear_bad(pud))
continue; continue;
pages += change_pmd_range(tlb, vma, pud, addr, next, newprot, pages += change_pmd_range(tlb, vma, pud, addr, next, newprot,
...@@ -349,6 +407,7 @@ static inline unsigned long change_p4d_range(struct mmu_gather *tlb, ...@@ -349,6 +407,7 @@ static inline unsigned long change_p4d_range(struct mmu_gather *tlb,
p4d = p4d_offset(pgd, addr); p4d = p4d_offset(pgd, addr);
do { do {
next = p4d_addr_end(addr, end); next = p4d_addr_end(addr, end);
change_prepare(vma, p4d, pud, addr, cp_flags);
if (p4d_none_or_clear_bad(p4d)) if (p4d_none_or_clear_bad(p4d))
continue; continue;
pages += change_pud_range(tlb, vma, p4d, addr, next, newprot, pages += change_pud_range(tlb, vma, p4d, addr, next, newprot,
...@@ -372,6 +431,7 @@ static unsigned long change_protection_range(struct mmu_gather *tlb, ...@@ -372,6 +431,7 @@ static unsigned long change_protection_range(struct mmu_gather *tlb,
tlb_start_vma(tlb, vma); tlb_start_vma(tlb, vma);
do { do {
next = pgd_addr_end(addr, end); next = pgd_addr_end(addr, end);
change_prepare(vma, pgd, p4d, addr, cp_flags);
if (pgd_none_or_clear_bad(pgd)) if (pgd_none_or_clear_bad(pgd))
continue; continue;
pages += change_p4d_range(tlb, vma, pgd, addr, next, newprot, pages += change_p4d_range(tlb, vma, pgd, addr, next, newprot,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment