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

multiple-pipeline: add debugging when pkill fails #1213

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

lyakh
Copy link
Contributor

@lyakh lyakh commented Jun 26, 2024

Sometimes pkill seems to fail terminating aplay and arecord processes. Add a system request to check state of those processes to investigate why they couldn't be killed.

Sometimes pkill seems to fail terminating aplay and arecord
processes. Add a system request to check state of those processes to
investigate why they couldn't be killed.

Signed-off-by: Guennadi Liakhovetski <[email protected]>
@@ -206,6 +206,11 @@ do
pkill -9 aplay || true
sleep 1 # try not to pollute the next iteration

if pgrep arecord || pgrep aplay; then
printf '%c' 't' | sudo tee /proc/sysrq-trigger > /dev/null
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
printf '%c' 't' | sudo tee /proc/sysrq-trigger > /dev/null
#
printf 't' | sudo tee /proc/sysrq-trigger > /dev/null

@lyakh
Copy link
Contributor Author

lyakh commented Jul 2, 2024

CI: all the failures in https://sof-ci.01.org/softestpr/PR1213/build576/devicetest/index.html, https://sof-ci.01.org/softestpr/PR1213/build573/devicetest/index.html and https://sof-ci.01.org/softestpr/PR1213/build575/devicetest/index.html are unrelated since this PR only affects the multiple-pipeline test family and the failures are in alsabat, pause-release and suspend-resume tests.

@marc-hb marc-hb merged commit 70c9d34 into thesofproject:main Jul 2, 2024
4 of 7 checks passed
@lyakh lyakh deleted the trace branch July 3, 2024 06:39
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