From: Kris Van Hees <kris.van.hees@oracle.com>
To: Sam James <sam@gentoo.org>
Cc: dtrace@lists.linux.dev
Subject: Re: [PATCH 3/6] test: awk -> gawk where we use strtonum
Date: Fri, 11 Oct 2024 23:30:20 -0400 [thread overview]
Message-ID: <ZwntTIoFuKtwlLOk@oracle.com> (raw)
In-Reply-To: <469de3f6038706e4589a497400090368c301c740.1724962546.git.sam@gentoo.org>
On Thu, Aug 29, 2024 at 09:15:43PM +0100, Sam James wrote:
Reviewed-by: Kris Van Hees <kris.van.hees@oracle.com>
... and committed with Signed-off-by added.
> ---
> test/unittest/aggs/tst.aggpercpu.sh | 2 +-
> test/unittest/arrays/tst.uregsarray-check.sh | 2 +-
> test/unittest/codegen/tst.kernel_read_scalar_regspill.r.p | 2 +-
> test/unittest/cpc/tst.args_kernel.sh | 2 +-
> test/unittest/cpc/tst.args_user.sh | 2 +-
> test/unittest/pid/tst.offsets.sh | 4 ++--
> test/unittest/profile-n/tst.args_kernel.sh | 2 +-
> test/unittest/profile-n/tst.args_user.sh | 2 +-
> 8 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/test/unittest/aggs/tst.aggpercpu.sh b/test/unittest/aggs/tst.aggpercpu.sh
> index 6092bd17..2103546c 100755
> --- a/test/unittest/aggs/tst.aggpercpu.sh
> +++ b/test/unittest/aggs/tst.aggpercpu.sh
> @@ -43,7 +43,7 @@ fi
> # Examine the results.
> #
>
> -awk '
> +gawk '
> # The expected value for the aggregation is aggval.
> # The expected value on a CPU is (m * cpu + b).
> # The default value on a CPU that did not fire is defval.
> diff --git a/test/unittest/arrays/tst.uregsarray-check.sh b/test/unittest/arrays/tst.uregsarray-check.sh
> index b672666e..99dd6980 100755
> --- a/test/unittest/arrays/tst.uregsarray-check.sh
> +++ b/test/unittest/arrays/tst.uregsarray-check.sh
> @@ -79,7 +79,7 @@ fi
>
> # post processing
>
> -awk '
> +gawk '
> BEGIN { DTrace_PC = DTrace_SP = instructions = local_variable = -1 }
>
> # the first file has DTrace output
> diff --git a/test/unittest/codegen/tst.kernel_read_scalar_regspill.r.p b/test/unittest/codegen/tst.kernel_read_scalar_regspill.r.p
> index 48aa84f7..3c0e9dd2 100755
> --- a/test/unittest/codegen/tst.kernel_read_scalar_regspill.r.p
> +++ b/test/unittest/codegen/tst.kernel_read_scalar_regspill.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/awk -f
> +#!/usr/bin/gawk -f
>
> NR == 1 { val = base = strtonum("0x"$1); }
> NR > 1 { val = strtonum("0x"$1); }
> diff --git a/test/unittest/cpc/tst.args_kernel.sh b/test/unittest/cpc/tst.args_kernel.sh
> index 23b45a7e..9d7bb652 100755
> --- a/test/unittest/cpc/tst.args_kernel.sh
> +++ b/test/unittest/cpc/tst.args_kernel.sh
> @@ -54,7 +54,7 @@ echo "summary of D output (occurrences, arg0, arg1)"
> cat D.out
>
> # check the PCs
> -read ntotal nwarn nerror <<< `awk '
> +read ntotal nwarn nerror <<< `gawk '
> BEGIN { ntotal = nwarn = nerror = 0; }
>
> # file reports 1:occurrences, 2:arg0, 3:arg1
> diff --git a/test/unittest/cpc/tst.args_user.sh b/test/unittest/cpc/tst.args_user.sh
> index f7b13dde..16feddf3 100755
> --- a/test/unittest/cpc/tst.args_user.sh
> +++ b/test/unittest/cpc/tst.args_user.sh
> @@ -58,7 +58,7 @@ echo "summary of D output (occurrences, arg0, arg1)"
> cat D.out
>
> # check the PCs
> -read ntotal narg0 narg1 nwarn nerror <<< `awk '
> +read ntotal narg0 narg1 nwarn nerror <<< `gawk '
> BEGIN { ntotal = narg0 = narg1 = nwarn = nerror = 0; }
>
> # one file has the PCs that are in the inner loop
> diff --git a/test/unittest/pid/tst.offsets.sh b/test/unittest/pid/tst.offsets.sh
> index c0b23fa9..63344698 100755
> --- a/test/unittest/pid/tst.offsets.sh
> +++ b/test/unittest/pid/tst.offsets.sh
> @@ -85,7 +85,7 @@ fi
> # hot and cold instructions in the function.
> #
>
> -objdump -d a.out | awk '
> +objdump -d a.out | gawk '
> BEGIN {
> pc0 = 0; # First PC of loopfunc()
> pcjump = 0; # PC of the jump
> @@ -142,7 +142,7 @@ fi
> # Use files pcs.out and D.out to check results.
> #
>
> -awk '
> +gawk '
> BEGIN {
> # Determine the expected counts for cold and hot instructions
> ncold = '$nouter';
> diff --git a/test/unittest/profile-n/tst.args_kernel.sh b/test/unittest/profile-n/tst.args_kernel.sh
> index f8582fc3..c47f699f 100755
> --- a/test/unittest/profile-n/tst.args_kernel.sh
> +++ b/test/unittest/profile-n/tst.args_kernel.sh
> @@ -49,7 +49,7 @@ echo "summary of D output (occurrences, arg0, arg1)"
> cat D.out
>
> # check the PCs
> -read ntotal nwarn nerror <<< `awk '
> +read ntotal nwarn nerror <<< `gawk '
> BEGIN { ntotal = nwarn = nerror = 0; }
>
> # file reports 1:occurrences, 2:arg0, 3:arg1
> diff --git a/test/unittest/profile-n/tst.args_user.sh b/test/unittest/profile-n/tst.args_user.sh
> index 2e7bee09..1eaadd9e 100755
> --- a/test/unittest/profile-n/tst.args_user.sh
> +++ b/test/unittest/profile-n/tst.args_user.sh
> @@ -53,7 +53,7 @@ echo "summary of D output (occurrences, arg0, arg1)"
> cat D.out
>
> # check the PCs
> -read ntotal narg0 narg1 nwarn nerror <<< `awk '
> +read ntotal narg0 narg1 nwarn nerror <<< `gawk '
> BEGIN { ntotal = narg0 = narg1 = nwarn = nerror = 0; }
>
> # one file has the PCs that are in the inner loop
> --
> 2.46.0
>
next prev parent reply other threads:[~2024-10-12 3:30 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-29 20:15 [PATCH 1/6] test: fix missing line continuation in tst.perf-types.sh Sam James
2024-08-29 20:15 ` [PATCH 2/6] test: fix broken shebang in tst.plddGrab32.sh Sam James
2024-10-12 3:30 ` Kris Van Hees
2024-08-29 20:15 ` [PATCH 3/6] test: awk -> gawk where we use strtonum Sam James
2024-10-12 3:30 ` Kris Van Hees [this message]
2024-08-29 20:15 ` [PATCH 4/6] test: fix 'CPU' typo Sam James
2024-10-12 3:30 ` Kris Van Hees
2024-08-29 20:15 ` [PATCH 5/6] test: libproc: cater to merged-usr systems Sam James
2024-10-12 3:30 ` Kris Van Hees
2024-08-29 20:15 ` [PATCH 6/6] test: awk -> gawk universally Sam James
2024-10-12 3:31 ` Kris Van Hees
2024-10-12 3:29 ` [PATCH 1/6] test: fix missing line continuation in tst.perf-types.sh Kris Van Hees
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZwntTIoFuKtwlLOk@oracle.com \
--to=kris.van.hees@oracle.com \
--cc=dtrace@lists.linux.dev \
--cc=sam@gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.