You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[FEATURE] Dump IPC state machine status on IPC timed out #846
IPC timeouts are often difficult to debug as we have two state machines running, one within the driver and the other within the FW.
This feature is to add useful debug output to the kernel when it detects and IPC timeout. i.e.
<< dmesg after current IPC timeout is printed >>
error: IPC host state: HOST_TO_DSP_CMD, DSP_TO_HOST_NOTIFICATION
error: IPC host mask: DSP_TO_HOST_NOTIFICATION
error: IPC host raw: status 0x00000003 mask 0x0000001
error: IPC dsp state: DSP_TO_HOST_NOTIFICATION
error: IPC dsp mask: DSP_TO_HOST_NOTIFICATION
error: IPC dsp raw: status 0x00000001 mask 0x0000001
The intention here is to show the IPC state machines for both DSP and host and show the pending operations in a high level manner alongside the raw status and mask.
changed the title [-][FEATURE] Dump IPC state machine status on IPC timeouts[/-][+][FEATURE] Dump IPC state machine status on IPC timed out[/+]on May 7, 2019
@ranj063
I would like to do that with IPC2.0 feature. To define a state machine for the IPC mechanism, otherwise I am not sure if we can get some parser status with the bits we have.
Activity
lgirdwood commentedon Apr 18, 2019
@ranj063 fyi, I think a feature like this would be useful for your debug today.
xiulipan commentedon Apr 23, 2019
@lgirdwood
ACK. will try to make a PR once I finished.
xiulipan commentedon Apr 26, 2019
@lgirdwood
A RFC is here: #872
[-][FEATURE] Dump IPC state machine status on IPC timeouts[/-][+][FEATURE] Dump IPC state machine status on IPC timed out[/+]wenqingfu commentedon May 8, 2019
should it be closed then, since #872 is merged now? @xiulipan
xiulipan commentedon May 8, 2019
@wenqingfu
YES, I am glad that the RFC is just got accepted.
Fixed by #872
ranj063 commentedon May 8, 2019
@xiulipan dont we want to add the parser to show the status from the raw ipc regsiter values?
xiulipan commentedon May 8, 2019
@ranj063
I would like to do that with IPC2.0 feature. To define a state machine for the IPC mechanism, otherwise I am not sure if we can get some parser status with the bits we have.
ranj063 commentedon May 8, 2019
@xiulipan fair enough. Lets modify the issue requirement and keep it open. Else it will get swept under the carpet
Merge tag 'drm-intel-fixes-2020-01-09-1' of git://anongit.freedesktop…
Merge tag 'drm-fixes-2020-01-10' of git://anongit.freedesktop.org/drm…
plbossart commentedon Jun 10, 2021
@ranj063 can you clarify what remains to be done, I can't follow this 2-yr old thread. thanks!
lgirdwood commentedon Jun 11, 2021
It's done and upstream.