Skip to content

[FEATURE] Checkers with limits of DSP load tracker #5518

Closed

Description

@johnylin76

Is your feature request related to a problem? Please describe.
We need a fair metrics to evaluate the loading on DSP cores during audio playback/capture/hotwording. To prevent from overloading, checkers should be added with configurable threshold value in CI or maybe in release xtos builds.

Describe the solution you'd like

  • DSP load tracker on upstream for xtos mode
  • Have a configurable (e.g. Kconfig) threshold value to report error or forcedly drop the task with the lowest priority (and etc.) when the measured DSP load is excessive.

Describe alternatives you've considered
There is the performance counter which also measures cycle count. However it only reports peak which can be blustered by one burst in some situations.

Additional context
DSP load tracker has been adopted in SOF Zephyr builds and proven to be very useful for debugging.

Activity

  1. lgirdwood commented on Mar 11, 2022

    @lgirdwood
    Member

    @jsarha can you link your PR here. Thanks

  2. johnylin76 commented on Mar 25, 2022

    @johnylin76
    ContributorAuthor

    #5407 is merged to upstream

  3. added this to the v2.2 milestone on Apr 6, 2022
  4. lgirdwood commented on Apr 6, 2022

    @lgirdwood
    Member

    Adding for v2.2 in case we want to improve.

  5. modified the milestones: v2.2, v2.6 on Feb 17, 2023
  6. lgirdwood commented on May 11, 2023

    @lgirdwood
    Member

    @kv2019i I think we have load reporting now, and CPC checking is pending merge to main. So will move to v2.7 for full completion.

  7. modified the milestones: v2.6, v2.7 on May 11, 2023
  8. kv2019i commented on May 12, 2023

    @kv2019i
    Collaborator

    I believe this can be closed. This was to backport (to XTOS SOF!) the load tracker we implemented for original Zephyr SOF code. This was done in #5407 , so I believe we are good to go. @johnylin76 @cujomalainey can you ack? We can continue perf observability in #5521 (still open).

  9. cujomalainey commented on May 15, 2023

    @cujomalainey
    Contributor

    @kv2019i agreed, and it looks like zephyr is much richer in terms of support than xtos for this

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

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions