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

Issue 698061 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

graphics_dEQP.vk.api.smoke is failing on glados and kunimitsu boards

Project Member Reported by rohi...@chromium.org, Mar 2 2017

Issue description

Labels: bvttriage

Comment 2 by ihf@chromium.org, Mar 3 2017

Cc: za...@chromium.org ihf@chromium.org
Labels: -Pri-1 -ReleaseBlock-Dev -M-58 Pri-3
Owner: chadversary@chromium.org
Basically Zach pulled in newest dEQP, which enabled a lot of new VK tests, of which about 120 are failing. 2 of which in graphics_dEQP.vk.api.smoke. This needs fixing, but is not time critical whatsoever. This bug should be mostly for tracking.

Comment 3 by ihf@chromium.org, Mar 3 2017

Chad, if you think graphics_dEQP.vk.api.smoke can't be fixed soon, then we may want to move that one to the daily suite.

Comment 4 by ihf@chromium.org, Mar 8 2017

Cc: chadversary@chromium.org
 Issue 699294  has been merged into this issue.

Comment 5 by ihf@chromium.org, Mar 8 2017

Chad, as suggested in #3 we should move a (semi-)permanently expected to be failing test from bvt to avoid that people start reverting your change.

Bernie, do you suspect the VK changes of other regressions? We are just trying to track the status of the new feature and the new deqp tests being developed.
Not sure about other regressions from it, my main goal was to have the R58 branch all green in autotest so we don't have to have the test team looking at this failure every week there, and if we are not going to ever use Vulcan on 58 we may as well just disable it more out right.

Having these on ToT is fine if we are working on them IMO, though if we think this will take a few weeks to get a green state we might want to do as you suggest in comment 3 as a temporary measure until we can pass.
On R58, the Vulkan tests will be disabled after the CQ flushes. See 

crbug/699294
https://chromium-review.googlesource.com/c/451550/
https://chromium-review.googlesource.com/c/450875/

For ToT, I'm building a new image and investigating now.

Status: Started (was: Assigned)
Fix uploaded to Gerrit
https://chromium-review.googlesource.com/c/452788/
Project Member

Comment 9 by bugdroid1@chromium.org, Mar 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/7361ae73ae77d92f9cc43e739b204c6c15d84db4

commit 7361ae73ae77d92f9cc43e739b204c6c15d84db4
Author: Chad Versace <chadversary@chromium.org>
Date: Fri Mar 10 06:18:44 2017

media-gfx/deqp: Fix Vulkan tests

Regressed by:
  commit 3d8e6fd4828764cd96938434729deb543c4fdcfb
  Author: Zach Reizner <zachr@google.com>
  Date:   Tue Feb 21 17:16:12 2017 -0800
  Subject: media-gfx/deqp: sync deqp to aosp

When upgrading dEQP, the above commit missed two things:
  - The spirv-tools commit changed.
  - dEQP added a new dependency, the spirv-headers.

BUG= chromium:698061 
TEST=emerge-sentry deqp; cros deploy --board=sentry $DUT deqp;
     test_that --board=sentry $DUT graphics_dEQP.vk.api.smoke

Change-Id: I0ee3998626a6c8121a5bd90376806009c24ef6e9
Reviewed-on: https://chromium-review.googlesource.com/452788
Commit-Ready: Chad Versace <chadversary@chromium.org>
Tested-by: Chad Versace <chadversary@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Zach Reizner <zachr@chromium.org>

[modify] https://crrev.com/7361ae73ae77d92f9cc43e739b204c6c15d84db4/media-gfx/deqp/Manifest
[rename] https://crrev.com/7361ae73ae77d92f9cc43e739b204c6c15d84db4/media-gfx/deqp/deqp-2017.02.22-r2.ebuild

Project Member

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

Labels: merge-merged-release-R58-9334.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/71674d70da7ace6556a1d21832d7cdf620688f30

commit 71674d70da7ace6556a1d21832d7cdf620688f30
Author: Chad Versace <chadversary@chromium.org>
Date: Wed Mar 15 20:23:22 2017

media-gfx/deqp: Fix Vulkan tests

Regressed by:
  commit 3d8e6fd4828764cd96938434729deb543c4fdcfb
  Author: Zach Reizner <zachr@google.com>
  Date:   Tue Feb 21 17:16:12 2017 -0800
  Subject: media-gfx/deqp: sync deqp to aosp

When upgrading dEQP, the above commit missed two things:
  - The spirv-tools commit changed.
  - dEQP added a new dependency, the spirv-headers.

BUG= chromium:698061 
TEST=emerge-sentry deqp; cros deploy --board=sentry $DUT deqp;
     test_that --board=sentry $DUT graphics_dEQP.vk.api.smoke

Change-Id: I0ee3998626a6c8121a5bd90376806009c24ef6e9
Reviewed-on: https://chromium-review.googlesource.com/452788
Commit-Ready: Chad Versace <chadversary@chromium.org>
Tested-by: Chad Versace <chadversary@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Zach Reizner <zachr@chromium.org>
(cherry picked from commit 7361ae73ae77d92f9cc43e739b204c6c15d84db4)
Reviewed-on: https://chromium-review.googlesource.com/455700
Tested-by: Katherine Hsieh <Katherine.Hsieh@quantatw.com>

[modify] https://crrev.com/71674d70da7ace6556a1d21832d7cdf620688f30/media-gfx/deqp/Manifest
[rename] https://crrev.com/71674d70da7ace6556a1d21832d7cdf620688f30/media-gfx/deqp/deqp-2017.02.22-r2.ebuild

Status: Fixed (was: Started)

Comment 12 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 14 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment