-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
bluetooth: honor log panic mode in monitor #76589
Conversation
cb3c9fb
to
175d4a7
Compare
175d4a7
to
2d3581c
Compare
2d3581c
to
140653a
Compare
subsys/bluetooth/host/monitor.c
Outdated
#if defined(CONFIG_BT_DEBUG_MONITOR_RTT) || !defined(CONFIG_LOG_MODE_MINIMAL) | ||
static bool panic_mode; | ||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Given the use of it is only for CONFIG_BT_DEBUG_MONITOR_RTT
then I suggest to move the declaration of the variable to the #if defined(CONFIG_BT_DEBUG_MONITOR_RTT)
part on line 100.
140653a
to
f9c5031
Compare
Once panic mode is entered, the log write functions are called from an ISR, so must not try to acquire locks. Signed-off-by: Armin Brauns <[email protected]>
f9c5031
to
023bfdc
Compare
This pull request 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 pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
Once panic mode is entered, the log write functions are called from an ISR, so must not try to acquire locks.