.github/ipc_fuzzer: use new -3 and -4 fuzz.sh options

Stop hardcoding -DCONFIG_IPC_MAJOR_x=y and use the new -3 and -4 fuzz.sh CLI flags. Signed-off-by: Marc Herbert <[email protected]>
marc-hb · Aug 27, 2024 · 7443d6e · 7443d6e
1 parent ed7ad60
commit 7443d6e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/.github/workflows/build_all.yml b/.github/workflows/build_all.yml
@@ -15,7 +15,7 @@ jobs:
     strategy:
       matrix:
         # Keep these names short due to questionable Github UI choices
-        IPC: [ipc3, ipc4]
+        IPC: [3, 4]
 
     steps:
       - name: add i386 arch
@@ -65,5 +65,5 @@ jobs:
           cd workspace
           clang --verbose
           set -x
-          sof/scripts/fuzz.sh -b -- -DEXTRA_CFLAGS='-Werror' -DEXTRA_CXXFLAGS='-Werror' \
-              -DEXTRA_CONF_FILE='stub_build_all_${{ matrix.IPC }}.conf'
+          sof/scripts/fuzz.sh -b -'${{ matrix.IPC }}' -- -DEXTRA_CFLAGS='-Werror' -DEXTRA_CXXFLAGS='-Werror' \
+              -DEXTRA_CONF_FILE='stub_build_all_ipc${{ matrix.IPC }}.conf'
diff --git a/.github/workflows/ipc_fuzzer.yml b/.github/workflows/ipc_fuzzer.yml
@@ -78,13 +78,13 @@ jobs:
           clang --verbose
           set -x
           case '${{ matrix.IPC }}' in
-            IPC3) cmake_arg='-DCONFIG_IPC_MAJOR_3=y' ;;
-            IPC4) cmake_arg='-DCONFIG_IPC_MAJOR_4=y' ;;
+            IPC3) _ipc='-3' ;;
+            IPC4) _ipc='-4' ;;
           esac
           duration="${{inputs.fuzzing_duration_s}}"
           duration="${duration:-301}"  # pull_request has not 'inputs.' :-(
           # Note libFuzzer makes a difference between -jobs and -workers (capped at nproc/2)
-          sof/scripts/fuzz.sh -o fuzz-stdout.txt -t "$duration" -j"$(nproc)" -- "$cmake_arg"
+          sof/scripts/fuzz.sh "$_ipc" -o fuzz-stdout.txt -t "$duration" -j"$(nproc)"
 
       - name: Upload stdout
         uses: actions/upload-artifact@v4