Re: [PATCH v9 5/7] mm,page_owner: Display all stacks and their count

From: Oscar Salvador
Date: Thu Feb 15 2024 - 06:58:21 EST


On Thu, Feb 15, 2024 at 12:10:38PM +0100, Vlastimil Babka wrote:
> On 2/14/24 18:01, Oscar Salvador wrote:
> > Signed-off-by: Oscar Salvador <osalvador@xxxxxxx>
> > Acked-by: Marco Elver <elver@xxxxxxxxxx>
>
> Reviewed-by: Vlastimil Babka <vbabka@xxxxxxx>

Thanks

> > + for (i = 0; i < nr_entries; i++)
> > + seq_printf(m, " %pS\n", (void *)entries[i]);
> > + seq_printf(m, "stack_count: %d\n\n", refcount_read(&stack_record->count));
>
> So count - 1 here to report actual usage, as explained in reply to 4/7?

Yes, will do.


--
Oscar Salvador
SUSE Labs