Skip to content

Tools: Testbench: Add useful script for run and profile#9607

Merged
lgirdwood merged 2 commits intothesofproject:mainfrom
singalsu:add_testbench_scripts
Nov 4, 2024
Merged

Tools: Testbench: Add useful script for run and profile#9607
lgirdwood merged 2 commits intothesofproject:mainfrom
singalsu:add_testbench_scripts

Conversation

@singalsu
Copy link
Collaborator

@singalsu singalsu commented Oct 23, 2024

No description provided.

@singalsu singalsu force-pushed the add_testbench_scripts branch from 41fa2f4 to 88fdcb4 Compare October 29, 2024 13:14
@singalsu singalsu marked this pull request as ready for review October 29, 2024 13:15
Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent stuff @singalsu ! One minor note on documentation for the second patch, please see inline.

The debugging and profiling of IPC4 testench is similar as with IPC3.
Note: Current version of testbench does not have functional quiet mode
"-q" switch to suppress trace. Due all debug traces print
out. Majority of performance is spent in printing.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Due all debug traces print out." is a bit unclear, what do you mean by this? Also "Majority of performance" sounds a bti off, "Majority of cycles" ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, oops, what did I write. Also the -q option was removed as duplicate of -d , need to fix this.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed now!

This patch adds scripts sof-testbench-helper.sh and
sof-testbench-build-profile.sh to ease audio module
developer's frequent tasks.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
The document is updated after changing build default to IPC4
and to show usage of added helper scripts.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
@singalsu singalsu force-pushed the add_testbench_scripts branch from 88fdcb4 to 258f823 Compare November 1, 2024 17:48
@lgirdwood lgirdwood merged commit 200efc3 into thesofproject:main Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants