Browse Source

[PATCH] __mod_page_state(): pass unsigned long instead of unsigned

By making the offset argument of __mod_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>
Benjamin LaHaise 20 years ago
parent
commit
83e5d8f725
2 changed files with 2 additions and 2 deletions
  1. 1 1
      include/linux/page-flags.h
  2. 1 1
      mm/page_alloc.c

+ 1 - 1
include/linux/page-flags.h

@@ -136,7 +136,7 @@ struct page_state {
 extern void get_page_state(struct page_state *ret);
 extern void get_page_state(struct page_state *ret);
 extern void get_full_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 offset);
-extern void __mod_page_state(unsigned offset, unsigned long delta);
+extern void __mod_page_state(unsigned long offset, unsigned long delta);
 
 
 #define read_page_state(member) \
 #define read_page_state(member) \
 	__read_page_state(offsetof(struct page_state, member))
 	__read_page_state(offsetof(struct page_state, member))

+ 1 - 1
mm/page_alloc.c

@@ -1158,7 +1158,7 @@ unsigned long __read_page_state(unsigned offset)
 	return ret;
 	return ret;
 }
 }
 
 
-void __mod_page_state(unsigned offset, unsigned long delta)
+void __mod_page_state(unsigned long offset, unsigned long delta)
 {
 {
 	unsigned long flags;
 	unsigned long flags;
 	void* ptr;
 	void* ptr;