ARM: 8109/1: mm: Modify pte_write and pmd_write logic for LPAE
authorSteven Capper <steve.capper@linaro.org>
Fri, 18 Jul 2014 15:16:15 +0000 (16:16 +0100)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Thu, 24 Jul 2014 13:27:08 +0000 (14:27 +0100)
commitded9477984690d026e46dd75e8157392cea3f13f
tree8faee3f6c299d1979290ce459687cf51c7e91e4c
parentf2950706871c4b6e8c0f0d7c3f62d35930b8de63
ARM: 8109/1: mm: Modify pte_write and pmd_write logic for LPAE

For LPAE, we have the following means for encoding writable or dirty
ptes:
                              L_PTE_DIRTY       L_PTE_RDONLY
    !pte_dirty && !pte_write        0               1
    !pte_dirty && pte_write         0               1
    pte_dirty && !pte_write         1               1
    pte_dirty && pte_write          1               0

So we can't distinguish between writeable clean ptes and read only
ptes. This can cause problems with ptes being incorrectly flagged as
read only when they are writeable but not dirty.

This patch renumbers L_PTE_RDONLY from AP[2] to a software bit #58,
and adds additional logic to set AP[2] whenever the pte is read only
or not dirty. That way we can distinguish between clean writeable ptes
and read only ptes.

HugeTLB pages will use this new logic automatically.

We need to add some logic to Transparent HugePages to ensure that they
correctly interpret the revised pgprot permissions (L_PTE_RDONLY has
moved and no longer matches PMD_SECT_AP2). In the process of revising
THP, the names of the PMD software bits have been prefixed with L_ to
make them easier to distinguish from their hardware bit counterparts.

Signed-off-by: Steve Capper <steve.capper@linaro.org>
Reviewed-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
arch/arm/include/asm/pgtable-3level-hwdef.h
arch/arm/include/asm/pgtable-3level.h
arch/arm/mm/dump.c
arch/arm/mm/proc-v7-3level.S