Skip to content

Commit 482698c

Browse files
captain5050acmel
authored andcommitted
perf test: Fix non-bash issue with stat bpf counters
$(( .. )) is a bash feature but the test's interpreter is !/bin/sh, switch the code to use expr. Signed-off-by: Ian Rogers <irogers@google.com> Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com> Cc: Jiri Olsa <jolsa@redhat.com> Cc: Mark Rutland <mark.rutland@arm.com> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Song Liu <songliubraving@fb.com> Cc: bpf@vger.kernel.org Link: http://lore.kernel.org/lkml/20210617184216.2075588-1-irogers@google.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
1 parent c087e94 commit 482698c

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

tools/perf/tests/shell/stat_bpf_counters.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ compare_number()
1111
second_num=$2
1212

1313
# upper bound is first_num * 110%
14-
upper=$(( $first_num + $first_num / 10 ))
14+
upper=$(expr $first_num + $first_num / 10 )
1515
# lower bound is first_num * 90%
16-
lower=$(( $first_num - $first_num / 10 ))
16+
lower=$(expr $first_num - $first_num / 10 )
1717

1818
if [ $second_num -gt $upper ] || [ $second_num -lt $lower ]; then
1919
echo "The difference between $first_num and $second_num are greater than 10%."

0 commit comments

Comments
 (0)