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

samples: shared_mem: fix exception printouts #22112

Merged
merged 1 commit into from
Jan 23, 2020

Conversation

andrewboie
Copy link
Contributor

Enable CONFIG_TEST so that we get the necessary defines for
console output when a fatal error happens, as well as assertion
checking.

Remove an unnecessary self-abort in main(), this causes an
assert to fail. Letting main() return does the same thing, more
gracefully.

Signed-off-by: Andrew Boie andrew.p.boie@intel.com

Enable CONFIG_TEST so that we get the necessary defines for
console output when a fatal error happens, as well as assertion
checking.

Remove an unnecessary self-abort in main(), this causes an
assert to fail. Letting main() return does the same thing, more
gracefully.

Signed-off-by: Andrew Boie <andrew.p.boie@intel.com>
@andrewboie
Copy link
Contributor Author

This help debugging #21317 , we weren't seeing the fatal errors that occurred. Root cause appears to be a single thread object scheduled on two cpus simultaneously.

@zephyrbot zephyrbot added the area: Samples Samples label Jan 22, 2020
@nashif nashif merged commit 8eecb37 into zephyrproject-rtos:master Jan 23, 2020
@andrewboie andrewboie deleted the shared-mem-fix branch September 24, 2020 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants