multiple-pipeline: fix set -e compatibility and add it

Notably remove top-level "expr" because "expr 0" is surprisingly a
failure and expr is antiquited anyway (SC2003)

Also add a dlogi() before killing so no one mistakes the deaths of aplay
and arecord for failures.

Progress towards #312.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>

Author: marc-hb

test-case/multiple-pipeline-capture.sh

@@ -22,6 +22,8 @@
 ##    all pipelines are alive and without kernel error
 ##
 
+set -e
+
 source $(dirname ${BASH_SOURCE[0]})/../case-lib/lib.sh
 
 OPT_OPT_lst['t']='tplg'     OPT_DESC_lst['t']='tplg file, default value is env TPLG: $TPLG'
@@ -87,8 +89,8 @@ func_run_pipeline_with_type()
         dlogc "${APP_LST[$1]} -D $dev -c $channel -r $rate -f $fmt ${DEV_LST[$1]} -q"
         "${APP_LST[$1]}" -D $dev -c $channel -r $rate -f $fmt "${DEV_LST[$1]}" -q &
 
-        tmp_count=$(expr $tmp_count - 1 )
-        [[ $tmp_count -le 0 ]] && return
+        : $((tmp_count--))
+        if [ "$tmp_count" -le 0 ]; then return 0; fi
     done
 }
 
@@ -116,9 +118,9 @@ do
     # check all refer capture pipeline status
     # 1. check process count:
     pcount=$(pidof arecord|wc -w)
-    tmp_count=$(expr $tmp_count + $pcount)
+    tmp_count=$((tmp_count + pcount))
     pcount=$(pidof aplay|wc -w)
-    tmp_count=$(expr $tmp_count + $pcount)
+    tmp_count=$((tmp_count + pcount))
     [[ $tmp_count -ne $max_count ]] && func_error_exit "Target pipeline count: $max_count, current process count: $tmp_count"
 
     # 2. check arecord process status
@@ -134,9 +136,9 @@ do
     # 3. check process count again:
     tmp_count=0
     pcount=$(pidof arecord|wc -w)
-    tmp_count=$(expr $tmp_count + $pcount)
+    tmp_count=$((tmp_count + pcount))
     pcount=$(pidof aplay|wc -w)
-    tmp_count=$(expr $tmp_count + $pcount)
+    tmp_count=$((tmp_count + pcount))
     [[ $tmp_count -ne $max_count ]] && func_error_exit "Target pipeline count: $max_count, current process count: $tmp_count"
 
     # 4. check arecord process status
@@ -147,7 +149,7 @@ do
     [[ $? -eq 1 ]] && func_error_exit "Catch the abnormal process status of aplay"
 
 
-    # kill all arecord
+    dlogc 'pkill -9 aplay arecord'
     pkill -9 arecord
     pkill -9 aplay