On Thu, Apr 11, 2024 at 10:57:23AM +1000, Alistair Popple wrote: > pud_huge() returns true only for a HugeTLB page. pud_devmap() is only > used by FS DAX pages. These two things are mutually exclusive so this > code is dead code and can be removed. I'm not sure this is true.. pud_huge() is mostly a misspelling of pud_leaf().. > - if (pud_huge(pud) && pud_devmap(pud)) { I suspect this should be written as: if (pud_leaf(pud) && pud_devmap(pud)) { In line with Peter's work here: https://lore.kernel.org/linux-mm/20240321220802.679544-1-peterx@xxxxxxxxxx/ Jason