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] Checkers with limits of DSP load tracker #5518
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.
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).
Activity
lgirdwood commentedon Mar 11, 2022
@jsarha can you link your PR here. Thanks
johnylin76 commentedon Mar 25, 2022
#5407 is merged to upstream
lgirdwood commentedon Apr 6, 2022
Adding for v2.2 in case we want to improve.
lgirdwood commentedon May 11, 2023
@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.
kv2019i commentedon May 12, 2023
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).
cujomalainey commentedon May 15, 2023
@kv2019i agreed, and it looks like zephyr is much richer in terms of support than xtos for this