* [PATCH] mm: delete stale comment about cachelines
@ 2026-06-01 11:40 Brendan Jackman
2026-06-01 12:08 ` David Hildenbrand (Arm)
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Brendan Jackman @ 2026-06-01 11:40 UTC (permalink / raw)
To: Andrew Morton, David Hildenbrand, Lorenzo Stoakes,
Liam R. Howlett, Vlastimil Babka, Mike Rapoport,
Suren Baghdasaryan, Michal Hocko, Kairui Song, Qi Zheng,
Shakeel Butt, Barry Song, Axel Rasmussen, Yuanchu Xie, Wei Xu
Cc: linux-mm, linux-kernel, Christoph Lameter, Johannes Weiner,
Brendan Jackman
These comments have been wrong since commit a211c6550efc ("mm:
page_alloc: defrag_mode kswapd/kcompactd watermarks") added
NR_FREE_PAGES_BLOCKS. Since nobody has complained about it in the last
year, it seems unlikely these comments were particularly useful anyway,
so delete them.
Signed-off-by: Brendan Jackman <jackmanb@google.com>
---
| 2 --
1 file changed, 2 deletions(-)
--git a/include/linux/mmzone.h b/include/linux/mmzone.h
index 1331a7b93f33c..e308f5ac7950a 100644
--- a/include/linux/mmzone.h
+++ b/include/linux/mmzone.h
@@ -214,7 +214,6 @@ enum numa_stat_item {
#endif
enum zone_stat_item {
- /* First 128 byte cacheline (assuming 64 bit words) */
NR_FREE_PAGES,
NR_FREE_PAGES_BLOCKS,
NR_ZONE_LRU_BASE, /* Used only for compaction and reclaim retry */
@@ -225,7 +224,6 @@ enum zone_stat_item {
NR_ZONE_UNEVICTABLE,
NR_ZONE_WRITE_PENDING, /* Count of dirty, writeback and unstable pages */
NR_MLOCK, /* mlock()ed pages found and moved off LRU */
- /* Second 128 byte cacheline */
#if IS_ENABLED(CONFIG_ZSMALLOC)
NR_ZSPAGES, /* allocated in zsmalloc */
#endif
---
base-commit: 4aedd91eef75addec6eb2702aa0d94625d01458d
change-id: 20260601-zone_stat_item-comment-45d660892420
Best regards,
--
Brendan Jackman <jackmanb@google.com>
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] mm: delete stale comment about cachelines
2026-06-01 11:40 [PATCH] mm: delete stale comment about cachelines Brendan Jackman
@ 2026-06-01 12:08 ` David Hildenbrand (Arm)
2026-06-01 12:13 ` Lorenzo Stoakes
2026-06-01 12:51 ` Vlastimil Babka (SUSE)
2 siblings, 0 replies; 4+ messages in thread
From: David Hildenbrand (Arm) @ 2026-06-01 12:08 UTC (permalink / raw)
To: Brendan Jackman, Andrew Morton, Lorenzo Stoakes, Liam R. Howlett,
Vlastimil Babka, Mike Rapoport, Suren Baghdasaryan, Michal Hocko,
Kairui Song, Qi Zheng, Shakeel Butt, Barry Song, Axel Rasmussen,
Yuanchu Xie, Wei Xu
Cc: linux-mm, linux-kernel, Christoph Lameter, Johannes Weiner
On 6/1/26 13:40, Brendan Jackman wrote:
> These comments have been wrong since commit a211c6550efc ("mm:
> page_alloc: defrag_mode kswapd/kcompactd watermarks") added
> NR_FREE_PAGES_BLOCKS. Since nobody has complained about it in the last
> year, it seems unlikely these comments were particularly useful anyway,
> so delete them.
Agreed, let's just drop them.
Acked-by: David Hildenbrand (Arm) <david@kernel.org>
--
Cheers,
David
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mm: delete stale comment about cachelines
2026-06-01 11:40 [PATCH] mm: delete stale comment about cachelines Brendan Jackman
2026-06-01 12:08 ` David Hildenbrand (Arm)
@ 2026-06-01 12:13 ` Lorenzo Stoakes
2026-06-01 12:51 ` Vlastimil Babka (SUSE)
2 siblings, 0 replies; 4+ messages in thread
From: Lorenzo Stoakes @ 2026-06-01 12:13 UTC (permalink / raw)
To: Brendan Jackman
Cc: Andrew Morton, David Hildenbrand, Liam R. Howlett,
Vlastimil Babka, Mike Rapoport, Suren Baghdasaryan, Michal Hocko,
Kairui Song, Qi Zheng, Shakeel Butt, Barry Song, Axel Rasmussen,
Yuanchu Xie, Wei Xu, linux-mm, linux-kernel, Christoph Lameter,
Johannes Weiner
On Mon, Jun 01, 2026 at 11:40:09AM +0000, Brendan Jackman wrote:
> These comments have been wrong since commit a211c6550efc ("mm:
> page_alloc: defrag_mode kswapd/kcompactd watermarks") added
> NR_FREE_PAGES_BLOCKS. Since nobody has complained about it in the last
> year, it seems unlikely these comments were particularly useful anyway,
> so delete them.
>
> Signed-off-by: Brendan Jackman <jackmanb@google.com>
Ah, bitrot :)
LGTM so:
Reviewed-by: Lorenzo Stoakes <ljs@kernel.org>
> ---
> include/linux/mmzone.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
> index 1331a7b93f33c..e308f5ac7950a 100644
> --- a/include/linux/mmzone.h
> +++ b/include/linux/mmzone.h
> @@ -214,7 +214,6 @@ enum numa_stat_item {
> #endif
>
> enum zone_stat_item {
> - /* First 128 byte cacheline (assuming 64 bit words) */
> NR_FREE_PAGES,
> NR_FREE_PAGES_BLOCKS,
> NR_ZONE_LRU_BASE, /* Used only for compaction and reclaim retry */
> @@ -225,7 +224,6 @@ enum zone_stat_item {
> NR_ZONE_UNEVICTABLE,
> NR_ZONE_WRITE_PENDING, /* Count of dirty, writeback and unstable pages */
> NR_MLOCK, /* mlock()ed pages found and moved off LRU */
> - /* Second 128 byte cacheline */
> #if IS_ENABLED(CONFIG_ZSMALLOC)
> NR_ZSPAGES, /* allocated in zsmalloc */
> #endif
>
> ---
> base-commit: 4aedd91eef75addec6eb2702aa0d94625d01458d
> change-id: 20260601-zone_stat_item-comment-45d660892420
>
> Best regards,
> --
> Brendan Jackman <jackmanb@google.com>
>
Cheers, Lorenzo
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mm: delete stale comment about cachelines
2026-06-01 11:40 [PATCH] mm: delete stale comment about cachelines Brendan Jackman
2026-06-01 12:08 ` David Hildenbrand (Arm)
2026-06-01 12:13 ` Lorenzo Stoakes
@ 2026-06-01 12:51 ` Vlastimil Babka (SUSE)
2 siblings, 0 replies; 4+ messages in thread
From: Vlastimil Babka (SUSE) @ 2026-06-01 12:51 UTC (permalink / raw)
To: Brendan Jackman, Andrew Morton, David Hildenbrand,
Lorenzo Stoakes, Liam R. Howlett, Mike Rapoport,
Suren Baghdasaryan, Michal Hocko, Kairui Song, Qi Zheng,
Shakeel Butt, Barry Song, Axel Rasmussen, Yuanchu Xie, Wei Xu
Cc: linux-mm, linux-kernel, Christoph Lameter, Johannes Weiner
On 6/1/26 13:40, Brendan Jackman wrote:
> These comments have been wrong since commit a211c6550efc ("mm:
> page_alloc: defrag_mode kswapd/kcompactd watermarks") added
> NR_FREE_PAGES_BLOCKS. Since nobody has complained about it in the last
> year, it seems unlikely these comments were particularly useful anyway,
> so delete them.
>
> Signed-off-by: Brendan Jackman <jackmanb@google.com>
Acked-by: Vlastimil Babka (SUSE) <vbabka@kernel.org>
> ---
> include/linux/mmzone.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
> index 1331a7b93f33c..e308f5ac7950a 100644
> --- a/include/linux/mmzone.h
> +++ b/include/linux/mmzone.h
> @@ -214,7 +214,6 @@ enum numa_stat_item {
> #endif
>
> enum zone_stat_item {
> - /* First 128 byte cacheline (assuming 64 bit words) */
> NR_FREE_PAGES,
> NR_FREE_PAGES_BLOCKS,
> NR_ZONE_LRU_BASE, /* Used only for compaction and reclaim retry */
> @@ -225,7 +224,6 @@ enum zone_stat_item {
> NR_ZONE_UNEVICTABLE,
> NR_ZONE_WRITE_PENDING, /* Count of dirty, writeback and unstable pages */
> NR_MLOCK, /* mlock()ed pages found and moved off LRU */
> - /* Second 128 byte cacheline */
> #if IS_ENABLED(CONFIG_ZSMALLOC)
> NR_ZSPAGES, /* allocated in zsmalloc */
> #endif
>
> ---
> base-commit: 4aedd91eef75addec6eb2702aa0d94625d01458d
> change-id: 20260601-zone_stat_item-comment-45d660892420
>
> Best regards,
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-06-01 12:51 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-01 11:40 [PATCH] mm: delete stale comment about cachelines Brendan Jackman
2026-06-01 12:08 ` David Hildenbrand (Arm)
2026-06-01 12:13 ` Lorenzo Stoakes
2026-06-01 12:51 ` Vlastimil Babka (SUSE)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox