Tests broken - issue 640651 and issue 640653 Tested Celes and Link on R54-8737.0.0 Audio jack and USB audio input is broken. There is no audio produced from microphone. Generated logs will be attached to this bug.
logs are present at https://pantheon.corp.google.com/storage/browser/chromiumos-test-logs/bugfiles/cros/640659/?debugUI=CLOUD
audio Microphone test is passing at R54-8733.0.0, but failing at R54-8734.0.0 https://screenshot.googleplex.com/W9HhsgbQbMt
log of link device: https://pantheon.corp.google.com/storage/browser/chromiumos-test-logs/bugfiles/cros/640659/?debugUI=CLOUD observed /var/log/messages error out : chronos@localhost / $ tail -f /var/log/messages 2016-08-24T17:10:25.753436+00:00 ERR cras_server[3645]: pcm_avail returned frames larger than buf_size: 22105776 > 16384
This Error log (pcm_avail) is flooding the messages logs - https://chromium-review.googlesource.com/#/c/371118/
Reproduce the issue on CrOS 8737.0.0/54.0.2837.0 - Candy.
Sounds like a dup of issue 640048
Nope. Issue 640048 is audio output issue on chell only.
I think this is caused by iodev refactoring change. I will check and revert it.
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/adhd/+/f70e99042ea6ed37127def9e98afa9758f870e3d commit f70e99042ea6ed37127def9e98afa9758f870e3d Author: Cheng-Yi Chiang <cychiang@chromium.org> Date: Thu Aug 25 03:05:03 2016 Revert "CRAS: iodev - Move no stream state transition into cras_iodev" This reverts commit 2a98fae9e8eb798f7a16481a578594b787b93ff7. The change somehow broke recording path. Revert it now and investigate it later. BUG= chromium:640659 , chromium:519942 TEST=check recording works fine on chell. Signed-off-by: Cheng-Yi Chiang <cychiang@chromium.org> Change-Id: Id8b0aa175b4cf1388889a7d7f00e9eb6c5243df8 Reviewed-on: https://chromium-review.googlesource.com/375698 [modify] https://crrev.com/f70e99042ea6ed37127def9e98afa9758f870e3d/cras/src/server/cras_iodev.h [modify] https://crrev.com/f70e99042ea6ed37127def9e98afa9758f870e3d/cras/src/tests/iodev_unittest.cc [modify] https://crrev.com/f70e99042ea6ed37127def9e98afa9758f870e3d/cras/src/server/audio_thread.c [modify] https://crrev.com/f70e99042ea6ed37127def9e98afa9758f870e3d/cras/src/tests/audio_thread_unittest.cc [modify] https://crrev.com/f70e99042ea6ed37127def9e98afa9758f870e3d/cras/src/server/cras_iodev.c
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/adhd/+/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7 commit d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7 Author: Cheng-Yi Chiang <cychiang@chromium.org> Date: Thu Aug 25 03:05:16 2016 Revert "CRAS: iodev - Add state variable in cras_iodev" This reverts commit e9e2c8e19fa28a469372bfa3e5fd942dcbb26f90. The change somehow broke recording path. Revert it now and investigate it later. BUG= chromium:640659 , chromium:519942 TEST=check recording works fine on chell. Signed-off-by: Cheng-Yi Chiang <cychiang@chromium.org> Change-Id: I2f6b17fd158f4063c69154810a14ed1b4e74e1b9 Reviewed-on: https://chromium-review.googlesource.com/375699 [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_a2dp_iodev.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_iodev_list.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/tests/iodev_list_unittest.cc [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_hfp_iodev.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/tests/hfp_iodev_unittest.cc [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_bt_device.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/tests/loopback_iodev_unittest.cc [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_empty_iodev.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_iodev.h [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/tests/iodev_unittest.cc [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_loopback_iodev.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/audio_thread.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/tests/audio_thread_unittest.cc [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/tests/alsa_io_unittest.cc [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/tests/bt_device_unittest.cc [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/test_iodev.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_iodev.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/tests/bt_io_unittest.cc [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_alsa_io.c [modify] https://crrev.com/d64e3f69de2bc7f96418d32c43ae7d3d1a6b4af7/cras/src/server/cras_bt_io.c
These two CL on ToT are waiting to be merged https://chromium-review.googlesource.com/#/c/375658/ https://chromium-review.googlesource.com/#/c/375659/1 I am investigating the root cause. Sorry for the trouble.
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/adhd/+/85502b7eea8bb8b4e2b6842d93ca892c2ff10a5a commit 85502b7eea8bb8b4e2b6842d93ca892c2ff10a5a Author: Cheng-Yi Chiang <cychiang@chromium.org> Date: Thu Aug 25 03:05:03 2016 Revert "CRAS: iodev - Move no stream state transition into cras_iodev" This reverts commit 2a98fae9e8eb798f7a16481a578594b787b93ff7. The change somehow broke recording path. Revert it now and investigate it later. BUG= chromium:640659 , chromium:519942 TEST=check recording works fine on chell. Signed-off-by: Cheng-Yi Chiang <cychiang@chromium.org> Change-Id: Id8b0aa175b4cf1388889a7d7f00e9eb6c5243df8 Reviewed-on: https://chromium-review.googlesource.com/375658 [modify] https://crrev.com/85502b7eea8bb8b4e2b6842d93ca892c2ff10a5a/cras/src/server/cras_iodev.h [modify] https://crrev.com/85502b7eea8bb8b4e2b6842d93ca892c2ff10a5a/cras/src/tests/iodev_unittest.cc [modify] https://crrev.com/85502b7eea8bb8b4e2b6842d93ca892c2ff10a5a/cras/src/server/audio_thread.c [modify] https://crrev.com/85502b7eea8bb8b4e2b6842d93ca892c2ff10a5a/cras/src/tests/audio_thread_unittest.cc [modify] https://crrev.com/85502b7eea8bb8b4e2b6842d93ca892c2ff10a5a/cras/src/server/cras_iodev.c
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/adhd/+/8ebc96cf4135d449b8473e191386e0f896372039 commit 8ebc96cf4135d449b8473e191386e0f896372039 Author: Cheng-Yi Chiang <cychiang@chromium.org> Date: Thu Aug 25 03:05:16 2016 Revert "CRAS: iodev - Add state variable in cras_iodev" This reverts commit e9e2c8e19fa28a469372bfa3e5fd942dcbb26f90. The change somehow broke recording path. Revert it now and investigate it later. BUG= chromium:640659 , chromium:519942 TEST=check recording works fine on chell. Signed-off-by: Cheng-Yi Chiang <cychiang@chromium.org> Change-Id: I2f6b17fd158f4063c69154810a14ed1b4e74e1b9 Reviewed-on: https://chromium-review.googlesource.com/375659 [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_a2dp_iodev.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_iodev_list.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/tests/iodev_list_unittest.cc [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_hfp_iodev.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/tests/hfp_iodev_unittest.cc [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_bt_device.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/tests/loopback_iodev_unittest.cc [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_empty_iodev.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_iodev.h [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/tests/iodev_unittest.cc [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_loopback_iodev.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/audio_thread.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/tests/audio_thread_unittest.cc [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/tests/alsa_io_unittest.cc [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/tests/bt_device_unittest.cc [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/test_iodev.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_iodev.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/tests/bt_io_unittest.cc [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_alsa_io.c [modify] https://crrev.com/8ebc96cf4135d449b8473e191386e0f896372039/cras/src/server/cras_bt_io.c
This is fixed as the two revert CLs were merged. Waiting for next build test result to mark verified.
Thanks Jimmy - R54-8740.0.0 build is passing audio_Microphone test. https://wmatrix.googleplex.com/unfiltered?tests=audio_Microphone Will confirm with chameleon tests too.
Comment 1 by son...@google.com
, Aug 24 2016