Re: [PATCH]: Adding a counter in vma to indicate the number of physical pages backing it

Andi Kleen
2006-06-12T11:21:50Z

On Sunday 11 June 2006 18:09, Arjan van de Ven wrote:
> On Fri, 2006-06-09 at 18:33 -0700, Rohit Seth wrote:
> > Below is a patch that adds number of physical pages that each vma is
> > using in a process. Exporting this information to user space
> > using /proc/<pid>/maps interface.
>
> is it really worth bloating the vma struct for this? there are quite a
> few workloads that have a gazilion vma's, and this patch adds both
> memory usage and cache pressure to those workloads...

I agree it's a bad idea. smaps is only a debugging kludge anyways
and it's not a good idea to we bloat core data structures for it.

-Andi