Commit c2f29ea1 authored by Benjamin LaHaise's avatar Benjamin LaHaise Committed by Linus Torvalds

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

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: default avatarBenjamin LaHaise <benjamin.c.lahaise@intel.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 83e5d8f7
...@@ -135,7 +135,7 @@ struct page_state { ...@@ -135,7 +135,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 long offset);
extern void __mod_page_state(unsigned long 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) \
......
...@@ -1144,7 +1144,7 @@ void get_full_page_state(struct page_state *ret) ...@@ -1144,7 +1144,7 @@ void get_full_page_state(struct page_state *ret)
__get_page_state(ret, sizeof(*ret) / sizeof(unsigned long)); __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; unsigned long ret = 0;
int cpu; int cpu;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment