buildtools: fix experimental symbols listing
authorDavid Marchand <david.marchand@redhat.com>
Mon, 25 Nov 2019 08:10:06 +0000 (09:10 +0100)
committerDavid Marchand <david.marchand@redhat.com>
Tue, 26 Nov 2019 08:04:36 +0000 (09:04 +0100)
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 <david.marchand@redhat.com>
Tested-by: Andrew Rybchenko <arybchenko@solarflare.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
buildtools/check-experimental-syms.sh

index 145dd70..abebb89 100755 (executable)
@@ -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