diff options
| author | Gavin Shan <[email protected]> | 2025-05-08 08:52:51 +0000 |
|---|---|---|
| committer | Will Deacon <[email protected]> | 2025-05-16 14:10:13 +0000 |
| commit | 13c63ce358832e01575d3b186210966e17f9ff68 (patch) | |
| tree | ddb45d1a13b65a6ccec6c1f80d69b2b2510d5ce9 /rust/helpers/helpers.c | |
| parent | arm64/mm: Permit lazy_mmu_mode to be nested (diff) | |
| download | kernel-13c63ce358832e01575d3b186210966e17f9ff68.tar.gz kernel-13c63ce358832e01575d3b186210966e17f9ff68.zip | |
arm64: mm: Drop redundant check in pmd_trans_huge()
pmd_val(pmd) is redundant because a positive pmd_present(pmd) ensures
a positive pmd_val(pmd) according to their definitions like below.
#define pmd_val(x) ((x).pmd)
#define pmd_present(pmd) pte_present(pmd_pte(pmd))
#define pte_present(pte) (pte_valid(pte) || pte_present_invalid(pte))
#define pte_valid(pte) (!!(pte_val(pte) & PTE_VALID))
#define pte_present_invalid(pte) \
((pte_val(pte) & (PTE_VALID | PTE_PRESENT_INVALID)) == PTE_PRESENT_INVALID)
pte_present() can't be positive unless either of the flag PTE_VALID or
PTE_PRESENT_INVALID is set. In this case, pmd_val(pmd) should be positive
either.
So lets drop the redundant check pmd_val(pmd) and no functional changes
intended.
Signed-off-by: Gavin Shan <[email protected]>
Reviewed-by: Dev Jain <[email protected]>
Reviewed-by: Anshuman Khandual <[email protected]>
Reviewed-by: Ryan Roberts <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Will Deacon <[email protected]>
Diffstat (limited to 'rust/helpers/helpers.c')
0 files changed, 0 insertions, 0 deletions
