Skip to content

Commit 83e5d8f

Browse files
bcrlLinus Torvalds
authored andcommitted
[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>
1 parent 1ad539b commit 83e5d8f

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

include/linux/page-flags.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ struct page_state {
136136
extern void get_page_state(struct page_state *ret);
137137
extern void get_full_page_state(struct page_state *ret);
138138
extern unsigned long __read_page_state(unsigned offset);
139-
extern void __mod_page_state(unsigned offset, unsigned long delta);
139+
extern void __mod_page_state(unsigned long offset, unsigned long delta);
140140

141141
#define read_page_state(member) \
142142
__read_page_state(offsetof(struct page_state, member))

mm/page_alloc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1158,7 +1158,7 @@ unsigned long __read_page_state(unsigned offset)
11581158
return ret;
11591159
}
11601160

1161-
void __mod_page_state(unsigned offset, unsigned long delta)
1161+
void __mod_page_state(unsigned long offset, unsigned long delta)
11621162
{
11631163
unsigned long flags;
11641164
void* ptr;

0 commit comments

Comments
 (0)