Skip to content

Conversation

@XiaoyunWu6666
Copy link
Contributor

@XiaoyunWu6666 XiaoyunWu6666 commented Mar 22, 2022

Currently, sof-logger is not supported when running IPC4 mode.

Thus, disable SOF logs collection globally when DUT runs IPC4 mode (this can be telled by system file /sys/module/snd_sof_pci/parameters/ipc_type).

This should be removed after sof-logger support for IPC4 has been provided.

Signed-off-by: Xiaoyun Wu(Iris) xiaoyun.wu@intel.com

Currently, sof-logger is not supported when running IPC4 mode.

Thus, disable SOF logs collection globally when DUT runs IPC4 mode.

This should be removed after sof-logger support for IPC4 has been
provided.

Signed-off-by: Xiaoyun Wu(Iris) <xiaoyun.wu@intel.com>
@XiaoyunWu6666 XiaoyunWu6666 requested a review from a team as a code owner March 22, 2022 09:13
@XiaoyunWu6666
Copy link
Contributor Author

this can replace PR https://github.com/intel-innersource/drivers.audio.ci.sof-sh/pull/53 and https://github.com/intel-innersource/drivers.audio.ci.sof-framework/pull/202.
We make sof-test disable sof-logger directly when it detects ipc4 setting in system file , instead of passing the SOF_LOGGING variable to test framework or setting it as the environment variable

# 1: DUT runs IPC4 mode
ipc_type=$(cat $ipcFile)
if [ $ipc_type -eq 1 ]; then
dlogi "/sys/module/snd_sof_pci/parameters/ipc_type is ${ipc_type}, DUT runs IPC4 mode"
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, by "runs" does it mean "will run in?" Or does it mean it ran in IPC4 mode? Current tense is ambiguous here and below.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hi @greg-intel I modified the tense in #884 .
@plbossart Sorry, I mistook the repo . Already prepare a new PR from my own forked repo #884 , thx~

@plbossart
Copy link
Member

please submit PRs from your forked repository, not from a branch on the sof-test GIT tree.

@plbossart plbossart closed this Mar 22, 2022
@keqiaozhang keqiaozhang deleted the sof-logger-IPC4 branch May 29, 2023 01:42
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.

4 participants