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

Issue 2611933002: Adding Mali Workaround Patch for latest GPU (Closed)

Created:
3 years, 11 months ago by sujith
Modified:
3 years, 11 months ago
CC:
chromium-reviews, piman+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Adding Mali Workaround Patch for latest GPU Adding Mali Workaround patch for the latest Mali-G71 https://www.arm.com/products/graphics-and-multimedia/mali-gpu BUG=678508 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2611933002 Cr-Commit-Position: refs/heads/master@{#442061} Committed: https://chromium.googlesource.com/chromium/src/+/2e52433cf945c3c78c998416eddeda91b2514775

Patch Set 1 #

Total comments: 2

Patch Set 2 : Adding Mali Workaround Patch for latest GPU #

Total comments: 1

Patch Set 3 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -1 line) Patch
M gpu/config/gpu_driver_bug_list_json.cc View 1 2 2 chunks +11 lines, -1 line 0 comments Download

Messages

Total messages: 16 (7 generated)
sujith
PTAL As suggested in the patch https://codereview.chromium.org/2593013002/ Added a new entry for Mali-G.* series.
3 years, 11 months ago (2017-01-04 09:19:17 UTC) #3
Zhenyao Mo
https://codereview.chromium.org/2611933002/diff/1/gpu/config/gpu_driver_bug_list_json.cc File gpu/config/gpu_driver_bug_list_json.cc (right): https://codereview.chromium.org/2611933002/diff/1/gpu/config/gpu_driver_bug_list_json.cc#newcode22 gpu/config/gpu_driver_bug_list_json.cc:22: "version": "9.26", Please update this version to 9.27
3 years, 11 months ago (2017-01-04 18:07:45 UTC) #4
aelias_OOO_until_Jul13
https://codereview.chromium.org/2611933002/diff/1/gpu/config/gpu_driver_bug_list_json.cc File gpu/config/gpu_driver_bug_list_json.cc (right): https://codereview.chromium.org/2611933002/diff/1/gpu/config/gpu_driver_bug_list_json.cc#newcode2307 gpu/config/gpu_driver_bug_list_json.cc:2307: "cr_bugs": [154715, 10068, 269829, 294779, 285292], Please file a ...
3 years, 11 months ago (2017-01-04 20:18:52 UTC) #5
sujith
PTAL
3 years, 11 months ago (2017-01-05 06:47:00 UTC) #7
Zhenyao Mo
https://codereview.chromium.org/2611933002/diff/20001/gpu/config/gpu_driver_bug_list_json.cc File gpu/config/gpu_driver_bug_list_json.cc (right): https://codereview.chromium.org/2611933002/diff/20001/gpu/config/gpu_driver_bug_list_json.cc#newcode22 gpu/config/gpu_driver_bug_list_json.cc:22: "version": "9.27", I think you need to rebase and ...
3 years, 11 months ago (2017-01-05 18:26:36 UTC) #8
aelias_OOO_until_Jul13
lgtm
3 years, 11 months ago (2017-01-06 01:49:50 UTC) #9
sujith
rebased the patch. PTAL
3 years, 11 months ago (2017-01-06 10:41:30 UTC) #10
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/2611933002/40001
3 years, 11 months ago (2017-01-06 20:18:45 UTC) #13
commit-bot: I haz the power
3 years, 11 months ago (2017-01-06 21:41:41 UTC) #16
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/2e52433cf945c3c78c998416edde...

Powered by Google App Engine
This is Rietveld 408576698