提交 24b8ff7c 编写于 作者: C Cesar Eduardo Barros 提交者: Linus Torvalds

mm: remove inline from scan_swap_map()

scan_swap_map() is a large function (224 lines), with several loops and a
complex control flow involving several gotos.

Given all that, it is a bit silly that it is marked as inline.  The
compiler agrees with me: on a x86-64 compile, it did not inline the
function.

Remove the "inline" and let the compiler decide instead.
Signed-off-by: NCesar Eduardo Barros <cesarb@cesarb.net>
Reviewed-by: NPekka Enberg <penberg@kernel.org>
Reviewed-by: NKOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Reviewed-by: NKAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Reviewed-by: NMinchan Kim <minchan.kim@gmail.com>
Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
上级 40531542
...@@ -212,8 +212,8 @@ static int wait_for_discard(void *word) ...@@ -212,8 +212,8 @@ static int wait_for_discard(void *word)
#define SWAPFILE_CLUSTER 256 #define SWAPFILE_CLUSTER 256
#define LATENCY_LIMIT 256 #define LATENCY_LIMIT 256
static inline unsigned long scan_swap_map(struct swap_info_struct *si, static unsigned long scan_swap_map(struct swap_info_struct *si,
unsigned char usage) unsigned char usage)
{ {
unsigned long offset; unsigned long offset;
unsigned long scan_base; unsigned long scan_base;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册