Re: numa_maps update

Andi Kleen
2006-03-04T04:59:16Z

On Saturday 04 March 2006 10:07, Andrew Morton wrote:
> Christoph Lameter <clameter@engr.sgi.com> wrote:
> >
> > Change the format
>
> uh-oh.

I guess it's better because it's clearly better than the old format.
But of course it would need to be done before 2.6.16
>
> > of numa_maps to be more compact and contain additional
> > information that is useful for managing and troubleshooting memory on a NUMA
> > system. Numa_maps can now also support huge pages.
>
> What will be the userspace impact (ie: breakage) due to this change?

It will at least break the manpages I think. But I suspect/hope no user space
is using it yet because it was only added recently.

> > + if (file) {
> > +
> > + seq_printf(m, " file=");
> > + seq_path(m, file->f_vfsmnt, file->f_dentry, "\n\t");
> > +
> > + } else if (vma->vm_start <= mm->brk &&
> > + vma->vm_end >= mm->start_brk)
> > +
> > + seq_printf(m, " heap");
> > +
> > + else if (vma->vm_start <= mm->start_stack &&
> > + vma->vm_end >= mm->start_stack)
> > +
> > + seq_printf(m, " stack");
> > +
> > + if (is_vm_hugetlb_page(vma)) {
> > +
> > + check_huge_range(vma, vma->vm_start, vma->vm_end, md);
> > + seq_printf(m, " huge");
> > +
> > + } else
>
> What bizarre layout!

The 16 space indents?

-Andi