diff options
author | Masami Hiramatsu <mhiramat@redhat.com> | 2009-08-27 13:23:25 -0400 |
---|---|---|
committer | Frederic Weisbecker <fweisbec@gmail.com> | 2009-08-29 21:08:27 -0400 |
commit | 8222d718b3ad3ae49c48f69ae4b6a1128c9a92cf (patch) | |
tree | 099a0eea1cae74c0ab24fe6ec3edb3cf4e3ac3e3 | |
parent | 8f270083587a4cb70fa14f0e2fd698eb08a4dd07 (diff) |
kprobes/x86-64: Fix to move common_interrupt to .kprobes.text
Since nmi, debug and int3 returns to irq_return inside common_interrupt,
probing this function will cause int3-loop, so it should be marked
as __kprobes.
Signed-off-by: Masami Hiramatsu <mhiramat@redhat.com>
Acked-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
Cc: Ingo Molnar <mingo@elte.hu>
LKML-Reference: <20090827172325.8246.40000.stgit@localhost.localdomain>
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
-rw-r--r-- | arch/x86/kernel/entry_64.S | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/arch/x86/kernel/entry_64.S b/arch/x86/kernel/entry_64.S index c251be745107..36e2ef5cc83f 100644 --- a/arch/x86/kernel/entry_64.S +++ b/arch/x86/kernel/entry_64.S | |||
@@ -809,6 +809,10 @@ END(interrupt) | |||
809 | call \func | 809 | call \func |
810 | .endm | 810 | .endm |
811 | 811 | ||
812 | /* | ||
813 | * Interrupt entry/exit should be protected against kprobes | ||
814 | */ | ||
815 | .pushsection .kprobes.text, "ax" | ||
812 | /* | 816 | /* |
813 | * The interrupt stubs push (~vector+0x80) onto the stack and | 817 | * The interrupt stubs push (~vector+0x80) onto the stack and |
814 | * then jump to common_interrupt. | 818 | * then jump to common_interrupt. |
@@ -947,6 +951,10 @@ ENTRY(retint_kernel) | |||
947 | 951 | ||
948 | CFI_ENDPROC | 952 | CFI_ENDPROC |
949 | END(common_interrupt) | 953 | END(common_interrupt) |
954 | /* | ||
955 | * End of kprobes section | ||
956 | */ | ||
957 | .popsection | ||
950 | 958 | ||
951 | /* | 959 | /* |
952 | * APIC interrupts. | 960 | * APIC interrupts. |