[kitty] cras crash encountered with external monitor connected |
|||||||||
Issue descriptionGoogle Chrome : 50.0.2661.11 (Official Build) dev (64-bit) Platform : 7978.4.0 (Official Build) dev-channel What steps will reproduce the problem? 1. Sign in the device with hdmi external monitor connected (here monroe) 2. Change between mirror/extended mode by pressing ALt-F4 3. VT2-VT1 if the issue is not observed What is the expected output? No crash, no hang, both internal and external monitors display What do you see instead? Crash crash, hang, both internal and external monitors only display mouse cursor Almost always - the device hang very easily - power off and on button does not seem work anymore after the crash. Crash report: https://crash.corp.google.com/browse?q=&ignore_case=false&stbtiq=F7C2D8BFC76647339E4088BB4461B225&reportid=3ed257f000000000 https://crash.corp.google.com/browse?q=&ignore_case=false&stbtiq=F7C2D8BFC76647339E4088BB4461B225&reportid=75e257f000000000 https://crash.corp.google.com/browse?q=&ignore_case=false&stbtiq=F7C2D8BFC76647339E4088BB4461B225&reportid=c60437f000000000 more info: when kitty connected to LG phone, with no external monitor connected, powerd_dbus_suspend/resume, kitty screen only shows mouse cursor, but no cras crash.
,
Mar 4 2016
,
Mar 4 2016
correction : - Ctr-F4 to switch between mode. - Audio play during the test.
,
Mar 4 2016
,
Mar 6 2016
Looks like the recent change: https://chromium-review.googlesource.com/328825 Frank could you please take a look since it is a release blocker ? Thanks!
,
Mar 6 2016
Oh, I saw it is reverted on master branch. Cherry-pick of reverting posted: https://chromium-review.googlesource.com/#/c/331020/ Merge request for R50. Thanks!
,
Mar 7 2016
Your change meets the bar and is auto-approved for M50 (branch: 2661)
,
Mar 7 2016
The dsp_util change which was reverted affected audio where samples was not a multiple of 8. It would cause up to 7 extra samples to be written.
,
Mar 8 2016
https://chromium-review.googlesource.com/#/c/331020/ was merged to R50. Hi Frank, thanks for the update. Please test on a kitty board when you plan to submit the updated patch.
,
Mar 8 2016
fbarchard@ - we are approaching 50 Beta season as of March 10th. If you would like this fix to go in please provide a CL that needs merge and a merge request as soon as possible.
,
Mar 11 2016
verified in ChromeOS 7978.10.0 / 50.0.2661.20
,
Apr 29 2016
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 2 2016
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by helenzhang@google.com
, Mar 4 2016