Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)).
This makes it more readable and also fix this warning detected
by err_cast.cocci:
tools/perf/util/bpf-loader.c:1606:11-18: WARNING: ERR_CAST can be used with op

Signed-off-by: Wen Yang <wen.yan...@zte.com.cn>
CC: Peter Zijlstra <pet...@infradead.org>
CC: Arnaldo Carvalho de Melo <a...@kernel.org>
CC: Alexander Shishkin <alexander.shish...@linux.intel.com>
CC: Jiri Olsa <jo...@redhat.com>
CC: Namhyung Kim <namhy...@kernel.org>
CC: Julia Lawall <julia.law...@lip6.fr>
CC: Wen Yang <yellowriver2...@hotmail.com>
CC: linux-kernel@vger.kernel.org
---
 tools/perf/util/bpf-loader.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/util/bpf-loader.c b/tools/perf/util/bpf-loader.c
index f9ae1a993806..9a280647d829 100644
--- a/tools/perf/util/bpf-loader.c
+++ b/tools/perf/util/bpf-loader.c
@@ -1603,7 +1603,7 @@ struct perf_evsel *bpf__setup_output_event(struct 
perf_evlist *evlist, const cha
 
                        op = bpf_map__add_newop(map, NULL);
                        if (IS_ERR(op))
-                               return ERR_PTR(PTR_ERR(op));
+                               return ERR_CAST(op);
                        op->op_type = BPF_MAP_OP_SET_EVSEL;
                        op->v.evsel = evsel;
                }
-- 
2.19.1

Reply via email to