public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [patch 14/19] remove set_page_buffers() and clear_page_buffers()
@ 2002-06-17  6:53 Andrew Morton
  0 siblings, 0 replies; only message in thread
From: Andrew Morton @ 2002-06-17  6:53 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: lkml



The set_page_buffers() and clear_page_buffers() macros are each used in
only one place.  Fold them into their callers.



--- 2.5.22/include/linux/buffer_head.h~move-set_page_buffers	Sun Jun 16 23:12:51 2002
+++ 2.5.22-akpm/include/linux/buffer_head.h	Sun Jun 16 23:22:44 2002
@@ -117,16 +117,6 @@ BUFFER_FNS(Boundary, boundary)
 		((struct buffer_head *)(page)->private);	\
 	})
 #define page_has_buffers(page)	PagePrivate(page)
-#define set_page_buffers(page, buffers)				\
-	do {							\
-		SetPagePrivate(page);				\
-		page->private = (unsigned long)buffers;		\
-	} while (0)
-#define clear_page_buffers(page)				\
-	do {							\
-		ClearPagePrivate(page);				\
-		page->private = 0;				\
-	} while (0)
 
 #define invalidate_buffers(dev)	__invalidate_buffers((dev), 0)
 #define destroy_buffers(dev)	__invalidate_buffers((dev), 1)
--- 2.5.22/fs/buffer.c~move-set_page_buffers	Sun Jun 16 23:12:51 2002
+++ 2.5.22-akpm/fs/buffer.c	Sun Jun 16 23:22:44 2002
@@ -152,14 +152,16 @@ __set_page_buffers(struct page *page, st
 {
 	if (page_has_buffers(page))
 		buffer_error();
-	set_page_buffers(page, head);
 	page_cache_get(page);
+	SetPagePrivate(page);
+	page->private = (unsigned long)head;
 }
 
 static inline void
 __clear_page_buffers(struct page *page)
 {
-	clear_page_buffers(page);
+	ClearPagePrivate(page);
+	page->private = 0;
 	page_cache_release(page);
 }
 

-

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2002-06-17  7:21 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-06-17  6:53 [patch 14/19] remove set_page_buffers() and clear_page_buffers() Andrew Morton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox