Re: [RFC v2 PATCH 4/5] vmstat: add page-cache numa hints

Donet Tom
2024-12-27T10:48:39Z

On 12/11/24 03:07, Gregory Price wrote:
> Count non-page-fault events as page-cache numa hints instead of
> fault hints in vmstat. Add a define to select the hint type to
> keep the code clean.
>
> Signed-off-by: Gregory Price <gourry@gourry.net>
> ---
> include/linux/vm_event_item.h | 8 ++++++++
> mm/memory.c | 6 +++---
> mm/vmstat.c | 2 ++
> 3 files changed, 13 insertions(+), 3 deletions(-)
>
> diff --git a/include/linux/vm_event_item.h b/include/linux/vm_event_item.h
> index f70d0958095c..c5abb0f7cca7 100644
> --- a/include/linux/vm_event_item.h
> +++ b/include/linux/vm_event_item.h
> @@ -63,6 +63,8 @@ enum vm_event_item { PGPGIN, PGPGOUT, PSWPIN, PSWPOUT,
> NUMA_HUGE_PTE_UPDATES,
> NUMA_HINT_FAULTS,
> NUMA_HINT_FAULTS_LOCAL,
> + NUMA_HINT_PAGE_CACHE,
> + NUMA_HINT_PAGE_CACHE_LOCAL,
> NUMA_PAGE_MIGRATE,
> #endif
> #ifdef CONFIG_MIGRATION
> @@ -185,6 +187,12 @@ enum vm_event_item { PGPGIN, PGPGOUT, PSWPIN, PSWPOUT,
> NR_VM_EVENT_ITEMS
> };
>
> +#ifdef CONFIG_NUMA_BALANCING
> +#define NUMA_HINT_TYPE(vmf) (vmf ? NUMA_HINT_FAULTS : NUMA_HINT_PAGE_CACHE)
> +#define NUMA_HINT_TYPE_LOCAL(vmf) (vmf ? NUMA_HINT_FAULTS_LOCAL : \
> + NUMA_HINT_PAGE_CACHE_LOCAL)
> +#endif
> +
> #ifndef CONFIG_TRANSPARENT_HUGEPAGE
> #define THP_FILE_ALLOC ({ BUILD_BUG(); 0; })
> #define THP_FILE_FALLBACK ({ BUILD_BUG(); 0; })
> diff --git a/mm/memory.c b/mm/memory.c
> index af7ba56a4e1e..620e2045af7b 100644
> --- a/mm/memory.c
> +++ b/mm/memory.c
> @@ -5578,10 +5578,10 @@ int numa_migrate_check(struct folio *folio, struct vm_fault *vmf,
> *last_cpupid = folio_last_cpupid(folio);
>
> #ifdef CONFIG_NUMA_BALANCING
> - count_vm_numa_event(NUMA_HINT_FAULTS);
> - count_memcg_folio_events(folio, NUMA_HINT_FAULTS, 1);
> + count_vm_numa_event(NUMA_HINT_TYPE(vmf));
> + count_memcg_folio_events(folio, NUMA_HINT_TYPE(vmf), 1);
> if (folio_nid(folio) == numa_node_id()) {
> - count_vm_numa_event(NUMA_HINT_FAULTS_LOCAL);
> + count_vm_numa_event(NUMA_HINT_TYPE_LOCAL(vmf));

I have tested this patch series on my system with my test program. I am able
to see unmapped page cache pages are getting promoted.
numa_hint_faults2269numa_hint_faults_local2245numa_hint_page_cache1244numa_hint_page_cache_local0numa_pages_migrated4501

In my test result numa_hint_page_cache_local is 0. I am seeing
numa_hint_page_cache_local will only be incremented if the folio's
node and the process's running node are the same. This condition
does not occur in the current implementation, correct?

Thanks
Donet

> *flags |= TNF_FAULT_LOCAL;
> }
> #endif
> diff --git a/mm/vmstat.c b/mm/vmstat.c
> index 4d016314a56c..bcd9be11e957 100644
> --- a/mm/vmstat.c
> +++ b/mm/vmstat.c
> @@ -1338,6 +1338,8 @@ const char * const vmstat_text[] = {
> "numa_huge_pte_updates",
> "numa_hint_faults",
> "numa_hint_faults_local",
> + "numa_hint_page_cache",
> + "numa_hint_page_cache_local",
> "numa_pages_migrated",
> #endif
> #ifdef CONFIG_MIGRATION