Repository navigation
[FEATURE] implement sof-test case that override nocodec with nocodec-bt and runs playback tests #1013
Description
Is your feature request related to a problem? Please describe.
Implement a test-case where an alternative nocodec topology (in this case for bt-offload) is loaded as part of the test case execution instead of nocodec topology. This allows to test features that are not described in the default nocodec topology and reuse same test content that is used for other topologies.
Describe the solution you'd like
Sof-test case that uses kernel module parameter to load an altenrative topology file, and an existing test case .
Describe alternatives you've considered
Merging BT to existing nocodec topology.
BT requires configuration of alternative sampling rate and formats that are in conflict with typical requirements for nocodec testing.
Additional context
This approach could be used to other features (like smart-amp, wake-on-voice, muxes) that do not depend on specific audio peripherals, but are too complex to integrate to nocodec topology.
cc:
Activity
marc-hb commentedon Mar 22, 2023
There are some focused exceptions but kernel modules are typically not reloaded from one test to the next in a test suite. So I don't think such a default topology override should belong to any particular test case. I rather think we should add the ability in the existing
sof-test/tools/kmod/sof_insert.shscript to temporarily override kernel parameters and override the loaded topology. For instance it could/should just look at the existing TPLG variable as the input.Then you run any test plan of your choice.
plbossart commentedon Mar 22, 2023
sof-insert.sh does not necessarily reload all intermediate modules, only top-level ones such as codecs and PCI.
We couldn't e.g. add parameters to snd_sof_intel_hda_common.
Somehow we have an interesting exception to the rule (maybe due to history), we do have "insert_module snd_sof_pci # ipc_type=1 # sof_pci_debug=1 ...", so that would solve the problem of loading an alternate topology is desired.
marc-hb commentedon Mar 22, 2023
We could dynamically create a one-line file in
/etc/modprobe.d/withsnd_sof_intel_hda_commonand other options , insert top modules and then delete it that file. That's what I had in mind in the previous comment.marc-hb commentedon May 31, 2023
Summarizing a discussion today: this feature is about the convenience and flexibility to override the default topology.
nocodec-*just happen to be these topologies we want to override but the test framework should not care about which particular topologies are relevant.