Re: [PATCH 3/6] mm/page_alloc.c: remove obsolete comment in free_pcppages_bulk()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Miaohe Lin <linmiaohe@xxxxxxxxxx>
Subject: Re: [PATCH 3/6] mm/page_alloc.c: remove obsolete comment in free_pcppages_bulk()
From: Mel Gorman <mgorman@xxxxxxxxxxxxxxxxxxx>
Date: Tue, 31 Aug 2021 14:38:30 +0100
Cc: akpm@xxxxxxxxxxxxxxxxxxxx, vbabka@xxxxxxx, sfr@xxxxxxxxxxxxxxxx, peterz@xxxxxxxxxxxxx, linux-mm@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20210830141051.64090-4-linmiaohe@huawei.com>
References: <20210830141051.64090-1-linmiaohe@huawei.com> <20210830141051.64090-4-linmiaohe@huawei.com>
User-agent: Mutt/1.10.1 (2018-07-13)

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.

-- 
Mel Gorman
SUSE Labs

Follow-Ups:
- Re: [PATCH 3/6] mm/page_alloc.c: remove obsolete comment in free_pcppages_bulk()
  - From: Miaohe Lin

References:
- [PATCH 0/6] Cleanups and fixup for page_alloc
  - From: Miaohe Lin
- [PATCH 3/6] mm/page_alloc.c: remove obsolete comment in free_pcppages_bulk()
  - From: Miaohe Lin

Prev by Date: Re: [PATCH 2/6] mm/page_alloc.c: simplify the code by using macro K()
Next by Date: Re: [PATCH 4/6] mm/page_alloc.c: use helper function zone_spans_pfn()
Previous by thread: [PATCH 3/6] mm/page_alloc.c: remove obsolete comment in free_pcppages_bulk()
Next by thread: Re: [PATCH 3/6] mm/page_alloc.c: remove obsolete comment in free_pcppages_bulk()
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]

Powered by Linux