Skip to content

Conversation

@fredoh9
Copy link
Contributor

@fredoh9 fredoh9 commented Jun 22, 2020

Make sure IPC flood test results are printed. If the test summary is not
available, it is failed case. This is a fix of false positive case.

[ 96.248808] sof-audio-pci 0000:00:1f.3: error: debugfs write failed to idle -16

Signed-off-by: Fred Oh fred.oh@linux.intel.com

Related issue is thesofproject/linux#2204
I will debug why ipc flood test is not performed and failed to transit to idle.

Make sure IPC flood test results are printed. If the test summary is not
available, it is failed case. This is a fix of false positive case.

[   96.248808] sof-audio-pci 0000:00:1f.3: error: debugfs write failed to idle -16

Signed-off-by: Fred Oh <fred.oh@linux.intel.com>
@fredoh9 fredoh9 requested review from Bin-QA and marc-hb as code owners June 22, 2020 23:11
@fredoh9 fredoh9 changed the title check-ipc-flood.sh: check ipc flood test logs from dmesg [DON't MERGE] check-ipc-flood.sh: check ipc flood test logs from dmesg Jun 22, 2020
@fredoh9
Copy link
Contributor Author

fredoh9 commented Jun 22, 2020

Checking logs may not be good idea. It may not come if debug level is not verbose. I will debug why dmesg 'error' didn't lead to test failure.

[   70.091602] sof-audio-pci 0000:00:1f.3: IPC Flood count: 10000, Avg response time: 50780ns
[   70.091603] sof-audio-pci 0000:00:1f.3: Max response time: 452093ns
[   70.091603] sof-audio-pci 0000:00:1f.3: Min response time: 32028ns

@fredoh9
Copy link
Contributor Author

fredoh9 commented Jun 22, 2020

Looks this error is known false alarm, added to ignoring messages.

https://github.com/thesofproject/sof-test/blob/master/tools/sof-kernel-log-check.sh#L6

ignore_str="error: debugfs write failed to idle -16|error: status|iteration [01]"

@fredoh9 fredoh9 closed this Jun 23, 2020
@marc-hb
Copy link
Collaborator

marc-hb commented Jun 23, 2020

ignore_str="error: debugfs write failed to idle -16|error: status|iteration [01]"

Ignoring errors is very dangerous and should be carefully reviewed and extensively documented. At the very least this code should be changed to something like:

# because blabla, see bug 1234
ignore_str="$ignore_str"'|error: debugfs write failed to idle -16'
# because blabla, see bug 5678
ignore_str="$ignore_str"'|error: status'

@xiulipan , @aiChaoSONG and @Bin-QA please help.

EDIT: ignore_str partially documented in #268

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