Subject: Re: [PATCH v1 12/29] mm/zsmalloc: stop using __ClearPageMovable()

From: Harry Yoo <[email-masked]>
Date: Tue, 1 Jul 2025 17:20:54 +0900

On Mon, Jun 30, 2025 at 02:59:53PM +0200, David Hildenbrand wrote:
> Instead, let's check in the callbacks if the page was already destroyed,
> which can be checked by looking at zpdesc->zspage (see reset_zpdesc()).
> 
> If we detect that the page was destroyed:
> 
> (1) Fail isolation, just like the migration core would
> 
> (2) Fake migration success just like the migration core would
> 
> In the putback case there is nothing to do, as we don't do anything just
> like the migration core would do.
>
> In the future, we should look into not letting these pages get destroyed
> while they are isolated -- and instead delaying that to the
> putback/migration call. Add a TODO for that.
>
> Signed-off-by: David Hildenbrand <[email-masked]>
> ---

Looks correct to me.
Reviewed-by: Harry Yoo <[email-masked]>

-- 
Cheers,
Harry / Hyeonggon