Messages in this thread | | | Date | Wed, 1 Sep 2021 16:14:11 +0100 | From | Mel Gorman <> | Subject | Re: [PATCH 3/6] mm/page_alloc.c: remove obsolete comment in free_pcppages_bulk() |
| |
On Wed, Sep 01, 2021 at 03:49:03PM +0800, Miaohe Lin wrote: > On 2021/8/31 21:38, Mel Gorman wrote: > > On Mon, Aug 30, 2021 at 10:10:48PM +0800, Miaohe Lin wrote: > >> It's also confusing now. Remove it. > >> > > > > Why is the whole comment obsolete? > > > > The second two paragraphs about "all pages pinned" and pages_scanned is > > obsolete and can go but the first paragraph is valid. > > > > I think the first paragraph is invalid due to the below statement: > "Assumes all pages on list are in same zone, and of same order." > There are NR_PCP_LISTS lists and PAGE_ALLOC_COSTLY_ORDER + 1 + NR_PCP_THP > orders in pcp. So I think it's obsolete. >
Ah.
> Should I delete this statement in the first paragraph only? >
Remove ", and of same order"
-- Mel Gorman SUSE Labs
| |