Skip to content

Commit

Permalink
.github/ipc_fuzzer: use new -i 3 and -i 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 authored and lgirdwood committed Sep 2, 2024
1 parent 2a23f3b commit ff57b79
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 9 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 -i '${{ matrix.IPC }}' -- \
-DEXTRA_CFLAGS='-Werror' -DEXTRA_CXXFLAGS='-Werror' \
-DEXTRA_CONF_FILE='stub_build_all_ipc${{ matrix.IPC }}.conf'
8 changes: 2 additions & 6 deletions .github/workflows/ipc_fuzzer.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
fail-fast: false
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 @@ -77,14 +77,10 @@ jobs:
cd workspace
clang --verbose
set -x
case '${{ matrix.IPC }}' in
IPC3) cmake_arg='-DCONFIG_IPC_MAJOR_3=y' ;;
IPC4) cmake_arg='-DCONFIG_IPC_MAJOR_4=y' ;;
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 -i '${{ matrix.IPC }}' -o fuzz-stdout.txt -t "$duration" -j"$(nproc)"
- name: Upload stdout
uses: actions/upload-artifact@v4
Expand Down

0 comments on commit ff57b79

Please sign in to comment.