diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2016-06-03 11:36:39 -0400 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2016-06-06 17:19:34 -0400 |
| commit | be834ffbd15ea9d73ba96fdbdcb1012add7e3bdf (patch) | |
| tree | 36a00d108c3d86f8b544d177539e393ea35a7f64 /tools/lib | |
| parent | a7fe0450b0142d0eb4a543840a43e22682debf25 (diff) | |
tools lib bpf: Make bpf_program__get_private() use IS_ERR()
For consistency with bpf_map__priv() and elsewhere.
Acked-by: Wang Nan <wangnan0@huawei.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Milian Wolff <milian.wolff@kdab.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/n/tip-x17nk5mrazkf45z0l0ahlmo8@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/lib')
| -rw-r--r-- | tools/lib/bpf/libbpf.c | 5 | ||||
| -rw-r--r-- | tools/lib/bpf/libbpf.h | 3 |
2 files changed, 3 insertions, 5 deletions
diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c index 0412182fb365..7eb7fb26e999 100644 --- a/tools/lib/bpf/libbpf.c +++ b/tools/lib/bpf/libbpf.c | |||
| @@ -1230,10 +1230,9 @@ int bpf_program__set_private(struct bpf_program *prog, | |||
| 1230 | return 0; | 1230 | return 0; |
| 1231 | } | 1231 | } |
| 1232 | 1232 | ||
| 1233 | int bpf_program__get_private(struct bpf_program *prog, void **ppriv) | 1233 | void *bpf_program__priv(struct bpf_program *prog) |
| 1234 | { | 1234 | { |
| 1235 | *ppriv = prog->priv; | 1235 | return prog ? prog->priv : ERR_PTR(-EINVAL); |
| 1236 | return 0; | ||
| 1237 | } | 1236 | } |
| 1238 | 1237 | ||
| 1239 | const char *bpf_program__title(struct bpf_program *prog, bool needs_copy) | 1238 | const char *bpf_program__title(struct bpf_program *prog, bool needs_copy) |
diff --git a/tools/lib/bpf/libbpf.h b/tools/lib/bpf/libbpf.h index ea65775e8302..372cecbde207 100644 --- a/tools/lib/bpf/libbpf.h +++ b/tools/lib/bpf/libbpf.h | |||
| @@ -81,8 +81,7 @@ typedef void (*bpf_program_clear_priv_t)(struct bpf_program *, | |||
| 81 | int bpf_program__set_private(struct bpf_program *prog, void *priv, | 81 | int bpf_program__set_private(struct bpf_program *prog, void *priv, |
| 82 | bpf_program_clear_priv_t clear_priv); | 82 | bpf_program_clear_priv_t clear_priv); |
| 83 | 83 | ||
| 84 | int bpf_program__get_private(struct bpf_program *prog, | 84 | void *bpf_program__priv(struct bpf_program *prog); |
| 85 | void **ppriv); | ||
| 86 | 85 | ||
| 87 | const char *bpf_program__title(struct bpf_program *prog, bool needs_copy); | 86 | const char *bpf_program__title(struct bpf_program *prog, bool needs_copy); |
| 88 | 87 | ||
