diff options
-rw-r--r-- | arch/arm/oprofile/common.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/arch/arm/oprofile/common.c b/arch/arm/oprofile/common.c index 7ae9eebe7efc..cec9305c1cce 100644 --- a/arch/arm/oprofile/common.c +++ b/arch/arm/oprofile/common.c | |||
@@ -89,28 +89,28 @@ static void op_perf_setup(void) | |||
89 | 89 | ||
90 | static int op_create_counter(int cpu, int event) | 90 | static int op_create_counter(int cpu, int event) |
91 | { | 91 | { |
92 | int ret = 0; | ||
93 | struct perf_event *pevent; | 92 | struct perf_event *pevent; |
94 | 93 | ||
95 | if (!counter_config[event].enabled || (perf_events[cpu][event] != NULL)) | 94 | if (!counter_config[event].enabled || perf_events[cpu][event]) |
96 | return ret; | 95 | return 0; |
97 | 96 | ||
98 | pevent = perf_event_create_kernel_counter(&counter_config[event].attr, | 97 | pevent = perf_event_create_kernel_counter(&counter_config[event].attr, |
99 | cpu, -1, | 98 | cpu, -1, |
100 | op_overflow_handler); | 99 | op_overflow_handler); |
101 | 100 | ||
102 | if (IS_ERR(pevent)) { | 101 | if (IS_ERR(pevent)) |
103 | ret = PTR_ERR(pevent); | 102 | return PTR_ERR(pevent); |
104 | } else if (pevent->state != PERF_EVENT_STATE_ACTIVE) { | 103 | |
104 | if (pevent->state != PERF_EVENT_STATE_ACTIVE) { | ||
105 | perf_event_release_kernel(pevent); | 105 | perf_event_release_kernel(pevent); |
106 | pr_warning("oprofile: failed to enable event %d " | 106 | pr_warning("oprofile: failed to enable event %d " |
107 | "on CPU %d\n", event, cpu); | 107 | "on CPU %d\n", event, cpu); |
108 | ret = -EBUSY; | 108 | return -EBUSY; |
109 | } else { | ||
110 | perf_events[cpu][event] = pevent; | ||
111 | } | 109 | } |
112 | 110 | ||
113 | return ret; | 111 | perf_events[cpu][event] = pevent; |
112 | |||
113 | return 0; | ||
114 | } | 114 | } |
115 | 115 | ||
116 | static void op_destroy_counter(int cpu, int event) | 116 | static void op_destroy_counter(int cpu, int event) |