diff options
author | Masami Hiramatsu <mhiramat@kernel.org> | 2018-09-11 19:20:11 +0900 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2018-09-12 08:01:16 +0200 |
commit | a6d18e65dff2b73ceeb187c598b48898e36ad7b1 (patch) | |
tree | 6060d7b2a4443773b87e4eed6e22a8858ab0995b /kernel/kprobes.c | |
parent | c72e6742f62d7bb82a77a41ca53940cb8f73e60f (diff) | |
download | lwn-a6d18e65dff2b73ceeb187c598b48898e36ad7b1.tar.gz lwn-a6d18e65dff2b73ceeb187c598b48898e36ad7b1.zip |
kprobes: Remove pointless BUG_ON() from reuse_unused_kprobe()
Since reuse_unused_kprobe() is called when the given kprobe
is unused, checking it inside again with BUG_ON() is
pointless. Remove it.
Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>
Cc: Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>
Cc: David S . Miller <davem@davemloft.net>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Naveen N . Rao <naveen.n.rao@linux.vnet.ibm.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/153666121154.21306.17540752948574483565.stgit@devbox
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'kernel/kprobes.c')
-rw-r--r-- | kernel/kprobes.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/kernel/kprobes.c b/kernel/kprobes.c index 231569e1e2c8..277a6cbe83db 100644 --- a/kernel/kprobes.c +++ b/kernel/kprobes.c @@ -704,7 +704,6 @@ static void reuse_unused_kprobe(struct kprobe *ap) { struct optimized_kprobe *op; - BUG_ON(!kprobe_unused(ap)); /* * Unused kprobe MUST be on the way of delayed unoptimizing (means * there is still a relative jump) and disabled. |