diff mbox series

nm01: Remove prefix zeros of the addresses output by nm before comparing

Message ID 1577257063-246792-1-git-send-email-zhe.he@windriver.com
State Changes Requested
Delegated to: Petr Vorel
Headers show
Series nm01: Remove prefix zeros of the addresses output by nm before comparing | expand

Commit Message

He Zhe Dec. 25, 2019, 6:57 a.m. UTC
From: He Zhe <zhe.he@windriver.com>

The latest nm v2.33.1.20191208 outputs symbols addresses without prefix zeros
for "nm -f posix", which causes the following error.
nm01 5 TFAIL: Got wrong format with -f bsd

Let's remove the prefix zeros before comparing.

Signed-off-by: He Zhe <zhe.he@windriver.com>
---
 testcases/commands/nm/nm01.sh | 3 +++
 1 file changed, 3 insertions(+)

Comments

Li Wang Dec. 30, 2019, 1:13 p.m. UTC | #1
Hi Zhe,

On Wed, Dec 25, 2019 at 2:58 PM <zhe.he@windriver.com> wrote:

> From: He Zhe <zhe.he@windriver.com>
>
> The latest nm v2.33.1.20191208 outputs symbols addresses without prefix
> zeros
> for "nm -f posix", which causes the following error.
> nm01 5 TFAIL: Got wrong format with -f bsd
>
> Let's remove the prefix zeros before comparing.
>
> Signed-off-by: He Zhe <zhe.he@windriver.com>
> ---
>  testcases/commands/nm/nm01.sh | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/testcases/commands/nm/nm01.sh b/testcases/commands/nm/nm01.sh
> index 30c41bd..29b6bd7 100755
> --- a/testcases/commands/nm/nm01.sh
> +++ b/testcases/commands/nm/nm01.sh
> @@ -84,6 +84,9 @@ test5()
>         EXPECT_PASS $NM -f bsd $TST_DATAROOT/f1 \> nm_bsd.out
>         EXPECT_PASS $NM -f posix $TST_DATAROOT/f1 \> nm_posix.out
>
> +       ROD awk '{print gensub(/(0+)([0-9a-fA-F]+)/, "\\2", "g")}'
> nm_bsd.out \> nm_bsd.out
>

AWK is base on stream editing, maybe we need the output to a new log file.

> +       ROD awk '{print gensub(/(0+)([0-9a-fA-F]+)/, "\\2", "g")}'
> nm_posix.out \> nm_posix.out
>

The regex is not fit for nm_posix.out since this format of address does not
start with 0000000000, e.g.

# cat nm_posix.out | grep 40108f
.annobin_init.c t 40108f
.annobin_init.c_end t 40108f

# echo 40108f | awk '{print gensub(/(0+)([0-9a-fA-F]+)/, "\\2", "g")}'
4108f


> +
>         ROD awk '{print $3 $2 $1}' nm_bsd.out \> nm1.out
>         ROD awk '{print $1 $2 $3}' nm_posix.out \> nm2.out
>
> --
> 2.7.4
>
>
> --
> Mailing list info: https://lists.linux.it/listinfo/ltp
>
>
He Zhe Dec. 31, 2019, 7:02 a.m. UTC | #2
Thanks for pointing it out. I'll send v2.

Zhe

On 12/30/19 9:13 PM, Li Wang wrote:
> Hi Zhe,
>
> On Wed, Dec 25, 2019 at 2:58 PM <zhe.he@windriver.com <mailto:zhe.he@windriver.com>> wrote:
>
>     From: He Zhe <zhe.he@windriver.com <mailto:zhe.he@windriver.com>>
>
>     The latest nm v2.33.1.20191208 outputs symbols addresses without prefix zeros
>     for "nm -f posix", which causes the following error.
>     nm01 5 TFAIL: Got wrong format with -f bsd
>
>     Let's remove the prefix zeros before comparing.
>
>     Signed-off-by: He Zhe <zhe.he@windriver.com <mailto:zhe.he@windriver.com>>
>     ---
>      testcases/commands/nm/nm01.sh | 3 +++
>      1 file changed, 3 insertions(+)
>
>     diff --git a/testcases/commands/nm/nm01.sh b/testcases/commands/nm/nm01.sh
>     index 30c41bd..29b6bd7 100755
>     --- a/testcases/commands/nm/nm01.sh
>     +++ b/testcases/commands/nm/nm01.sh
>     @@ -84,6 +84,9 @@ test5()
>             EXPECT_PASS $NM -f bsd $TST_DATAROOT/f1 \> nm_bsd.out
>             EXPECT_PASS $NM -f posix $TST_DATAROOT/f1 \> nm_posix.out
>
>     +       ROD awk '{print gensub(/(0+)([0-9a-fA-F]+)/, "\\2", "g")}' nm_bsd.out \> nm_bsd.out
>
>
> AWK is base on stream editing, maybe we need the output to a new log file.
>
>     +       ROD awk '{print gensub(/(0+)([0-9a-fA-F]+)/, "\\2", "g")}' nm_posix.out \> nm_posix.out
>
>
> The regex is not fit for nm_posix.out since this format of address does not start with 0000000000, e.g.
>
> # cat nm_posix.out | grep 40108f
> .annobin_init.c t 40108f
> .annobin_init.c_end t 40108f 
>
> # echo 40108f |awk '{print gensub(/(0+)([0-9a-fA-F]+)/, "\\2", "g")}'
> 4108f
>  
>
>     +
>             ROD awk '{print $3 $2 $1}' nm_bsd.out \> nm1.out
>             ROD awk '{print $1 $2 $3}' nm_posix.out \> nm2.out
>
>     -- 
>     2.7.4
>
>
>     -- 
>     Mailing list info: https://lists.linux.it/listinfo/ltp
>
>
>
> -- 
> Regards,
> Li Wang
diff mbox series

Patch

diff --git a/testcases/commands/nm/nm01.sh b/testcases/commands/nm/nm01.sh
index 30c41bd..29b6bd7 100755
--- a/testcases/commands/nm/nm01.sh
+++ b/testcases/commands/nm/nm01.sh
@@ -84,6 +84,9 @@  test5()
 	EXPECT_PASS $NM -f bsd $TST_DATAROOT/f1 \> nm_bsd.out
 	EXPECT_PASS $NM -f posix $TST_DATAROOT/f1 \> nm_posix.out
 
+	ROD awk '{print gensub(/(0+)([0-9a-fA-F]+)/, "\\2", "g")}' nm_bsd.out \> nm_bsd.out
+	ROD awk '{print gensub(/(0+)([0-9a-fA-F]+)/, "\\2", "g")}' nm_posix.out \> nm_posix.out
+
 	ROD awk '{print $3 $2 $1}' nm_bsd.out \> nm1.out
 	ROD awk '{print $1 $2 $3}' nm_posix.out \> nm2.out