Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tools: Testbench: Switch testbench to IPC4 mode #9590

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

singalsu
Copy link
Collaborator

The library_defconfig is changed to CONFIG_IPC_MAJOR_4=y.

To prepare release of testbench with SOF the executable name is changed to sof-testbench4.

The script comp_run.sh for process_test.m and other test scripts is changed to execute the IPC4 testbench version with use of other topology v2 topologies sof-hda-benchmark-.tplg.

the script host-testbench.sh is changed to run tests for component names in IPC4 test topologies.

TODO: Fix valgrind issue in testbench, fix audio quality issue in config blob for multiband-DRC.

Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

@singalsu I assume ipc3 testbench will be called sof-testbench3 ?

@singalsu
Copy link
Collaborator Author

@singalsu I assume ipc3 testbench will be called sof-testbench3 ?

I need to figure out cmake stuff to build it. This patch hard-codes the executable name to sof-testbench4. I'd like to make it dependent on library_defconfig choice. Or better, somehow with scripts/rebuild-testbench.sh build for both ipc versions. But I don't yet now how to achieve it with cmake.

@singalsu singalsu force-pushed the testbench_use_ipc4_mode branch 5 times, most recently from 961c77c to b514e09 Compare October 21, 2024 14:14
The library_defconfig is changed to CONFIG_IPC_MAJOR_4=y.

To prepare release of testbench with SOF the executable name
is changed to sof-testbench4 for IPC4. The name is selected
based on choice in library_defconfig.

The script comp_run.sh for process_test.m and other test scripts
is changed to execute the IPC4 testbench version with use of
other topology v2 topologies sof-hda-benchmark-<comp><bits>.tplg.

the script host-testbench.sh is changed to run tests for
component names in IPC4 test topologies.

The github workflow is changed due to change in used topologies
for the test. The scripts/build-tools.sh need to be executed
with "-Y" option to build topology2 files. Also need to use
alsatplg from docker. The octave-io package is not needed for
this step.

Signed-off-by: Seppo Ingalsuo <[email protected]>
@singalsu
Copy link
Collaborator Author

Seems the tplg2 build for CI testbench step is now resolved. But this PR depends on #9599 to pass the multiband-DRC test.

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.

2 participants