Stephen Rothwell <[email protected]> writes: > Hi Ritesh, > > On Sat, 30 Aug 2025 09:21:47 +0530 "Ritesh Harjani (IBM)" > <[email protected]> wrote: >> >> diff --git a/mm/vmstat.c b/mm/vmstat.c >> index 71cd1ceba191..8cd17a5fc72b 100644 >> --- a/mm/vmstat.c >> +++ b/mm/vmstat.c >> @@ -1464,6 +1464,11 @@ const char * const vmstat_text[] = { >> [I(DIRECT_MAP_LEVEL2_COLLAPSE)] = "direct_map_level2_collapses", >> [I(DIRECT_MAP_LEVEL3_COLLAPSE)] = "direct_map_level3_collapses", >> #endif >> +#ifdef CONFIG_PPC_BOOK3S_64 >> + "slb_kernel_faults", >> + "slb_user_faults", >> +#endif >> + >> #ifdef CONFIG_PER_VMA_LOCK_STATS >> [I(VMA_LOCK_SUCCESS)] = "vma_lock_success", >> [I(VMA_LOCK_ABORT)] = "vma_lock_abort", > > Should you be using explicit indexes and the I() macro?
Aah yes, I guess the branch where I developed the patches was not upstream tip and when I rebased and tested, I missed to see the I() macro change in mm/vmstat. Thanks Stephen for pointing it out. I will fix that in the next revision. -ritesh > > -- > Cheers, > Stephen Rothwell
