hw-breakpoints: Simplify error handling in breakpoint creation requests

This simplifies the error handling when we create a breakpoint.
We don't need to check the NULL return value corner case anymore
since we have improved perf_event_create_kernel_counter() to
always return an error code in the failure case.

Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Prasad <prasad@linux.vnet.ibm.com>
LKML-Reference: <1259210142-5714-3-git-send-regression-fweisbec@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>

2 files changed, 6 insertions, 14 deletions

diff --git a/kernel/hw_breakpoint.c b/kernel/hw_breakpoint.c
index 06d372fc026d..dd3fb4a999d3 100644
--- a/kernel/hw_breakpoint.c
+++ b/kernel/hw_breakpoint.c
@@ -442,7 +442,7 @@ register_wide_hw_breakpoint(unsigned long addr,
 
                 *pevent = bp;
 
-                if (IS_ERR(bp) || !bp) {
+                if (IS_ERR(bp)) {
                         err = PTR_ERR(bp);
                         goto fail;
                 }
@@ -453,7 +453,7 @@ register_wide_hw_breakpoint(unsigned long addr,
 fail:
         for_each_possible_cpu(cpu) {
                 pevent = per_cpu_ptr(cpu_events, cpu);
-                if (IS_ERR(*pevent) || !*pevent)
+                if (IS_ERR(*pevent))
                         break;
                 unregister_hw_breakpoint(*pevent);
         }
diff --git a/kernel/trace/trace_ksym.c b/kernel/trace/trace_ksym.c
index 9f040e42f516..c538b15b95d6 100644
--- a/kernel/trace/trace_ksym.c
+++ b/kernel/trace/trace_ksym.c
@@ -200,12 +200,9 @@ int process_new_ksym_entry(char *ksymname, int op, unsigned long addr)
         entry->ksym_hbp = register_wide_hw_breakpoint(entry->ksym_addr,
                                         entry->len, entry->type,
                                         ksym_hbp_handler, true);
+
         if (IS_ERR(entry->ksym_hbp)) {
-                entry->ksym_hbp = NULL;
                 ret = PTR_ERR(entry->ksym_hbp);
-        }
-
-        if (!entry->ksym_hbp) {
                 printk(KERN_INFO "ksym_tracer request failed. Try again"
                                         " later!!\n");
                 goto err;
@@ -332,21 +329,16 @@ static ssize_t ksym_trace_filter_write(struct file *file,
         if (changed) {
                 unregister_wide_hw_breakpoint(entry->ksym_hbp);
                 entry->type = op;
+                ret = 0;
                 if (op > 0) {
                         entry->ksym_hbp =
                                 register_wide_hw_breakpoint(entry->ksym_addr,
                                         entry->len, entry->type,
                                         ksym_hbp_handler, true);
                         if (IS_ERR(entry->ksym_hbp))
-                                entry->ksym_hbp = NULL;
-
-                        /* modified without problem */
-                        if (entry->ksym_hbp) {
-                                ret = 0;
+                                ret = PTR_ERR(entry->ksym_hbp);
+                        else
                                 goto out;
-                        }
-                } else {
-                        ret = 0;
                 }
                 /* Error or "symbol:---" case: drop it */
                 ksym_filter_entry_count--;