arm64: Ignore hardware dirty bit updates in ptep_set_wrprotect()
authorCatalin Marinas <catalin.marinas@arm.com>
Wed, 5 Jul 2017 09:59:42 +0000 (10:59 +0100)
committerCatalin Marinas <catalin.marinas@arm.com>
Mon, 21 Aug 2017 10:13:00 +0000 (11:13 +0100)
commit64c26841b34957ef8f33f7a9e8663aeee59c3ded
treefe899931712ae0637bfc7b6eead18e7881d02e7e
parent73e86cb03cf2ec0aa3789dc8621c6d53619cac5e
arm64: Ignore hardware dirty bit updates in ptep_set_wrprotect()

ptep_set_wrprotect() is only called on CoW mappings which are private
(!VM_SHARED) with the pte either read-only (!PTE_WRITE && PTE_RDONLY) or
writable and software-dirty (PTE_WRITE && !PTE_RDONLY && PTE_DIRTY).
There is no race with the hardware update of the dirty state: clearing
of PTE_RDONLY when PTE_WRITE (a.k.a. PTE_DBM) is set. This patch removes
the code setting the software PTE_DIRTY bit in ptep_set_wrprotect() as
superfluous. A VM_WARN_ONCE is introduced in case the above logic is
wrong or the core mm code changes its use of ptep_set_wrprotect().

Reviewed-by: Will Deacon <will.deacon@arm.com>
Acked-by: Steve Capper <steve.capper@arm.com>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
arch/arm64/include/asm/pgtable.h