New issue
Advanced search Search tips

Issue 694094 link

Starred by 10 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Lots of media CTS/GTS regression

Project Member Reported by kcwu@chromium.org, Feb 20 2017

Issue description

Since 9294.0.0, lots of CTS/GTS tests failed on many devices.
With bisect, I found the regression introduced by https://codereview.chromium.org/2538883002

CTS android.media, android.mediastress, videoperf
GTS google.exoplayer, google.media, google.youtube
are all affected.
on devices (at least, minnie, samus, and more).

FYI, google.youtube is a small package (4 of 4 tests failed), which should be easier to reproduce.

UI logs:
Unexpected crash report id length
System crash_reporter failed to process crash report.
Report Id:
[1432:1432:0220/003829.335954:ERROR:gpu_process_transport_factory.cc(888)] Lost UI shared context.
SYS_ioctl: VIDIOC_ENUM_FMT(2): fd=30, ret=-1, errno=22
SYS_ioctl: VIDIOC_ENUM_FMT(2): fd=30, ret=-1, errno=22
SYS_ioctl: VIDIOC_ENUM_FMT(2): fd=30, ret=-1, errno=22
SYS_ioctl: VIDIOC_ENUM_FMT(2): fd=30, ret=-1, errno=22
[6112:6112:0220/003829.462268:ERROR:gles2_cmd_decoder.cc(2467)] [GroupMarkerNotSet( crbug.com/242999 )!:6898F5B0]GL ERROR :GL_INVALID_FRAMEBUFFER_OPERATION : BackFramebuffer::Create: <- error from previous GL command
[6112:6112:0220/003829.483804:ERROR:texture_manager.cc(3224)] [.DisplayCompositor-0xc2d54200]GL ERROR :GL_INVALID_FRAMEBUFFER_OPERATION : glTexImage2D: <- error from previous GL command

 
Labels: -Pri-3 Pri-1
Owner: kcwu@chromium.org
Status: Assigned (was: Untriaged)
Please revert the CL today. Thanks.
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 20 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/43364fb75871b5f530f4a63d8f78fe7a3134efbb

commit 43364fb75871b5f530f4a63d8f78fe7a3134efbb
Author: kcwu <kcwu@chromium.org>
Date: Mon Feb 20 07:22:34 2017

Revert of Detect H264 slices of the same frame in VEAUnittest (patchset #6 id:140001 of https://codereview.chromium.org/2538883002/ )

Reason for revert:
This CL breaks many CTS/GTS tests.  crbug.com/694094 

Original issue's description:
> Detect H264 slices of the same frame in VEAUnittest
>
> This CL adds checks similar to H264Decoder::IsNewPrimaryCodedPicture() to
> determine if the given H264 slice belongs to a new frame. On Mac and Win,
> we can have multiple slices per frame and this causes problems.
>
> Note that after this fix, I will add this test to chromium.gpu.fyi Mac bots.
> Windows will follow when bots get upgraded to Win 8.1.
>
> BUG= 669678 
> TEST=Tested VEAUnittest on Macbook Air 2013.
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
>
> Review-Url: https://codereview.chromium.org/2538883002
> Cr-Commit-Position: refs/heads/master@{#451086}
> Committed: https://chromium.googlesource.com/chromium/src/+/e80ab729b5fbe7b6ee7cf44551b0412a2e03f51f

TBR=posciak@chromium.org,wuchengli@chromium.org,emircan@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 694094 

Review-Url: https://codereview.chromium.org/2707793002
Cr-Commit-Position: refs/heads/master@{#451575}

[modify] https://crrev.com/43364fb75871b5f530f4a63d8f78fe7a3134efbb/media/gpu/BUILD.gn
[modify] https://crrev.com/43364fb75871b5f530f4a63d8f78fe7a3134efbb/media/gpu/h264_decoder.cc
[modify] https://crrev.com/43364fb75871b5f530f4a63d8f78fe7a3134efbb/media/gpu/h264_decoder.h
[modify] https://crrev.com/43364fb75871b5f530f4a63d8f78fe7a3134efbb/media/gpu/video_encode_accelerator_unittest.cc

Status: Fixed (was: Assigned)
Issue 694774 has been merged into this issue.
Cc: hsiangc@chromium.org rohi...@chromium.org vsu...@chromium.org avkodipelli@chromium.org
Status: Verified (was: Fixed)
It seems Kuang-che already reverted and verified this.
Issue 694117 has been merged into this issue.

Sign in to add a comment