diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2007-05-02 13:27:09 -0400 |
---|---|---|
committer | Andi Kleen <andi@basil.nowhere.org> | 2007-05-02 13:27:09 -0400 |
commit | 692174b97d5b871f4b0f648b1fb17aa37b955876 (patch) | |
tree | 17a91a0549cb5dbd22b7375becd937ee13d193d4 /include | |
parent | 1b523fb54977c9bb81b16c4badce581a2b455994 (diff) |
[PATCH] i386: Initialize esp0 properly all the time
Whenever we schedule, __switch_to calls load_esp0 which does:
tss->esp0 = thread->esp0;
This is never initialized for the initial thread (ie "swapper"), so when we're
scheduling that, we end up setting esp0 to 0. This is fine: the swapper never
leaves ring 0, so this field is never used.
lguest, however, gets upset that we're trying to used an unmapped page as our
kernel stack. Rather than work around it there, let's initialize it.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Andi Kleen <ak@suse.de>
Cc: Andi Kleen <ak@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/asm-i386/processor.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/asm-i386/processor.h b/include/asm-i386/processor.h index 11bf899de8aa..01ae0ffcf236 100644 --- a/include/asm-i386/processor.h +++ b/include/asm-i386/processor.h | |||
@@ -421,6 +421,7 @@ struct thread_struct { | |||
421 | }; | 421 | }; |
422 | 422 | ||
423 | #define INIT_THREAD { \ | 423 | #define INIT_THREAD { \ |
424 | .esp0 = sizeof(init_stack) + (long)&init_stack, \ | ||
424 | .vm86_info = NULL, \ | 425 | .vm86_info = NULL, \ |
425 | .sysenter_cs = __KERNEL_CS, \ | 426 | .sysenter_cs = __KERNEL_CS, \ |
426 | .io_bitmap_ptr = NULL, \ | 427 | .io_bitmap_ptr = NULL, \ |