Re: [PATCH] mm: remove some useless comments of node_stat_item

From: Matthew Wilcox
Date: Thu Jul 13 2023 - 08:10:28 EST


On Thu, Jul 13, 2023 at 07:49:15PM +0800, Miaohe Lin wrote:
> Some comments of node_stat_item are not that helpful and even confusing,
> so remove them. No functional change intended.

No, that's very useful and important. Why does it confuse you?

> enum node_stat_item {
> NR_LRU_BASE,
> NR_INACTIVE_ANON = NR_LRU_BASE, /* must match order of LRU_[IN]ACTIVE */
> - NR_ACTIVE_ANON, /* " " " " " */
> - NR_INACTIVE_FILE, /* " " " " " */
> - NR_ACTIVE_FILE, /* " " " " " */
> - NR_UNEVICTABLE, /* " " " " " */
> + NR_ACTIVE_ANON,
> + NR_INACTIVE_FILE,
> + NR_ACTIVE_FILE,
> + NR_UNEVICTABLE,
> NR_SLAB_RECLAIMABLE_B,
> NR_SLAB_UNRECLAIMABLE_B,
> NR_ISOLATED_ANON, /* Temporary isolated pages from anon lru */
> --
> 2.33.0
>
>