Messages in this thread |  | | Subject | Re: [PATCH 1/8] tools lib bpf: add error functions | From | "Wangnan (F)" <> | Date | Mon, 17 Oct 2016 09:47:14 +0800 |
| |
On 2016/10/17 5:18, Eric Leblond wrote: > The include of err.h is not explicitely needed in exported > functions and it was causing include conflict with some existing > code due to redefining some macros. > > To fix this, let's have error handling functions provided by the > library. Furthermore this will allow user to have an homogeneous > API. > > Signed-off-by: Eric Leblond <eric@regit.org> > --- > tools/lib/bpf/libbpf.c | 11 +++++++++++ > tools/lib/bpf/libbpf.h | 4 +++- > 2 files changed, 14 insertions(+), 1 deletion(-) > > diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c > index b699aea..90932f1 100644 > --- a/tools/lib/bpf/libbpf.c > +++ b/tools/lib/bpf/libbpf.c > @@ -31,6 +31,7 @@ > #include <linux/kernel.h> > #include <linux/bpf.h> > #include <linux/list.h> > +#include <linux/err.h> > #include <libelf.h> > #include <gelf.h> > > @@ -1447,3 +1448,13 @@ bpf_object__find_map_by_name(struct bpf_object *obj, const char *name) > } > return NULL; > } > + > +bool bpf__is_error(const void *ptr)
Please use libbpf_is_error(), like libbpf_set_print. We use '__' because we want to use the OO concept. This utility is not OO.
> +{ > + return IS_ERR(ptr); > +} > + > +long bpf__get_error(const void *ptr)
Same, please call it libbpf_get_error().
Thank you.
|  |