On Thu, May 7, 2020 at 7:57 AM Yauheni Kaliuta
<yauheni.kaliuta@xxxxxxxxxx> wrote:
>
> The patch makes it fail on the output when the comment is printed
> after the symbol name (RHEL8 powerpc):
>
> 400: 000000000000c714 144 FUNC GLOBAL DEFAULT 1 bpf_object__open_file@LIBBPF_0.0.4 [<localentry>: 8]
>
> But after commit aa915931ac3e ("libbpf: Fix readelf output parsing
> for Fedora") it is not needed anymore, the parsing should work in
> both cases.
>
> This reverts commit 3464afdf11f9a1e031e7858a05351ceca1792fea.
>
> Signed-off-by: Yauheni Kaliuta <yauheni.kaliuta@xxxxxxxxxx>
> ---
Looks good, though would be nice to have people originally involved in
those fixes you mentioned to confirm it works fine still. Added them
to cc.
If no one shouts loudly in next few days:
Acked-by: Andrii Nakryiko <andriin@xxxxxx>
> tools/lib/bpf/Makefile | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/lib/bpf/Makefile b/tools/lib/bpf/Makefile
> index aee7f1a83c77..908dac9eb562 100644
> --- a/tools/lib/bpf/Makefile
> +++ b/tools/lib/bpf/Makefile
> @@ -149,7 +149,7 @@ TAGS_PROG := $(if $(shell which etags 2>/dev/null),etags,ctags)
> GLOBAL_SYM_COUNT = $(shell readelf -s --wide $(BPF_IN_SHARED) | \
> cut -d "@" -f1 | sed 's/_v[0-9]_[0-9]_[0-9].*//' | \
> sed 's/\[.*\]//' | \
> - awk '/GLOBAL/ && /DEFAULT/ && !/UND/ {print $$NF}' | \
> + awk '/GLOBAL/ && /DEFAULT/ && !/UND/ {print $$8}' | \
> sort -u | wc -l)
> VERSIONED_SYM_COUNT = $(shell readelf -s --wide $(OUTPUT)libbpf.so | \
> grep -Eo '[^ ]+@LIBBPF_' | cut -d@ -f1 | sort -u | wc -l)
> @@ -216,7 +216,7 @@ check_abi: $(OUTPUT)libbpf.so
> readelf -s --wide $(BPF_IN_SHARED) | \
> cut -d "@" -f1 | sed 's/_v[0-9]_[0-9]_[0-9].*//' | \
> sed 's/\[.*\]//' | \
> - awk '/GLOBAL/ && /DEFAULT/ && !/UND/ {print $$NF}'| \
> + awk '/GLOBAL/ && /DEFAULT/ && !/UND/ {print $$8}'| \
> sort -u > $(OUTPUT)libbpf_global_syms.tmp; \
> readelf -s --wide $(OUTPUT)libbpf.so | \
> grep -Eo '[^ ]+@LIBBPF_' | cut -d@ -f1 | \
> --
> 2.26.2
>
