diff options
author | Benjamin LaHaise <bcrl@kvack.org> | 2005-06-21 20:14:55 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-06-21 21:46:17 -0400 |
commit | c2f29ea111e3344ed48257c2a142c3db514e1529 (patch) | |
tree | 6750edefb1f08a6be545250fdaca72fa211ef338 | |
parent | 83e5d8f7253cb7b14472385a6d57df1e9f848e8e (diff) |
[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: Benjamin LaHaise <benjamin.c.lahaise@intel.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | include/linux/page-flags.h | 2 | ||||
-rw-r--r-- | mm/page_alloc.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/page-flags.h b/include/linux/page-flags.h index f2ee9b2332e3..f5a6695d4d21 100644 --- a/include/linux/page-flags.h +++ b/include/linux/page-flags.h | |||
@@ -135,7 +135,7 @@ struct page_state { | |||
135 | 135 | ||
136 | extern void get_page_state(struct page_state *ret); | 136 | extern void get_page_state(struct page_state *ret); |
137 | extern void get_full_page_state(struct page_state *ret); | 137 | extern void get_full_page_state(struct page_state *ret); |
138 | extern unsigned long __read_page_state(unsigned offset); | 138 | extern unsigned long __read_page_state(unsigned long offset); |
139 | extern void __mod_page_state(unsigned long offset, unsigned long delta); | 139 | extern void __mod_page_state(unsigned long offset, unsigned long delta); |
140 | 140 | ||
141 | #define read_page_state(member) \ | 141 | #define read_page_state(member) \ |
diff --git a/mm/page_alloc.c b/mm/page_alloc.c index c976628f9d95..10446a646374 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c | |||
@@ -1144,7 +1144,7 @@ void get_full_page_state(struct page_state *ret) | |||
1144 | __get_page_state(ret, sizeof(*ret) / sizeof(unsigned long)); | 1144 | __get_page_state(ret, sizeof(*ret) / sizeof(unsigned long)); |
1145 | } | 1145 | } |
1146 | 1146 | ||
1147 | unsigned long __read_page_state(unsigned offset) | 1147 | unsigned long __read_page_state(unsigned long offset) |
1148 | { | 1148 | { |
1149 | unsigned long ret = 0; | 1149 | unsigned long ret = 0; |
1150 | int cpu; | 1150 | int cpu; |