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

Issue 901227 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 10
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

video_VideoEncodeAccelerator.vp8 fail on intel platform

Project Member Reported by deanliao@chromium.org, Nov 2

Issue description

video_VideoEncodeAccelerator.vp8 FAIL
video_VideoEncodeAccelerator.vp8.bvt GOOD

on guado 
https://stainless.corp.google.com/search?view=list&build=R72&test=video_VideoEncodeAccelerator.vp8&board=%5Eguado%24&first_date=2018-10-05&last_date=2018-11-02&status=GOOD&status=WARN&status=FAIL&status=ERROR

and buddy
https://stainless.corp.google.com/search?view=list&first_date=2018-10-05&last_date=2018-11-02&test=video_VideoEncodeAccelerator.vp8&build=R72&board=%5Ebuddy%24&status=GOOD&status=WARN&status=FAIL&status=ERROR&exclude_cts=false&exclude_not_run=false&exclude_non_release=true&exclude_au=true&exclude_acts=true&exclude_retried=true&exclude_non_production=false

Reason:
1 failures; <ForceBitrate/VideoEncodeAcceleratorTest.TestSimpleEncode/0>: ../../../../../../../home/chrome-bot/chrome_root/src/media/gpu/video_encode_accelerator_unittest.cc:2047; The difference between bitrate and current_requested_bitrate_ is 95858, which exceeds kBitrateTolerance * current_requested_bitrate_, where; bitrate evaluates to 695858,; current_requested_bitrate_ evaluates to 600000, and; kBitrateTolerance * current_requested_bitrate_ evaluates to 60000.

 
Status: Assigned (was: Untriaged)
Currently all Intel device cannot pass the bitrate switch test. We should disable it first. After we fix this issue, enable the test case again.
Summary: video_VideoEncodeAccelerator.vp8 fail on intel platform (was: video_VideoEncodeAccelerator.vp8 fail on guado and buddy)
Change the title of the issue.

Now all intel devices failed to switch bitrate at these test cases:
- 1.2Mbps => 300kbps
- 300kbps => 600kbps
- 600kbps => 1.2Mbps
Project Member

Comment 5 by bugdroid1@chromium.org, Dec 1

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

commit 1188c658f48640b2965055f20fd62252e3216618
Author: Chih-Yu Huang <akahuang@google.com>
Date: Sat Dec 01 09:08:48 2018

video_VideoEncodeAccelerator.vp8: Disable bitrate switch test.

Currently all Intel device cannot pass the bitrate switch tests. This
CL disabled these test cases.

BUG= chromium:901227 
TEST=Run video_VideoEncodeAccelerator.vp8 on Eve

Change-Id: I5c5e7f9d25829a5a52a1733db0515db66f8fe4ff
Reviewed-on: https://chromium-review.googlesource.com/1351322
Commit-Ready: Chih-Yu Huang <akahuang@chromium.org>
Tested-by: Chih-Yu Huang <akahuang@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[modify] https://crrev.com/1188c658f48640b2965055f20fd62252e3216618/client/site_tests/video_VideoEncodeAccelerator/control.vp8

Labels: Merge-Request-72
Hi,

The CL in #5 is landed at M73. Please approve to merge to M72.
Project Member

Comment 7 by sheriffbot@chromium.org, Dec 4

Labels: -Merge-Request-72 Hotlist-Merge-Approved Merge-Approved-72
Your change meets the bar and is auto-approved for M72. Please go ahead and merge the CL to branch 3626 manually. Please contact milestone owner if you have questions.
Owners: govind@(Android), kariahda@(iOS), djmm@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 8 by bugdroid1@chromium.org, Dec 4

Labels: merge-merged-release-R72-11316.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/f787765fd57aac41d8e09c2a0411041c44762d15

commit f787765fd57aac41d8e09c2a0411041c44762d15
Author: Chih-Yu Huang <akahuang@google.com>
Date: Tue Dec 04 01:48:59 2018

video_VideoEncodeAccelerator.vp8: Disable bitrate switch test.

Currently all Intel device cannot pass the bitrate switch tests. This
CL disabled these test cases.

BUG= chromium:901227 
TEST=Run video_VideoEncodeAccelerator.vp8 on Eve

Change-Id: I5c5e7f9d25829a5a52a1733db0515db66f8fe4ff
Reviewed-on: https://chromium-review.googlesource.com/1351322
Commit-Ready: Chih-Yu Huang <akahuang@chromium.org>
Tested-by: Chih-Yu Huang <akahuang@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
(cherry picked from commit 1188c658f48640b2965055f20fd62252e3216618)
Reviewed-on: https://chromium-review.googlesource.com/c/1360014
Reviewed-by: Chih-Yu Huang <akahuang@chromium.org>
Commit-Queue: Chih-Yu Huang <akahuang@chromium.org>
Trybot-Ready: Chih-Yu Huang <akahuang@chromium.org>

[modify] https://crrev.com/f787765fd57aac41d8e09c2a0411041c44762d15/client/site_tests/video_VideoEncodeAccelerator/control.vp8

Project Member

Comment 9 by sheriffbot@chromium.org, Dec 7

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
Labels: -Merge-Approved-72
Status: Fixed (was: Assigned)

Sign in to add a comment