diff options
author | Oleg Nesterov <oleg@redhat.com> | 2014-06-04 19:07:55 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-06-04 19:54:03 -0400 |
commit | dc6f6c97f1d3d58fef81f0f9db0c7d068b2cf392 (patch) | |
tree | 14b7ab2ab8060b41a7e262ea36b0dee1b2809e30 /init | |
parent | 39af1765f1255b2bbadc3064e16270781abf24a1 (diff) |
memcg: kill start_kernel()->mm_init_owner(&init_mm)
Remove start_kernel()->mm_init_owner(&init_mm, &init_task).
This doesn't really hurt but unnecessary and misleading. init_task is the
"swapper" thread == current, its ->mm is always NULL. And init_mm can
only be used as ->active_mm, not as ->mm.
mm_init_owner() has a single caller with this patch, perhaps it should
die. mm_init() can initialize ->owner under #ifdef.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Reviewed-by: Michal Hocko <mhocko@suse.cz>
Cc: Balbir Singh <bsingharora@gmail.com>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Michal Hocko <mhocko@suse.cz>
Cc: Peter Chiang <pchiang@nvidia.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'init')
-rw-r--r-- | init/main.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/init/main.c b/init/main.c index 48655ceb66f4..e08c0b2065a1 100644 --- a/init/main.c +++ b/init/main.c | |||
@@ -507,7 +507,6 @@ asmlinkage __visible void __init start_kernel(void) | |||
507 | page_address_init(); | 507 | page_address_init(); |
508 | pr_notice("%s", linux_banner); | 508 | pr_notice("%s", linux_banner); |
509 | setup_arch(&command_line); | 509 | setup_arch(&command_line); |
510 | mm_init_owner(&init_mm, &init_task); | ||
511 | mm_init_cpumask(&init_mm); | 510 | mm_init_cpumask(&init_mm); |
512 | setup_command_line(command_line); | 511 | setup_command_line(command_line); |
513 | setup_nr_cpu_ids(); | 512 | setup_nr_cpu_ids(); |