diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2011-03-17 11:02:15 +0100 |
---|---|---|
committer | Clark Williams <williams@redhat.com> | 2011-11-14 11:01:25 -0600 |
commit | 7a6c1ab23db02c3f37cd6d03c17b2159e62b4e4b (patch) | |
tree | 0f9ae496803810d025cad4baea63098a12b8d409 /arch | |
parent | b07f1aacd86c90168fec44bdf2b3de7ffb7280d7 (diff) |
x86: kprobes: Remove remove bogus preempt_enable
The CONFIG_PREEMPT=n section of setup_singlestep() contains:
preempt_enable_no_resched();
That's bogus as it is asymetric - no preempt_disable() - and it just
never blew up because preempt_enable_no_resched() is a NOP when
CONFIG_PREEMPT=n. Remove it.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/x86/kernel/kprobes.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/arch/x86/kernel/kprobes.c b/arch/x86/kernel/kprobes.c index 7da647d8b64c..5604455ebc75 100644 --- a/arch/x86/kernel/kprobes.c +++ b/arch/x86/kernel/kprobes.c @@ -478,7 +478,6 @@ static void __kprobes setup_singlestep(struct kprobe *p, struct pt_regs *regs, * stepping. */ regs->ip = (unsigned long)p->ainsn.insn; - preempt_enable_no_resched(); return; } #endif |