[PATCH]: Bug in MM_RB debugging?

From: David Miller
Date: Tue Feb 27 2007 - 14:19:22 EST



The code is seemingly trying to make sure that rb_next()
brings us to successive increasing vma entries.

But the two variables, prev and pend, used to perform these
checks, are never advanced.

Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>

diff --git a/mm/mmap.c b/mm/mmap.c
index eb509ae..84f997d 100644
--- a/mm/mmap.c
+++ b/mm/mmap.c
@@ -299,6 +299,8 @@ static int browse_rb(struct rb_root *root)
printk("vm_end %lx < vm_start %lx\n", vma->vm_end, vma->vm_start);
i++;
pn = nd;
+ prev = vma->vm_start;
+ pend = vma->vm_end;
}
j = 0;
for (nd = pn; nd; nd = rb_prev(nd)) {
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/