On Sun, Dec 03, 2023 at 12:23:19AM +0100, Borislav Petkov wrote: > On Thu, Nov 23, 2023 at 06:05:11AM +0530, ankita@xxxxxxxxxx wrote: > > - pfn = PHYS_PFN(physical_addr); > > - if (!pfn_valid(pfn) && !arch_is_platform_page(physical_addr)) { > > - pr_warn_ratelimited(FW_WARN GHES_PFX > > - "Invalid address in generic error data: %#llx\n", > > - physical_addr); > > - return false; > > - } > > You don't just remove a pfn valid test just because your weird device > can't stomach it - you extend it, like It wasn't removed. patch 1 moved it to memory_failure() where it makes a lot more sense. Jason