Re: [PATCH v2] mm: memory-failure: make action_result() return int

From: HORIGUCHI NAOYA(堀口 直也)
Date: Mon Oct 24 2022 - 03:36:11 EST


On Mon, Oct 24, 2022 at 11:51:38AM +0800, Kefeng Wang wrote:
> Check mf_result in action_result(), only return 0 when
> MF_RECOVERED/MF_DELAYED, or return -EBUSY, which will
> simplify code a bit.
>
> Signed-off-by: Kefeng Wang <wangkefeng.wang@xxxxxxxxxx>
> ---
> v2:
> - MF_DELAYED is considered as success, suggested by HORIGUCHI
> - adjust order between unlock_page(head) and action_result()
> after !hwpoison_user_mappings() to clean code more in
> try_memory_failure_hugetlb()
>

Thank you for the update. The patch looks good to me.

Acked-by: Naoya Horiguchi <naoya.horiguchi@xxxxxxx>