On 1/4/22 21:04, Matthew Wilcox wrote:
On Tue, Jan 04, 2022 at 01:40:36PM -0800, John Hubbard wrote:
- page = compound_head(page);
- return PageCompound(page) && compound_order(page) > 1;
+ return folio_order(folio) > 1;
I see, no need to look at the compound page head, because folio_order() returns
zero for tail pages, and neatly avoids all of that.
Did you mean base pages instead of tail pages? A folio can never be a
tail page.
Oh right. I was thinking (incorrectly) that there might be some cases during
the page-to-folio conversion in which a tail page could sort of slip through,
but good point.
thanks,
--
John Hubbard
NVIDIA