Re: [PATCH 12/24] task_diag: add a new group to get tasks memory mappings (v2)

On 7/14/15 2:08 PM, Oleg Nesterov wrote:

On 07/06, Andrey Vagin wrote:

quoted

+static int task_vma_num(struct mm_struct *mm)
+{
+	struct vm_area_struct *vma;
+	int n_vma = 0;
+
+	if (!mm || !atomic_inc_not_zero(&mm->mm_users))
+		return 0;
+
+	down_read(&mm->mmap_sem);
+	for (vma = mm->mmap; vma; vma = vma->vm_next, n_vma++)
+		;
+
+	up_read(&mm->mmap_sem);
+	mmput(mm);
+
+	return n_vma;
+}

Hmm. How about

	int task_vma_num(struct mm_struct *mm)
	{
		return mm->map_count;
	}

?

makes sense. Thanks for the pointer.