Commit 8ef41786 authored by Anshuman Khandual's avatar Anshuman Khandual Committed by Catalin Marinas
Browse files

arm64/mm: Change pgattr_change_is_safe() arguments as pteval_t 



pgattr_change_is_safe() processes two distinct page table entries that just
happen to be 64 bits for all levels. This changes both arguments to reflect
the actual data type being processed in the function.

This change is important when moving to FEAT_D128 based 128 bit page tables
because it makes it simple to change the entry size in one place.

Cc: Will Deacon <will@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Ard Biesheuvel <ardb@kernel.org>
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Reviewed-by: default avatarRyan Roberts <ryan.roberts@arm.com>
Signed-off-by: default avatarAnshuman Khandual <anshuman.khandual@arm.com>
Link: https://lore.kernel.org/r/20241001045804.1119881-1-anshuman.khandual@arm.com


Signed-off-by: default avatarCatalin Marinas <catalin.marinas@arm.com>
parent 9852d85e

arch/arm64/include/asm/pgtable.h

+1 −1
Original line number Diff line number Diff line
@@ -338,7 +338,7 @@ static inline pte_t __ptep_get(pte_t *ptep) 
}



extern void __sync_icache_dcache(pte_t pteval);

bool pgattr_change_is_safe(u64 old, u64 new);

bool pgattr_change_is_safe(pteval_t old, pteval_t new);



/*

 * PTE bits configuration in the presence of hardware Dirty Bit Management

arch/arm64/mm/mmu.c

+1 −1
Original line number Diff line number Diff line
@@ -119,7 +119,7 @@ static phys_addr_t __init early_pgtable_alloc(int shift) 
	return phys;

}



bool pgattr_change_is_safe(u64 old, u64 new)

bool pgattr_change_is_safe(pteval_t old, pteval_t new)

{

	/*

	 * The following mapping attributes may be updated in live