Re: [PATCH v4 18/22] mm/zsmalloc: introduce __zpdesc_clear_movable

From: Sergey Senozhatsky
Date: Tue Jul 30 2024 - 05:39:07 EST


On (24/07/29 19:25), alexs@xxxxxxxxxx wrote:
[..]
> +static inline void __zpdesc_clear_movable(struct zpdesc *zpdesc)
> +{
> + __ClearPageMovable(zpdesc_page(zpdesc));
> +}

[..]

> @@ -846,7 +846,7 @@ static void reset_zpdesc(struct zpdesc *zpdesc)
> {
> struct page *page = zpdesc_page(zpdesc);
>
> - __ClearPageMovable(page);
> + __zpdesc_clear_movable(zpdesc);
> ClearPagePrivate(page);

Just a quick question, I see that you wrote wrappers for pretty
much everything, including SetPagePrivate(), but not for
ClearPagePrivate()?