Skip to content

Commit

Permalink
.github/ipc_fuzzer: use new -3 and -4 fuzz.sh options
Browse files Browse the repository at this point in the history
Stop hardcoding -DCONFIG_IPC_MAJOR_x=y and use the new -3 and -4 fuzz.sh
CLI flags.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
  • Loading branch information
marc-hb committed Aug 27, 2024
1 parent 891c219 commit 5cf08b2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/build_all.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -65,5 +65,6 @@ 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'
6 changes: 3 additions & 3 deletions .github/workflows/ipc_fuzzer.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 5cf08b2

Please sign in to comment.