2007-12-14 03:35:57

by Qi Yong

[permalink] [raw]
Subject: [patch] set_page_refcounted VM_BUG_ON fix

Hello,

The current PageTail semantic is that a PageTail page is first a PageCompound page.
So remove the redundant PageCompound test in set_page_refcounted().

Signed-off-by: Qi Yong <[email protected]>
---
diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c
diff --git a/mm/internal.h b/mm/internal.h
index 953f941..79797db 100644
--- a/mm/internal.h
+++ b/mm/internal.h
@@ -24,7 +24,7 @@ static inline void set_page_count(struct page *page, int v)
*/
static inline void set_page_refcounted(struct page *page)
{
- VM_BUG_ON(PageCompound(page) && PageTail(page));
+ VM_BUG_ON(PageTail(page));
VM_BUG_ON(atomic_read(&page->_count));
set_page_count(page, 1);
}

--
Qi Yong