[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y5JQJINs4LPG510r@casper.infradead.org>
Date: Thu, 8 Dec 2022 20:59:16 +0000
From: Matthew Wilcox <willy@...radead.org>
To: "Vishal Moola (Oracle)" <vishal.moola@...il.com>
Cc: linux-mm@...ck.org, damon@...ts.linux.dev,
linux-kernel@...r.kernel.org, akpm@...ux-foundation.org,
sj@...nel.org
Subject: Re: [PATCH v3 4/4] mm/swap: Convert deactivate_page() to
folio_deactivate()
On Thu, Dec 08, 2022 at 12:35:03PM -0800, Vishal Moola (Oracle) wrote:
> Deactivate_page() has already been converted to use folios, this change
> converts it to take in a folio argument instead of calling page_folio().
> It also renames the function folio_deactivate() to be more consistent
> with other folio functions.
>
> Signed-off-by: Vishal Moola (Oracle) <vishal.moola@...il.com>
Reviewed-by: Matthew Wilcox (Oracle) <willy@...radead.org>
(for future series like this, it's slightly fewer changes to introduce
folio_deactivate() first and change deactivate_page() to be a wrapper.
Then patches 2 & 3 in this series can just be converted straight to
folio_deactivate() instead of being changed twice. wouldn't ask
you to redo the patch series at this point, but next time ...)
Powered by blists - more mailing lists