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

Issue 691464 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocked on:
issue chrome-os-partner:63040

Blocking:
issue 694372



Sign in to add a comment

video_VideoDecodeAccelerator is failed on ELM since 9246.0

Project Member Reported by owenlin@chromium.org, Feb 13 2017

Issue description

Cc: wuchengli@chromium.org
First, without the --single-process-tests, the test cannot finish. 
After adding the flag, the md5sum doesn't match to golden value. Not sure about the "PVR:(ERROR)".

(7046) PVR:(Error): IsTextureConsistent: Texture is not mipmap complete [ :0 ]
(7046) PVR:(Error): IsTextureConsistent: _EGLImage is not consistent [ :0 ]
(7046) PVR:(Error): IsTextureConsistent: Texture is not mipmap complete [ :0 ]
(7046) PVR:(Error): IsTextureConsistent: _EGLImage is not consistent [ :0 ]


[7045:7045:0213/005613.760606:3433310798:INFO:video_decode_accelerator_unittest.cc(1478)] Decoder 0 fps: 280.872
[7045:7045:0213/005614.148436:3433698623:ERROR:video_decode_accelerator_unittest.cc(1515)] Unknown thumbnails MD5: af21b8aa5ea63c4e88f0472ee948d01a
../../media/gpu/video_decode_accelerator_unittest.cc:1523: Failure
Value of: static_cast<int>(png.size())
  Actual: 234628
Expected: num_bytes
Which is: -1
Chrome version of 9246.0.0 and 9245.0.0 are the same.

Here are ChromeOS CLs between 9245 and 9246:
https://crosland.corp.google.com/log/9245.0.0..9246.0.0
Status: Available (was: Untriaged)
This cl may be related: 
media-libs/[arc-]img-ddk: Update IMG PowerVR Rogue driver to 1.7 release
Labels: -Pri-2 Pri-1
Status: Assigned (was: Available)
Owen. Can you see if new img ddk changes the output of thumbnail? If yes, please help update md5. 
Cc: djkurtz@chromium.org
Owner: drinkcat@chromium.org
When I tried to generate new md5 checksum, I found the value varies for different runs. It could be the same for several times, but after reboot, it usually changed to another.

I think there are some other issues. Besides, The error messages shown in #1 was not seen at 9245 but shown at 9246.

https://cs.corp.google.com/chromeos_internal/src/partner_private/img-ddk/opengles3/texmgmt.c?q=%22Texture+is+not+mipmap%22&dr=C&l=4119

We use GL command to get the thumbnails, so I assume this is a rendering (GL) issuel.

Hi Nicolas, could you help to triage the issue and assign it to the right owner? Thanks.

Cc: drinkcat@chromium.org conradlo@chromium.org littlecvr@chromium.org
That's when we upgraded the DDK from 1.6 to 1.7: https://crosland.corp.google.com/log/9245.0.0..9246.0.0

Owen: Can you provide simple instructions to reproduce this? I assume:
test_that IP video_VideoDecodeAccelerator

Might be best to file a partner bug and assign to oak-img@chromium.org (I can't seem to assign to them here).
Owen. Can you upload a video_decode_accelerator_unittest binary to google drive for drinkcat@? It's faster to run the test binary than autotest.
Nicolas, Thank you. Will file a partner bug. 

Wucheng, that's a good idea. Will upload the binary as well as the test streams.
Blockedon: chrome-os-partner:63040
#9: Thanks!
Blocking: 694372
Status: Fixed (was: Assigned)
Should be fixed from R58-9310.0.0.
Status: Available (was: Fixed)
Actually, I still see failures on 9310.0.0 (the similar-looking glbench issue is fixed though).

Owen: Can you check if this is the same issue?
Owner: owenlin@chromium.org
Project Member

Comment 15 by bugdroid1@chromium.org, Mar 1 2017

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

commit 3171948f74e9f5ffab0d31e928d14524cb4fcc20
Author: Owen Lin <owenlin@google.com>
Date: Wed Mar 01 13:04:02 2017

video_VideoDecodeAccelerator: Skipped the thumbnail test on ELM.

Before chromium:691464 getting fixed, skip the thumbnail test on ELM so
other breakages won't get ignored.

BUG= chromium:691464 
TEST=Run the test on ELM.

Change-Id: Iecda21ec376162b8daa68951d8df94735174522c
Reviewed-on: https://chromium-review.googlesource.com/445417
Commit-Ready: Owen Lin <owenlin@chromium.org>
Tested-by: Owen Lin <owenlin@chromium.org>
Reviewed-by: Pawel Osciak <posciak@chromium.org>
Reviewed-by: Wu-cheng Li <wuchengli@chromium.org>

[modify] https://crrev.com/3171948f74e9f5ffab0d31e928d14524cb4fcc20/client/site_tests/video_VideoDecodeAccelerator/video_VideoDecodeAccelerator.py

Status: Assigned (was: Available)
Owen. According to b/35648505, the issue on elm is fixed in R58-9310.0.0. Can you try re-enabling the test? Thanks.
Status: Fixed (was: Assigned)
Let's use https://bugs.chromium.org/p/chromium/issues/detail?id=694372 to track re-enabling the test.
Project Member

Comment 18 by bugdroid1@chromium.org, Mar 7 2017

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

commit 7c14a585ef7d98436f3f4eb72145245821cbe2d6
Author: Owen Lin <owenlin@chromium.org>
Date: Tue Mar 07 10:32:40 2017

Revert "video_VideoDecodeAccelerator: Skipped the thumbnail test on ELM."

This reverts commit 3171948f74e9f5ffab0d31e928d14524cb4fcc20.

Reason for revert: The issue has been fixed on ELM.

Original change's description:
> video_VideoDecodeAccelerator: Skipped the thumbnail test on ELM.
>
> Before chromium:691464 getting fixed, skip the thumbnail test on ELM so
> other breakages won't get ignored.
>
> BUG= chromium:691464 
> TEST=Run the test on ELM.
>
> Change-Id: Iecda21ec376162b8daa68951d8df94735174522c
> Reviewed-on: https://chromium-review.googlesource.com/445417
> Commit-Ready: Owen Lin <owenlin@chromium.org>
> Tested-by: Owen Lin <owenlin@chromium.org>
> Reviewed-by: Pawel Osciak <posciak@chromium.org>
> Reviewed-by: Wu-cheng Li <wuchengli@chromium.org>
>

TBR=posciak@chromium.org,wuchengli@chromium.org,owenlin@google.com,owenlin@chromium.org
BUG= chromium:694372 

Change-Id: I1f2286824c4b94424a0747f8805de75437f10460
Reviewed-on: https://chromium-review.googlesource.com/450200
Commit-Ready: Owen Lin <owenlin@chromium.org>
Tested-by: Owen Lin <owenlin@chromium.org>
Reviewed-by: Pawel Osciak <posciak@chromium.org>

[modify] https://crrev.com/7c14a585ef7d98436f3f4eb72145245821cbe2d6/client/site_tests/video_VideoDecodeAccelerator/video_VideoDecodeAccelerator.py

Project Member

Comment 19 by bugdroid1@chromium.org, Mar 7 2017

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

commit 7c14a585ef7d98436f3f4eb72145245821cbe2d6
Author: Owen Lin <owenlin@chromium.org>
Date: Tue Mar 07 10:32:40 2017

Revert "video_VideoDecodeAccelerator: Skipped the thumbnail test on ELM."

This reverts commit 3171948f74e9f5ffab0d31e928d14524cb4fcc20.

Reason for revert: The issue has been fixed on ELM.

Original change's description:
> video_VideoDecodeAccelerator: Skipped the thumbnail test on ELM.
>
> Before chromium:691464 getting fixed, skip the thumbnail test on ELM so
> other breakages won't get ignored.
>
> BUG= chromium:691464 
> TEST=Run the test on ELM.
>
> Change-Id: Iecda21ec376162b8daa68951d8df94735174522c
> Reviewed-on: https://chromium-review.googlesource.com/445417
> Commit-Ready: Owen Lin <owenlin@chromium.org>
> Tested-by: Owen Lin <owenlin@chromium.org>
> Reviewed-by: Pawel Osciak <posciak@chromium.org>
> Reviewed-by: Wu-cheng Li <wuchengli@chromium.org>
>

TBR=posciak@chromium.org,wuchengli@chromium.org,owenlin@google.com,owenlin@chromium.org
BUG= chromium:694372 

Change-Id: I1f2286824c4b94424a0747f8805de75437f10460
Reviewed-on: https://chromium-review.googlesource.com/450200
Commit-Ready: Owen Lin <owenlin@chromium.org>
Tested-by: Owen Lin <owenlin@chromium.org>
Reviewed-by: Pawel Osciak <posciak@chromium.org>

[modify] https://crrev.com/7c14a585ef7d98436f3f4eb72145245821cbe2d6/client/site_tests/video_VideoDecodeAccelerator/video_VideoDecodeAccelerator.py

Comment 20 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59
Cc: vsu...@chromium.org avkodipelli@chromium.org
Again this is failing on M60 from R60-9496.0.0.

https://wmatrix.googleplex.com/platform/unfiltered?platforms=elm&tests=video_VideoDecodeAccelerator&days_back=60&releases=60&hide_missing=True

Need to file separate bug or reopen same issue?
Please file a new bug, the root is different. It is timeout this time.

05/15 08:19:48.585 DEBUG|             utils:0297| [stdout] [39/39] DecodeVariations/VideoDecodeAcceleratorParamTest.TestSimpleDecode/0 (TIMED OUT)
05/15 08:19:48.587 DEBUG|             utils:0297| [stdout] 3 tests timed out:
05/15 08:19:48.588 DEBUG|             utils:0297| [stdout]     DecodeVariations/VideoDecodeAcceleratorParamTest.TestSimpleDecode/0 (../../../../../../../home/chrome-bot/chrome_root/src/media/gpu/video_decode_accelerator_unittest.cc:1381)
05/15 08:19:48.589 DEBUG|             utils:0297| [stdout]     TearDownTiming/VideoDecodeAcceleratorParamTest.TestSimpleDecode/2 (../../../../../../../home/chrome-bot/chrome_root/src/media/gpu/video_decode_accelerator_unittest.cc:1381)
05/15 08:19:48.590 DEBUG|             utils:0297| [stdout]     TearDownTiming/VideoDecodeAcceleratorParamTest.TestSimpleDecode/3 (../../../../../../../home/chrome-bot/chrome_root/src/media/gpu/video_decode_accelerator_unittest.cc:1381)
05/15 08:19:48.591 DEBUG|             utils:0297| [stdout] Tests took 1461 seconds.
Status: Verified (was: Fixed)
Closing this issue and opened new issue https://bugs.chromium.org/p/chromium/issues/detail?id=723035

Sign in to add a comment