New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 651603 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Catch HDMI audio after suspend-resume failure

Project Member Reported by ka...@chromium.org, Sep 29 2016

Issue description

To prevent issues like  issue 651594 .

I haven't realized we don't have this scenario for HDMI audio after suspend. Will add to the basic test.


 

Comment 1 by ka...@chromium.org, Sep 29 2016

I reimaged a peppy DUT(android1758-audiobox2-host2) to 8743.44.0 and added suspend-resume before record-play starts, but oddly enough test passes.
Then I tried falco at M55 build and it passes too.

DUTs in lab are on test image. Still they have to fail, esp. as the bugs is consistent.


Comment 2 by ka...@chromium.org, Oct 3 2016

Reimaged falco tyoo with this same build R54-8743.44.0, and re-ran the test. It passed. Logs attached.

Next step - will try through close/iopen lid.

Comment 3 by ka...@chromium.org, Oct 5 2016

It appears the only way to reproduce this issue is when the suspend is while audio(Speaker or HDMI node) is playing. Any susbsequent suspend, even while not playing, will trigger it, after resume.

Comment 4 by ka...@chromium.org, Oct 5 2016

Respectively the test failed with  FAIL: Unhandled Fault: <Fault 1: "<class 'chameleond.utils.audio_utils.AudioCaptureManagerError'>:No audio data was captured. Perhaps this input is not plugged ?">

Preparing a test change.

Comment 5 by ka...@chromium.org, Oct 10 2016

Owner: cychiang@chromium.org
I have played with different DUTs and sleep times before and after suspend-resume, and could not come to consistent behavior for the failing(8743.44.0) and passing(8554.0.0 or ToT after fix) cases. It some times passes and some times fails. Within one same binding or separate one(just like the manual steps dictate).

I noticed the audio file to play is quite short - 2 to 3 seconds. It could be that while device is suspending the audio finishes(before full suspension).

Here is my change to the test - https://chromium-review.googlesource.com/396130

Jimmy, 
Can a longer playback be done at this test. Can you pass a longer golden file to check this?
Owner: ka...@chromium.org
Status: Started (was: Untriaged)
Hi Kalin, I have posted some comment on the CL.
Thanks!

Comment 7 by ka...@chromium.org, Dec 9 2016

Cc: -shrawan@chromium.org
I have this CL change up for review: https://chromium-review.googlesource.com/#/c/415115/
Project Member

Comment 8 by bugdroid1@chromium.org, Dec 14 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/9d9eeba7bd960d3c50b2ddca65963fb448950f91

commit 9d9eeba7bd960d3c50b2ddca65963fb448950f91
Author: Kalin Stoyanov <kalin@chromium.org>
Date: Wed Nov 30 18:33:34 2016

Adding suspend tests to HDMI test

- suspend - suspend and resume before testing HDMI audio
- suspend_while_playback - play audio, suspend and resume,
stop audio before testing HDMI audio

BUG=chromium:651603
TEST=tested DUTs in lab

Change-Id: Ifb5e65a795cd255d18357850f7b402a4d83cdd32
Reviewed-on: https://chromium-review.googlesource.com/415115
Commit-Ready: Kalin Stoyanov <kalin@chromium.org>
Tested-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Ruchi Jahagirdar <rjahagir@chromium.org>

[modify] https://crrev.com/9d9eeba7bd960d3c50b2ddca65963fb448950f91/server/site_tests/audio_AudioBasicHDMI/audio_AudioBasicHDMI.py
[add] https://crrev.com/9d9eeba7bd960d3c50b2ddca65963fb448950f91/server/site_tests/audio_AudioBasicHDMI/control.suspend_while_playback
[add] https://crrev.com/9d9eeba7bd960d3c50b2ddca65963fb448950f91/server/site_tests/audio_AudioBasicHDMI/control.suspend

You started fixing this bug over two years ago. Are you still working on it? You can update the status to "archived", "wontfix", or "closed". You can remove yourself as owner and change status to "untriaged", but if this is still a real bug, please do not sit on it.

Sign in to add a comment