perf probe: Cleanup find_perf_probe_point_from_map to reduce redundancy
authorMasami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Fri, 6 Nov 2015 08:30:03 +0000 (17:30 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 6 Nov 2015 13:47:33 +0000 (10:47 -0300)
commit0a62f6869f2768687af2c94d97f3b2fcf5b73367
tree3c5dee18c74a0d40c7edf0d67d862cf2c88cd366
parent62ec9b3f02a9bccaf699bd4691db98f779c3075f
perf probe: Cleanup find_perf_probe_point_from_map to reduce redundancy

In find_perf_probe_point_from_map(), the 'ret' variable is initialized
with -ENOENT but overwritten by the return code of
kernel_get_symbol_address_by_name(), and after that it is re-initialized
with -ENOENT again.

Setting ret=-ENOENT twice looks a bit redundant. This avoids the
overwriting and just returns -ENOENT if some error happens to simplify
the code.

Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Cc: Zefan Li <lizefan@huawei.com>
Link: http://lkml.kernel.org/n/tip-ufp1zgbktzmttcputozneomd@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/probe-event.c