Skip to content

ASoC: SOF: ipc4-mtrace: process pending logs upon FW crash - #4186

Merged
plbossart merged 1 commit into
thesofproject:topic/sof-devfrom
kv2019i:202302-mtrace-panic-flush
Feb 10, 2023
Merged

plbossart merged 1 commit into
thesofproject:topic/sof-devfrom
kv2019i:202302-mtrace-panic-flush

Conversation

@kv2019i

@kv2019i kv2019i commented Feb 9, 2023

Copy link
Copy Markdown
Collaborator

If the DSP firmware has crashed, some log messages may be pending in the shared memory, but DSP was not able to send a IPC notification anymore. Check the buffer status for all mtrace slots and ensure any pending log messages are processed before powering down the DSP.

Signed-off-by: Kai Vehmanen kai.vehmanen@linux.intel.com

@kv2019i

kv2019i commented Feb 10, 2023

Copy link
Copy Markdown
Collaborator Author

Verified that with this PR, we can capture the DSP crash logs via mtrace and make DSP crashes visible in CI results that are published here on github. I used thesofproject/sof#7071 as a test PR to trigger a crash.

Comment thread sound/soc/sof/ipc4-mtrace.c Resolved
If the DSP firmware has crashed, some log messages may be pending in the
mtrace buffer, but not consumed by the driver as no IPC notification has
been sent by the firmware. Check the buffer status for all mtrace slots
and ensure any pending log messages are processed before DSP is possibly
powered down and the log buffer contents is lost.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
@kv2019i
kv2019i force-pushed the 202302-mtrace-panic-flush branch from d581aba to 4f18d87 Compare February 10, 2023 12:35

@ujfalusi ujfalusi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@kv2019i, great that we can get the trace in it's entirety!

@plbossart
plbossart merged commit 63602b7 into thesofproject:topic/sof-dev Feb 10, 2023
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