Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Issue 2185953003: Add gyp define to enable vulkan debug layers (Closed)

Created:
4 years, 4 months ago by egdaniel
Modified:
4 years, 4 months ago
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Add gyp define to enable vulkan debug layers On Debug vulkan bots, running with the debug layers on seems to be adding more than an hour to the total running time. Since we suppress any output on the bots anyways the debug layers are serving no purpose. Thus I am adding a gyp define to disable the layers on the bot. With this change, by default when running vulkan in Debug, the debug_layers will be enabled. The bots should disable the layers. Android framework should also have them disabled by default. TBR=djsollen@google.com BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2185953003 Committed: https://skia.googlesource.com/skia/+/735109c24892a95290974c9665527b57b01e5440

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+252 lines, -397 lines) Patch
M gyp/common.gypi View 1 chunk +7 lines, -0 lines 0 comments Download
M gyp/common_variables.gypi View 3 chunks +3 lines, -0 lines 0 comments Download
M include/gpu/vk/GrVkBackendContext.h View 1 chunk +0 lines, -4 lines 0 comments Download
M infra/bots/recipe_modules/skia/fake_specs.py View 2 chunks +5 lines, -4 lines 0 comments Download
M infra/bots/recipes/swarm_compile.expected/Build-Win-MSVC-x86_64-Release-Vulkan.json View 2 chunks +2 lines, -2 lines 0 comments Download
M infra/bots/recipes/swarm_trigger.expected/Build-Win-MSVC-x86_64-Release-Vulkan.json View 1 chunk +1 line, -1 line 0 comments Download
M infra/bots/recipes/swarm_trigger.expected/Test-Android-GCC-NVIDIA_Shield-GPU-TegraX1-Arm64-Debug-Vulkan.json View 1 chunk +1 line, -1 line 0 comments Download
M src/gpu/vk/GrVkBackendContext.cpp View 3 chunks +3 lines, -3 lines 0 comments Download
M src/gpu/vk/GrVkGpu.h View 1 chunk +1 line, -1 line 0 comments Download
M src/gpu/vk/GrVkGpu.cpp View 3 chunks +3 lines, -3 lines 0 comments Download
M tools/buildbot_spec.json View 85 chunks +225 lines, -378 lines 0 comments Download
M tools/buildbot_spec.py View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 22 (13 generated)
egdaniel
Adding Jim for general change, Eric for bots, and Derek for android.
4 years, 4 months ago (2016-07-27 14:15:26 UTC) #6
egdaniel
Adding Jim for general change, Eric for bots, and Derek for android.
4 years, 4 months ago (2016-07-27 14:15:26 UTC) #7
djsollen
android looks good
4 years, 4 months ago (2016-07-27 14:24:25 UTC) #10
borenet
Bots stuff lgtm
4 years, 4 months ago (2016-07-27 14:36:36 UTC) #11
jvanverth1
lgtm
4 years, 4 months ago (2016-07-27 14:44:41 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2185953003/1
4 years, 4 months ago (2016-07-27 14:56:46 UTC) #15
commit-bot: I haz the power
Try jobs failed on following builders: skia_presubmit-Trybot on master.client.skia.fyi (JOB_FAILED, http://build.chromium.org/p/client.skia.fyi/builders/skia_presubmit-Trybot/builds/11769)
4 years, 4 months ago (2016-07-27 14:58:15 UTC) #17
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2185953003/1
4 years, 4 months ago (2016-07-27 14:59:55 UTC) #20
commit-bot: I haz the power
4 years, 4 months ago (2016-07-27 15:04:01 UTC) #22
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/735109c24892a95290974c9665527b57b01e5440

Powered by Google App Engine
This is Rietveld 408576698