diff options
author | Andi Kleen <ak@suse.de> | 2006-10-05 12:47:22 -0400 |
---|---|---|
committer | Andi Kleen <andi@basil.nowhere.org> | 2006-10-05 12:47:22 -0400 |
commit | 51ec28e1b2cb267a09a0b8eac1ccf8d61b7268bf (patch) | |
tree | 4686d439a49b98b98fc5048641ddfb50db3fbcd6 /arch/x86_64 | |
parent | f015c6c4d733f68cbc1c5d231bb158abaa5c9606 (diff) |
[PATCH] x86: Terminate the kernel stacks for the unwinder
Always make sure RIP/EIP is 0 in the registers stored on the top
of the stack of a kernel thread. This makes sure the unwinder code
won't try a fallback but knows the stack has ended.
AK: this patch is a bit mysterious. in theory they should be terminated
anyways, but it seems to fix at least one crash. Anyways double termination
probably doesn't hurt.
Signed-off-by: Andi Kleen <ak@suse.de>
Diffstat (limited to 'arch/x86_64')
-rw-r--r-- | arch/x86_64/kernel/entry.S | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/arch/x86_64/kernel/entry.S b/arch/x86_64/kernel/entry.S index b8285cf1a9c3..ab9b2c4726f2 100644 --- a/arch/x86_64/kernel/entry.S +++ b/arch/x86_64/kernel/entry.S | |||
@@ -978,6 +978,11 @@ ENTRY(kernel_thread) | |||
978 | call do_fork | 978 | call do_fork |
979 | movq %rax,RAX(%rsp) | 979 | movq %rax,RAX(%rsp) |
980 | xorl %edi,%edi | 980 | xorl %edi,%edi |
981 | test %rax,%rax | ||
982 | jnz 1f | ||
983 | /* terminate stack in child */ | ||
984 | movq %rdi,RIP(%rsp) | ||
985 | 1: | ||
981 | 986 | ||
982 | /* | 987 | /* |
983 | * It isn't worth to check for reschedule here, | 988 | * It isn't worth to check for reschedule here, |