All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Get rid of arch_free_page() warning
@ 2004-12-14 20:14 David Howells
  0 siblings, 0 replies; only message in thread
From: David Howells @ 2004-12-14 20:14 UTC (permalink / raw)
  To: akpm; +Cc: linux-kernel


The attached patch gets rid of a warning produced when compiling
mm/page_alloc.c if the arch doesn't supply its own arch_free_page().

The problem is that there's one place in there that ignores the return value
of this function, and so you get a warning from gcc about an ineffectual
statement.

Signed-Off-By: David Howells <dhowells@redhat.com>
---
warthog>diffstat ../archfreepage-2610rc3.diff 
 gfp.h |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletion(-)

diff -uNr linux-2.6.10-rc3-mm1-mmcleanup/include/linux/gfp.h linux-2.6.10-rc3-mm1-misc/include/linux/gfp.h
--- linux-2.6.10-rc3-mm1-mmcleanup/include/linux/gfp.h	2004-12-13 17:34:21.000000000 +0000
+++ linux-2.6.10-rc3-mm1-misc/include/linux/gfp.h	2004-12-14 14:08:48.836858832 +0000
@@ -79,7 +79,10 @@
  * immediately bail: the arch-specific function has done all the work.
  */
 #ifndef HAVE_ARCH_FREE_PAGE
-#define arch_free_page(page, order) 0
+static inline int arch_free_page(struct page *page, unsigned int order)
+{
+	return 0;
+}
 #endif
 
 extern struct page *

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

only message in thread, other threads:[~2004-12-14 20:14 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-12-14 20:14 [PATCH] Get rid of arch_free_page() warning David Howells

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.