Skip to content

[FEATURE] Dump IPC state machine status on IPC timed out #846

Closed

Description

@lgirdwood

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.

Activity

  1. lgirdwood commented on Apr 18, 2019

    @lgirdwood
    MemberAuthor

    @ranj063 fyi, I think a feature like this would be useful for your debug today.

  2. xiulipan commented on Apr 23, 2019

    @xiulipan

    @lgirdwood
    ACK. will try to make a PR once I finished.

  3. self-assigned this
    on Apr 23, 2019
  4. xiulipan commented on Apr 26, 2019

    @xiulipan

    @lgirdwood
    A RFC is here: #872

  5. 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
  6. wenqingfu commented on May 8, 2019

    @wenqingfu

    @lgirdwood
    A RFC is here: #872

    should it be closed then, since #872 is merged now? @xiulipan

  7. xiulipan commented on May 8, 2019

    @xiulipan

    @wenqingfu
    YES, I am glad that the RFC is just got accepted.

    Fixed by #872

  8. ranj063 commented on May 8, 2019

    @ranj063
    Collaborator

    @xiulipan dont we want to add the parser to show the status from the raw ipc regsiter values?

  9. xiulipan commented on May 8, 2019

    @xiulipan

    @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.

  10. ranj063 commented on May 8, 2019

    @ranj063
    Collaborator

    @xiulipan fair enough. Lets modify the issue requirement and keep it open. Else it will get swept under the carpet

  11. plbossart commented on Jun 10, 2021

    @plbossart
    Member

    @ranj063 can you clarify what remains to be done, I can't follow this 2-yr old thread. thanks!

  12. lgirdwood commented on Jun 11, 2021

    @lgirdwood
    MemberAuthor

    It's done and upstream.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions