-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Logging (deferred v2) with a lot of output causes MPU fault #46986
Comments
@nordic-krch, can this issue be closed? |
This issue is not resolved |
I tried to reproduced it by logging from timer interrupt but it does not fail: e253a49 Any suggestion how can test be improved to trigger the issue. |
I no longer have the hardware or firmware which was used to reproduce this issue, I'll have a look around what breakout boards I have and see if I can create a new application which demonstrates the issue |
Hey I'll bump this to low priority since it's been here for a couple months, don't have a case for reproducing it and there's no other report for this. Happy to bump ti back up if that changes. |
This issue has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this issue will automatically be closed in 14 days. Note, that you can always re-open a closed issue at any time. |
Describe the bug
A device with an accelerometer which outputs a message on the sensor being triggered can be shaken vigorously to generate a lot of messages, upon doing so, it can be observed that the logging thread faults.
To Reproduce
Have not attempted to reproduce with a sample application because of the requirement of a motion sensor on the unit we are testing, but essentially the application outputs a small message using LOG_ERR() when the sensor trigger callback is used, a 1000ms logging sleep delay is used with 4096 byte logging stack size and 4096 byte logging buffer size, performance over size option is also selected. The drop oldest messages on full option is selected too.
Expected behavior
Logging to work, and if there is not enough space, to drop older messages.
Impact
Showstopper, if someone can DoS a device with this tactic, it isn't a viable device
Logs and console output
Where
0x0002964c
iszephyr/include/zephyr/sys/atomic_builtin.h:88
which is theatomic_add
functionEnvironment (please complete the following information):
The text was updated successfully, but these errors were encountered: