diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2011-03-17 11:02:15 +0100 |
|---|---|---|
| committer | Clark Williams <williams@redhat.com> | 2012-04-13 11:01:14 -0500 |
| commit | 2ba7dbe93e0b4678ce77c9446a53ea30564039da (patch) | |
| tree | 47201efb7b54914a0fe632d9f599f9996b456f40 /arch/x86/kernel | |
| parent | 1472e119c1563a9ad9125c0dcfbfea0bc3ec9a2f (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/x86/kernel')
| -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 e213fc8408d2..b9edf1a32cf8 100644 --- a/arch/x86/kernel/kprobes.c +++ b/arch/x86/kernel/kprobes.c @@ -486,7 +486,6 @@ setup_singlestep(struct kprobe *p, struct pt_regs *regs, struct kprobe_ctlblk *k * stepping. */ regs->ip = (unsigned long)p->ainsn.insn; - preempt_enable_no_resched(); return; } #endif |