Skip to content

drivers/syslog: reuse rmutex_t for the recursive check#6435

Merged
pkarashchenko merged 1 commit into
apache:masterfrom
xiaoxiang781216:mutex
Jun 16, 2022
Merged

drivers/syslog: reuse rmutex_t for the recursive check#6435
pkarashchenko merged 1 commit into
apache:masterfrom
xiaoxiang781216:mutex

Conversation

@xiaoxiang781216

@xiaoxiang781216 xiaoxiang781216 commented Jun 14, 2022

Copy link
Copy Markdown
Contributor

Summary

Bring back the change from PR #6320 which is revert by #6414 and fix the issue found by @pkarashchenko

Impact

Testing

@pkarashchenko could you try this change with your config?

Comment thread include/nuttx/mutex.h
@xiaoxiang781216 xiaoxiang781216 force-pushed the mutex branch 2 times, most recently from caf33a1 to 81e1461 Compare June 15, 2022 03:51
Comment thread drivers/syslog/syslog_device.c
Comment thread drivers/syslog/syslog_device.c
Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com>

@davids5 davids5 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@pkarashchenko pkarashchenko merged commit 22548d7 into apache:master Jun 16, 2022
@xiaoxiang781216 xiaoxiang781216 deleted the mutex branch June 16, 2022 16:05
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.

4 participants