On Fri, Feb 04, 2022 at 07:58:07PM +0000, Matthew Wilcox (Oracle) wrote: > #ifdef CONFIG_MEMCG_SWAP > -extern void mem_cgroup_swapout(struct page *page, swp_entry_t entry); > +extern void mem_cgroup_swapout(struct folio *folio, swp_entry_t entry); Might be worth to drop the extern while you're at it. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>