search

thesofproject / sof-test

BSD 3-Clause "New" or "Revised" License
13 stars 46 forks source link

Catch aplay error messages in apause.exp and exit #1226

Closed marc-hb closed 3 months ago

marc-hb commented 3 months ago

Always better to "fail fast".

See commit messages.

marc-hb commented 3 months ago

Perfect demonstration of the value of this PR:

https://sof-ci.01.org/softestpr/PR1226/build710/devicetest/index.html?model=LNLM_RVP_NOCODEC&testcase=check-pause-resume-playback-100

t=13242 ms: aplay: Pressing SPACE
t=14869 ms: aplay: (32/100) Found   === PAUSE ===  ,  pausing for 137 ms
t=14869 ms: aplay: ERROR: aplay: do_pause:1567:
pause push error: File descriptor in bad state

Apparent memory corruption here:

https://sof-ci.01.org/softestpr/PR1226/build710/devicetest/index.html?model=LNLM_RVP_NOCODEC&testcase=multiple-pause-resume-50

[ 1515.873143] <inf> ipc: ipc_cmd: rx   : 0x13000004|0x0
[ 1515.873160] <inf> pipe: pipeline_trigger: pipe:0 0x0 pipe trigger cmd 8
[ 1515.873168] <inf> ll_schedule: zephyr_ll_task_schedule_common: task add 0xa0119000 0xa00ebf54U priority 0 flags 0x0
������������������������������������������������������������������������������������������������������������������������������������������������������
������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������
��������������������������������������������������
���������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������

https://sof-ci.01.org/softestpr/PR1226/build710/devicetest/index.html?model=LNLM_SDW_AIOC&testcase=verify-kernel-boot-log is unrelated issue https://github.com/thesofproject/linux/issues/5124

Everything else is 100% green.

marc-hb commented 3 months ago

SOFCI TEST

marc-hb commented 3 months ago

LNL https://sof-ci.01.org/softestpr/PR1226/build716/devicetest/index.html?model=LNLM_SDW_AIOC&testcase=multiple-pause-resume-50 has pause push error: File descriptor in bad state across three models.

plbossart commented 3 months ago

"pause push error: File descriptor in bad state" is not necessarily a problem, we see it all the time when resuming and INFO_RESUME is not supported.

I've seen this log countless times and things were just fine, alsa-lib reinitialized whatever was needed with a prepare.

Adding @ranj063 and @ujfalusi to make sure I am not completely mistaken.

marc-hb commented 3 months ago

"pause push error: File descriptor in bad state" is not necessarily a problem, we see it all the time when resuming and INFO_RESUME is not supported.

Mmmm.... questions:

marc-hb commented 3 months ago

Can you please elaborate on "INFO_RESUME is not supported." ? Is it the same as "pause not supported"

Yes it's the same according to @ranj063 .

Thanks for the reviews. This is a well tested and small change - easy to revert parts of it if needed so I'm merging it.