Re: mm/rmap.c negative page map count BUG.

[Dave Jones]

On Wed, Jan 04, 2006 at 04:16:40PM -0800, Andrew Morton wrote:
 > Dave Jones <davej@xxxxxxxxxx> wrote:
 > >
 > > +			printk (KERN_EMERG "Eeek! page_mapcount(page) went negative! (%d)\n", page->_mapcount);
 > 
 > page_mapcount(page);
 > 
 > > +			printk (KERN_EMERG "  page->flags = %x\n", page->flags);
 > 
 > %lx
 > 
 > > +			printk (KERN_EMERG "  page->count = %x\n", page->_count);
 > 
 > page_count(page);

Ugh, almost an error per line. I suck.

		Dave

--- linux-2.6.14/mm/rmap.c~	2006-01-03 08:53:32.000000000 -0500
+++ linux-2.6.14/mm/rmap.c	2006-01-03 08:58:19.000000000 -0500
@@ -484,6 +484,13 @@ void page_remove_rmap(struct page *page)
 	BUG_ON(PageReserved(page));
 
 	if (atomic_add_negative(-1, &page->_mapcount)) {
+		if (page_mapcount(page) < 0) {
+			printk (KERN_EMERG "Eeek! page_mapcount(page) went negative! (%d)\n", page_mapcount(page));
+			printk (KERN_EMERG "  page->flags = %lx\n", page->flags);
+			printk (KERN_EMERG "  page->count = %x\n", page_count(page));
+			printk (KERN_EMERG "  page->mapping = %p\n", page->mapping);
+		}
+		
 		BUG_ON(page_mapcount(page) < 0);
 		/*
 		 * It would be tidy to reset the PageAnon mapping here,
-
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/