Hi,
The generic implementation of huge_pte_offset() has inconsistent
behaviour when looking up hugepage PUDs vs PMDs entries that are not
present (returning NULL vs pte_t*).
Similarly, it returns NULL when encountering swap entries although all
the callers have special checks to properly deal with swap entries.
Without clear semantics, it is difficult to determine what is the
expected behaviour of huge_pte_offset() without going through all the
scenarios where it used.
I faced this recently when updating the arm64 implementation of
huge_pte_offset() to handle swap entries (related to enabling poisoned
memeory)[0]. And will come across again when I update it for
contiguous hugepage support now that core changes have been merged.
To address these issues, this small series -
* makes huge_pte_offset() consistent between PUD and PMDs
* adds support for returning swap entries
* and most importantly, documents the expected behaviour of
huge_pte_offset()
All feedback welcome.
Thanks,
Punit
[0]
Punit Agrawal (2):
mm/hugetlb: Make huge_pte_offset() consistent between PUD and PMD
entries
mm/hugetlb: Support swap entries in huge_pte_offset()
mm/hugetlb.c | 22 +++++++++++++++++++---
1 file changed, 19 insertions(+), 3 deletions(-)
--
2.11.0
