Re: [PATCH] slab: Remove slab_folio()

[Harry Yoo <harry@kernel.org>]

On Wed, Sep 23, 2026 at 07:36:51PM +0100, Matthew Wilcox (Oracle) wrote:
> This is the final step of divorcing struct slab from struct folio.
> It's still tied to struct page, but there are no more mentions of folios
> in the slab code.
> 
> Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
> ---

Looks good to me,
Reviewed-by: Harry Yoo (Meta) <harry@kernel.org>

With a tiny little nit:

> @@ -184,15 +169,16 @@ static inline struct slab *page_slab(const struct page *page)
>   * slab_page - The first struct page allocated for a slab
>   * @s: The slab.
>   *
> - * A convenience wrapper for converting slab to the first struct page of the
> - * underlying folio, to communicate with code not yet converted to folio or
> - * struct slab.
> + * A convenience wrapper for returning the first page allocated to this slab.
> + * to communicate with code not yet converted to struct slab.
    * A convenience wrapper for returning the first "struct" page allocated to this
    * "slab" to communicate with code not yet converted to struct slab.
>   */

Perhaps you meant this?

-- 
Cheers,
Harry / Hyeonggon