Re: [PATCH v1] mm, hwpoison: enable error handling on shmem thp

From: Oscar Salvador
Date: Thu Feb 11 2021 - 03:07:10 EST


On Tue, Feb 09, 2021 at 03:21:28PM +0900, Naoya Horiguchi wrote:
> From: Naoya Horiguchi <naoya.horiguchi@xxxxxxx>
>
> Currently hwpoison code checks PageAnon() for thp and refuses to handle
> errors on non-anonymous thps (just for historical reason). We now
> support non-anonymou thp like shmem one, so this patch suggests to enable
> to handle shmem thps. Fortunately, we already have can_split_huge_page()
> to check if a give thp is splittable, so this patch relies on it.
>
> Signed-off-by: Naoya Horiguchi <naoya.horiguchi@xxxxxxx>

Looks good to me, thanks Naoya:

Reviewed-by: Oscar Salvador <osalvador@xxxxxxx>


--
Oscar Salvador
SUSE L3