From 72e75e1f1bd72f22e5a31d9d0a95b46ddf247c54 Mon Sep 17 00:00:00 2001 From: David Marchand Date: Mon, 25 Nov 2019 09:10:06 +0100 Subject: [PATCH] buildtools: fix experimental symbols listing The map-list-symbol.sh script displays the filename, section and symbol names of map files. Example: $ buildtools/map-list-symbol.sh -S EXPERIMENTAL \ lib/librte_ethdev/rte_ethdev_version.map |grep rte_mtr_create lib/librte_ethdev/rte_ethdev_version.map EXPERIMENTAL rte_mtr_create The experimental symbol check should only consider the symbol name. Fixes: 3290ac14eb94 ("buildtools: detect discrepancies for experimental symbols") Cc: stable@dpdk.org Signed-off-by: David Marchand Tested-by: Andrew Rybchenko Acked-by: Neil Horman --- buildtools/check-experimental-syms.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/buildtools/check-experimental-syms.sh b/buildtools/check-experimental-syms.sh index 145dd70ebf..abebb89f12 100755 --- a/buildtools/check-experimental-syms.sh +++ b/buildtools/check-experimental-syms.sh @@ -23,7 +23,7 @@ trap 'rm -f "$DUMPFILE"' EXIT objdump -t $OBJFILE >$DUMPFILE ret=0 -for SYM in `$LIST_SYMBOL -S EXPERIMENTAL $MAPFILE` +for SYM in `$LIST_SYMBOL -S EXPERIMENTAL $MAPFILE |cut -d ' ' -f 3` do if grep -q "\.text.*$SYM$" $DUMPFILE && ! grep -q "\.text\.experimental.*$SYM$" $DUMPFILE -- 2.20.1