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

Issue 866577 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 24
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

graphics_dEQP.bvt fails on grunt

Project Member Reported by sjg@chromium.org, Jul 23

Issue description

Cc: ddavenp...@chromium.org ihf@chromium.org
Components: OS>Kernel>Graphics
Labels: OS-Chrome
Owner: basni@chromium.org
http://cautotest-prod/tko/retrieve_logs.cgi?job=/results/219677741-chromeos-test/

07/22 15:12:56.131 INFO |     graphics_dEQP:0318| [35/38] TestCase: dEQP-VK.api.info.device.properties
07/22 15:12:56.132 DEBUG|     graphics_dEQP:0344| Running single: /usr/local/deqp/external/vulkancts/modules/vulkan/deqp-vk --deqp-case=dEQP-VK.api.info.device.properties --deqp-surface-type=pbuffer --deqp-gl-config-name=rgba8888d24s8ms0 --deqp-log-images=disable --deqp-watchdog=enable --deqp-surface-width=256 --deqp-surface-height=256 --deqp-log-filename=/tmp/bvt.txt-logs/dEQP-VK.api.info.device.properties.log
07/22 15:12:56.132 DEBUG|             utils:0219| Running '/usr/local/deqp/external/vulkancts/modules/vulkan/deqp-vk --deqp-case=dEQP-VK.api.info.device.properties --deqp-surface-type=pbuffer --deqp-gl-config-name=rgba8888d24s8ms0 --deqp-log-images=disable --deqp-watchdog=enable --deqp-surface-width=256 --deqp-surface-height=256 --deqp-log-filename=/tmp/bvt.txt-logs/dEQP-VK.api.info.device.properties.log'
07/22 15:13:00.662 INFO |     graphics_dEQP:0401| Result: Fail

No terribly surprising, since we just enabled Vulkan on Friday:
b/111559548
https://chromium-review.googlesource.com/1141872


Status: Assigned (was: Untriaged)
Cc: chadversary@chromium.org
The mesa uprev is in the cq-ish. Could that fix the Vulkan failures? I'll take a quick look at the failures.
Labels: M-70
The log just shows the one failure in dEQP-VK.api.info.device.properties. The other VK tests are passing on grunt. We could remove the test from 
~/trunk/src/third_party/autotest/files/client/site_tests/graphics_dEQP/master/bvt.txt
until fixed (and file separate bugs for all the new VK failures after the mesa uprev).

From running it locally:

 <Text>deviceProperties - API Minor Version 1 is not valid for this version of dEQP</Text>


Seems like the deqp is tool old for vulkan 1.1. 

We have 3 options:

1) uprev deqp
2) Whitelist vulkan 1.1 with a patch until next deqp uprev.
3) Downgrade radv for 1.0 with a patch. (I think this is totally reasonable as we likely cannot enable vulkan 1.1 for ARC++ as is, need some extra ext work which is amost but not yet done)

Let me know what would be preferred. Long term I'm guessing (1), but if it is blocking builds I think we should go for one of the other two.
Cc: basni@chromium.org
Owner: ihf@chromium.org
I will remove the test from bvt. We should uprev deqp sometimes soon, but that will take a few weeks.
Project Member

Comment 7 by bugdroid1@chromium.org, Jul 23

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

commit 8ec2212c457ea045fb1b08ce9595aadb45792a52
Author: Ilja H. Friedel <ihf@chromium.org>
Date: Mon Jul 23 20:17:37 2018

graphics_dEQP: remove dEQP-VK.api.info.device.properties

Remove dEQP-VK.api.info.device.properties from bvt set as it
is incorrectly failing on grunt.

This change should be reverted after a deqp uprev.

BUG= chromium:866577 
TEST=None.

Change-Id: I7044fbcf3ad96f251e91d1e8e10173e9fb507a72
Reviewed-on: https://chromium-review.googlesource.com/1147259
Commit-Queue: Ilja H. Friedel <ihf@chromium.org>
Tested-by: Ilja H. Friedel <ihf@chromium.org>
Trybot-Ready: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Bas Nieuwenhuizen <basni@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>

[modify] https://crrev.com/8ec2212c457ea045fb1b08ce9595aadb45792a52/client/site_tests/graphics_dEQP/master/bvt.txt

Status: Fixed (was: Assigned)
Project Member

Comment 9 by bugdroid1@chromium.org, Jul 24

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

commit d837e20ef2aab9190aabc5b515d5049175db89f1
Author: Ilja H. Friedel <ihf@chromium.org>
Date: Tue Jul 24 06:03:01 2018

graphics_dEQP: move VK to daily suite.

Especially on grunt graphics_dEQP.vk-master runs for many hours.
The timeout on bvt-perbuild is one hour.

BUG= chromium:866577 
TEST=None.

Change-Id: I79c981b19e692121b0fecef98d0b665a85c2be91
Reviewed-on: https://chromium-review.googlesource.com/1147927
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Tested-by: Ilja H. Friedel <ihf@chromium.org>

[modify] https://crrev.com/d837e20ef2aab9190aabc5b515d5049175db89f1/client/site_tests/graphics_dEQP/control.vk-master
[modify] https://crrev.com/d837e20ef2aab9190aabc5b515d5049175db89f1/client/site_tests/graphics_dEQP/generate_controlfiles.py

Sign in to add a comment