diff options
| author | Tiwei Bie <tiwei.btw@antgroup.com> | 2024-08-26 18:08:10 +0800 |
|---|---|---|
| committer | Richard Weinberger <richard@nod.at> | 2024-09-12 20:34:39 +0200 |
| commit | 669afa4e8715c5730fb353166f9aaaa14d4fed64 (patch) | |
| tree | 20f1e056c0a7fd2d3e074e78ae71db0faf161403 /arch/um/kernel | |
| parent | 2fcd16fbab9f448c7174bf4c3eeda53ef84e28ee (diff) | |
um: Remove the redundant newpage check in update_pte_range
The two checks have been identical since commit ef714f15027c ("um:
remove force_flush_all from fork_handler"). And the inner one isn't
necessary anymore.
Signed-off-by: Tiwei Bie <tiwei.btw@antgroup.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'arch/um/kernel')
| -rw-r--r-- | arch/um/kernel/tlb.c | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/arch/um/kernel/tlb.c b/arch/um/kernel/tlb.c index 44c6fc697f3a..548af31d4111 100644 --- a/arch/um/kernel/tlb.c +++ b/arch/um/kernel/tlb.c @@ -82,16 +82,12 @@ static inline int update_pte_range(pmd_t *pmd, unsigned long addr, (x ? UM_PROT_EXEC : 0)); if (pte_newpage(*pte)) { if (pte_present(*pte)) { - if (pte_newpage(*pte)) { - __u64 offset; - unsigned long phys = - pte_val(*pte) & PAGE_MASK; - int fd = phys_mapping(phys, &offset); - - ret = ops->mmap(ops->mm_idp, addr, - PAGE_SIZE, prot, fd, - offset); - } + __u64 offset; + unsigned long phys = pte_val(*pte) & PAGE_MASK; + int fd = phys_mapping(phys, &offset); + + ret = ops->mmap(ops->mm_idp, addr, PAGE_SIZE, + prot, fd, offset); } else ret = ops->unmap(ops->mm_idp, addr, PAGE_SIZE); } else if (pte_newprot(*pte)) |