Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SKIP SOF-TEST] fuzz.sh: add new boards/native_sim_libfuzzer.conf #9389

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Aug 22, 2024

Extract -DCONFIG_* definitions hardcoded inside the script and move them to a new .conf file where they belong.

This is a first, baby-step towards addressing the more general lack of .config control described in #9386

Extract -DCONFIG_* definitions hardcoded inside the script and move them
to a new .conf file where they belong.

This is a first, baby-step towards addressing the more general lack of
.config control described in thesofproject#9386

Signed-off-by: Marc Herbert <[email protected]>
@marc-hb
Copy link
Collaborator Author

marc-hb commented Aug 22, 2024

The failures are unrelated, caused by #9365 which was merged broken. This PR affects fuzzing only, all other errors are unrelated.

-DCONFIG_ASAN=y
)
# https://docs.zephyrproject.org/latest/build/kconfig/setting.html#initial-conf
local conf_files_list='prj.conf;boards/native_sim_libfuzzer.conf'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we make it so we can append to this list. I.e. the ASAN config would be split so we can use the base config with other sanitzer configs.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes that's the idea. In some future PR(s) anyone can just go and (for instance) do:

if $something; then
   conf_files_list+=';debug.conf'

@lgirdwood lgirdwood merged commit 577c9ae into thesofproject:main Aug 23, 2024
39 of 43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants