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

logging: Sync timestamp between cores #23021

Closed
koffes opened this issue Feb 21, 2020 · 3 comments · Fixed by #38495
Closed

logging: Sync timestamp between cores #23021

koffes opened this issue Feb 21, 2020 · 3 comments · Fixed by #38495
Labels
area: Logging area: Multicore Enhancement Changes/Updates/Additions to existing features

Comments

@koffes
Copy link
Collaborator

koffes commented Feb 21, 2020

The logging system has a useful feature where a timestamp is prepended each log message.

This works well on single core MCU's, but in some situations you want to investigate multi-core applications. E.g. both cores crash, and you need to figure ut what took place first.
Suggest adding a config of type CONFIG_LOG_TIMESTAMP_SYNC. This will ensure that e.g. a LOG_DBG which is called on each core at the same point in time gets the same timestamp.

I realise that this is may well be far from trivial, as well as platform specific.
@nordic-krch

@koffes koffes added the Feature Request A request for a new feature label Feb 21, 2020
@carlescufi
Copy link
Member

This should probably be generalized to synchronization between cores of several data items, including timestamps.

@koffes
Copy link
Collaborator Author

koffes commented Aug 17, 2020

@nordic-krch: Is this something that is feasible? We often come across issues where figuring out which log message comes first is essential.

@nordic-krch
Copy link
Contributor

@koffes I've prepared PR which adds mechanism for synchronization. It's nrf53 specific. See #38495

@nashif nashif added Enhancement Changes/Updates/Additions to existing features area: Multicore and removed Feature Request A request for a new feature labels Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Logging area: Multicore Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants