check-ipc-flood.sh: stop asking IPC4 users to use CONFIG_DEBUG

Add is_ipc4() call and a new SKIP message to stop telling Zephyr users to build a "debug firmware" because it won't change anything for them: IPC4 has no flood command right now, see #1066 Also clarify the XTOS skip message + other cosmetic improvements. Signed-off-by: Marc Herbert <[email protected]>
thesofproject · Aug 14, 2024 · 65d7174 · 65d7174
1 parent 000633a
commit 65d7174
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/test-case/check-ipc-flood.sh b/test-case/check-ipc-flood.sh
@@ -36,16 +36,21 @@ loop_cnt=${OPT_VAL['l']}
 
 start_test
 
+if is_ipc4; then
+    skip_test 'IPC4 detected: see sof-test#1066'
+fi
+
+# See SOF_IPC_INFO_BUILD in linux/sound/soc/sof/ipc3.c:sof_ipc3_validate_fw_version()
 sof-kernel-dump.sh | grep sof-audio | grep -q "Firmware debug" ||
-     skip_test "need debug version firmware"
+     skip_test "CONFIG_DEBUG firmware needed for SOF_IPC_TEST_IPC_FLOOD cmd"
 
 func_lib_check_sudo
 
-dlogi "Check sof debug fs environment"
-if sudo file $ipc_flood_dfs | grep 'No such file'; then
+dlogi "Check $ipc_flood_dfs"
+sudo test -e $ipc_flood_dfs ||
   skip_test "${BASH_SOURCE[0]} need $ipc_flood_dfs to run the test case"
-fi
-dlogi "Checking ipc flood test!"
+
+dlogi "Running ipc flood test!"
 
 for i in $(seq 1 $loop_cnt)
 do