zephyr: shell: add initial SOF custom commands#9174
Merged
lgirdwood merged 1 commit intothesofproject:mainfrom Jul 4, 2024
Merged
zephyr: shell: add initial SOF custom commands#9174lgirdwood merged 1 commit intothesofproject:mainfrom
lgirdwood merged 1 commit intothesofproject:mainfrom
Conversation
This was referenced May 28, 2024
lgirdwood
approved these changes
Jun 18, 2024
Member
lgirdwood
left a comment
lgirdwood
left a comment
There was a problem hiding this comment.
LGTM, will be a nice tool to help reproduce issues.
One of the benefits and common ways to use the Zephyr shell subsystem is definition of application specific commands. This commit adds the initial SOF custom command, "sof test_inject_sched_gap". This new command allows to inject scheduling gaps into low-latency timer scheduler execution, using the domain_block() SOF interface. Optional argument can be used to specify duration of the block in micro seconds. The intent is to stress test a SOF configuration and test how the system behaves when audio pipeline is briefly starved. Example log of what is observed when test_inkect_sched_gap is run: [ 0.052431] <inf> ll_schedule: zephyr_domain_thread_fn: ll core 0 timer avg 3674, max 4326, overruns 0 [ 0.052831] <inf> ll_schedule: zephyr_domain_thread_fn: ll core 0 timer avg 3673, max 4318, overruns 0 ~$ ~$ sof test_inject_sched_gap 12000 [ 0.052968] <inf> ll_schedule: zephyr_domain_block: Blocking LL scheduler [ 0.052973] <inf> ll_schedule: zephyr_domain_unblock: Unblocking LL scheduler [ 0.052973] <inf> host_comp: host_get_copy_bytes_normal: comp:1 0x10004 no bytes to copy, available samples: 0, free_samples: 384 [ 0.053231] <inf> ll_schedule: zephyr_domain_thread_fn: ll core 0 timer avg 4074, max 437230, overruns 1 Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
kv2019i
force-pushed
the
202405-shell-plus
branch
from
2c039a1 to
85e1d68
Compare
kv2019i
requested review from
abonislawski,
dbaluta,
iuliana-prodan,
lyakh and
marc-hb
as code owners
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Builds on (and contains all commits of) PRs #9133
#9133 adds the basic shell support and this PR adds SOF custom commands to the shell. Both are draft until the Zephyr side PRs are merged.