Skip to content

testcases: Reorder remoteproc tests to fix AudioRecord dependency#23

Open
qcom-anilyada wants to merge 1 commit intoqualcomm-linux:masterfrom
qcom-anilyada:test/shuffleAudioRecord
Open

testcases: Reorder remoteproc tests to fix AudioRecord dependency#23
qcom-anilyada wants to merge 1 commit intoqualcomm-linux:masterfrom
qcom-anilyada:test/shuffleAudioRecord

Conversation

@qcom-anilyada
Copy link

Move cdsp_remoteproc and adsp_remoteproc tests to run after AudioRecord test to prevent service dependency conflicts. The remoteproc tests hold services that AudioRecord waits for, causing AudioRecord to hang on iq-9075-evk when remoteproc runs first.

Remove AudioRecord from iq-9075-evk exclusion list as this reordering resolves the test failure, allowing AudioRecord to pass successfully.

Signed-off-by: Anil Yadav anilyada@qti.qualcomm.com

Move cdsp_remoteproc and adsp_remoteproc tests to run after AudioRecord
test to prevent service dependency conflicts. The remoteproc tests hold
services that AudioRecord waits for, causing AudioRecord to hang on
iq-9075-evk when remoteproc runs first.

Remove AudioRecord from iq-9075-evk exclusion list as this reordering
resolves the test failure, allowing AudioRecord to pass successfully.

Signed-off-by: Anil Yadav <anilyada@qti.qualcomm.com>
@qcom-anilyada
Copy link
Author

Failure before the shuffle on two different targets:

PASSED job after changing the order: https://lava.infra.foundries.io/scheduler/job/144019 [hyd-worker-11]

if this gets merged, we can close the PR#1566

Copy link

@lumag lumag left a comment

Choose a reason for hiding this comment

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

You are basically stating that things freez, let's stop testing it. Waht is the reason for the freeze?

@mattface
Copy link

I also tried moving the audiorecord test into a different test action so I could give it a sensible timeout, and then was surprised that all the tests passed LAVA Job 143047
I agree with @lumag that altering the tests to make them pass means the original failure will not get correctly triaged.

How about keeping the tests in the original order, but putting each test suite into a separate test action so that each test suite can have a sensible timeout. I believe AudioRecord only takes a few seconds to successfully run.
This will stop our LAVA jobs getting stuck for a long period of time, whilst also showing the failing tests as a failure to be investigated.

@qcom-anilyada
Copy link
Author

We raised this to unblock PR testing, which was getting stalled due to a single testcase running for 4 hours. Our analysis showed the issue was not build-related but caused by a testkit script problem in the remoteproc testcase, so the tests were temporarily shuffled until it is fixed.

So, we can either wait for the below PR to get merged and getting timed out, or merge this. Anyway, now we know what was causing this and provided fix for that.

The below issue and PR have RCA and fix for this issue:

@lumag
Copy link

lumag commented Feb 16, 2026

We raised this to unblock PR testing, which was getting stalled due to a single testcase running for 4 hours. Our analysis showed the issue was not build-related but caused by a testkit script problem in the remoteproc testcase, so the tests were temporarily shuffled until it is fixed.

So, we can either wait for the below PR to get merged and getting timed out, or merge this. Anyway, now we know what was causing this and provided fix for that.

The below issue and PR have RCA and fix for this issue:

If the issue is on the AudioReach side, my preference would be to drop AudioReach from qcom-distro rather than disabling the tests. Please send corresponding change.

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