On Fri, Jun 30, 2023 at 1:35 AM Jiri Olsa <jolsa@xxxxxxxxxx> wrote: > > Adding new elf object that will contain elf related functions. > There's no functional change. > > Suggested-by: Andrii Nakryiko <andrii@xxxxxxxxxx> > Signed-off-by: Jiri Olsa <jolsa@xxxxxxxxxx> > --- > tools/lib/bpf/Build | 2 +- > tools/lib/bpf/elf.c | 198 +++++++++++++++++++++++++++++++++++++ > tools/lib/bpf/libbpf.c | 186 +--------------------------------- > tools/lib/bpf/libbpf_elf.h | 11 +++ > 4 files changed, 211 insertions(+), 186 deletions(-) > create mode 100644 tools/lib/bpf/elf.c > create mode 100644 tools/lib/bpf/libbpf_elf.h > [...] > diff --git a/tools/lib/bpf/libbpf_elf.h b/tools/lib/bpf/libbpf_elf.h > new file mode 100644 > index 000000000000..1b652220fabf > --- /dev/null > +++ b/tools/lib/bpf/libbpf_elf.h > @@ -0,0 +1,11 @@ > +/* SPDX-License-Identifier: (LGPL-2.1 OR BSD-2-Clause) */ > + > +#ifndef __LIBBPF_LIBBPF_ELF_H > +#define __LIBBPF_LIBBPF_ELF_H > + > +#include <libelf.h> > + > +long elf_find_func_offset(Elf *elf, const char *binary_path, const char *name); > +long elf_find_func_offset_from_file(const char *binary_path, const char *name); > + > +#endif /* *__LIBBPF_LIBBPF_ELF_H */ we have libbpf_internal.h, let's put all this there for now, it's already all the internal stuff together, I don't know if separate header with few functions gives us much > -- > 2.41.0 >
