[PATCH] __read_page_state(): pass unsigned long instead of unsigned
authorBenjamin LaHaise <bcrl@kvack.org>
Wed, 22 Jun 2005 00:14:55 +0000 (17:14 -0700)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Wed, 22 Jun 2005 01:46:17 +0000 (18:46 -0700)
By making the offset argument of __read_page_state an unsigned long instead of
unsigned, we can avoid forcing the compiler to sign extend a usually constant
argument.  This saves 1 instruction on x86-64.

Signed-off-by: Benjamin LaHaise <benjamin.c.lahaise@intel.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
include/linux/page-flags.h
mm/page_alloc.c

index f2ee9b2332e3da6426cb69213c46837642cc26b7..f5a6695d4d21335dddf2aa0da868c31fad454270 100644 (file)
@@ -135,7 +135,7 @@ struct page_state {
 
 extern void get_page_state(struct page_state *ret);
 extern void get_full_page_state(struct page_state *ret);
-extern unsigned long __read_page_state(unsigned offset);
+extern unsigned long __read_page_state(unsigned long offset);
 extern void __mod_page_state(unsigned long offset, unsigned long delta);
 
 #define read_page_state(member) \
index c976628f9d9507bd586dd1fd3afb31d10ae32c86..10446a6463743f940830baf6b103fc593cc09bec 100644 (file)
@@ -1144,7 +1144,7 @@ void get_full_page_state(struct page_state *ret)
        __get_page_state(ret, sizeof(*ret) / sizeof(unsigned long));
 }
 
-unsigned long __read_page_state(unsigned offset)
+unsigned long __read_page_state(unsigned long offset)
 {
        unsigned long ret = 0;
        int cpu;