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

Issue 2267163003: Re-enable program memory cache for Adreno 4/5xx (Closed)

Created:
4 years, 4 months ago by ericrk
Modified:
4 years, 3 months ago
Reviewers:
no sievers
CC:
chromium-reviews, piman+watch_chromium.org, cc-bugs_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Re-enable program memory cache for Adreno 4/5xx The program cache was disabled on Adreno 4/5xx due to crashes. I'm hoping that this issue was caused by invalidations/issues with the on-disk program cache, possibly with driver updates or other issues. To test this, I'm re-enabling the in-memory portion of the cache only. The cached objects will not outlive the GPU process. I've also confirmed that the related Adreno 3xx bug, crbug.com/510673, does not impact the Adreno 400 (and I'm guessing 500, but will track down a device to double check). We should watch crash rates for gfx::GLApiBase::glProgramBinaryFn to make sure this change doesn't re-introduce these crashes. BUG=598060, 486117 CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel Committed: https://crrev.com/30bcb92ee59a6778c4025081d7da82d2af8cccc2 Cr-Commit-Position: refs/heads/master@{#414490}

Patch Set 1 #

Total comments: 2

Patch Set 2 : bump the version #

Patch Set 3 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -7 lines) Patch
M gpu/command_buffer/service/in_process_command_buffer.cc View 1 chunk +5 lines, -3 lines 0 comments Download
M gpu/config/gpu_driver_bug_list_json.cc View 1 2 2 chunks +2 lines, -2 lines 0 comments Download
M gpu/config/gpu_driver_bug_workaround_type.h View 1 2 1 chunk +2 lines, -0 lines 0 comments Download
M gpu/ipc/service/gpu_channel_manager.cc View 1 2 1 chunk +4 lines, -2 lines 0 comments Download

Messages

Total messages: 19 (11 generated)
ericrk
4 years, 4 months ago (2016-08-23 00:20:34 UTC) #6
no sievers
lgtm https://codereview.chromium.org/2267163003/diff/40001/gpu/config/gpu_driver_bug_list_json.cc File gpu/config/gpu_driver_bug_list_json.cc (right): https://codereview.chromium.org/2267163003/diff/40001/gpu/config/gpu_driver_bug_list_json.cc#newcode22 gpu/config/gpu_driver_bug_list_json.cc:22: "version": "8.83", bump the version
4 years, 4 months ago (2016-08-23 21:28:19 UTC) #7
ericrk
https://codereview.chromium.org/2267163003/diff/40001/gpu/config/gpu_driver_bug_list_json.cc File gpu/config/gpu_driver_bug_list_json.cc (right): https://codereview.chromium.org/2267163003/diff/40001/gpu/config/gpu_driver_bug_list_json.cc#newcode22 gpu/config/gpu_driver_bug_list_json.cc:22: "version": "8.83", On 2016/08/23 21:28:18, sievers wrote: > bump ...
4 years, 3 months ago (2016-08-25 00:31:26 UTC) #8
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/2267163003/60001
4 years, 3 months ago (2016-08-25 00:33:24 UTC) #11
commit-bot: I haz the power
Try jobs failed on following builders: ios-device on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds/58011) ios-simulator on master.tryserver.chromium.mac (JOB_FAILED, ...
4 years, 3 months ago (2016-08-25 00:37:33 UTC) #13
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/2267163003/80001
4 years, 3 months ago (2016-08-25 17:43:40 UTC) #16
commit-bot: I haz the power
Committed patchset #3 (id:80001)
4 years, 3 months ago (2016-08-25 18:47:53 UTC) #17
commit-bot: I haz the power
4 years, 3 months ago (2016-08-25 18:49:59 UTC) #19
Message was sent while issue was closed.
Patchset 3 (id:??) landed as
https://crrev.com/30bcb92ee59a6778c4025081d7da82d2af8cccc2
Cr-Commit-Position: refs/heads/master@{#414490}

Powered by Google App Engine
This is Rietveld 408576698