diff options
author | Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it> | 2006-03-31 05:30:18 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-31 15:18:51 -0500 |
commit | 12523bdce1a6b5a3a2627b9901162bbd3eef33ed (patch) | |
tree | 02e9807abdd6df71468d61a9883510541f765ecb /arch/um | |
parent | e11c0cdf4c6f7976e6f4fe221369a7b420245389 (diff) |
[PATCH] uml: idle thread needn't take access to init_mm
Comparing this code which is the actual body of the arch-independent
cpu_idle(), it is clear that it's unnecessary to set ->mm and ->active_mm;
beyond that, a kernel thread is not supposed to have ->mm != NULL, only
active_mm.
This showed up because I used the assumption (which is IMHO valid) that kernel
thread have their ->mm == NULL, and it failed for this thread.
Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>
Acked-by: Jeff Dike <jdike@addtoit.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/um')
-rw-r--r-- | arch/um/kernel/process_kern.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/arch/um/kernel/process_kern.c b/arch/um/kernel/process_kern.c index 6922bdfb5ad5..ba8a52c1f7ae 100644 --- a/arch/um/kernel/process_kern.c +++ b/arch/um/kernel/process_kern.c | |||
@@ -185,10 +185,6 @@ void default_idle(void) | |||
185 | { | 185 | { |
186 | CHOOSE_MODE(uml_idle_timer(), (void) 0); | 186 | CHOOSE_MODE(uml_idle_timer(), (void) 0); |
187 | 187 | ||
188 | atomic_inc(&init_mm.mm_count); | ||
189 | current->mm = &init_mm; | ||
190 | current->active_mm = &init_mm; | ||
191 | |||
192 | while(1){ | 188 | while(1){ |
193 | /* endless idle loop with no priority at all */ | 189 | /* endless idle loop with no priority at all */ |
194 | 190 | ||