ARM graphics_WebGLAquarium failure starting with 9636.0.0..9637.0.0 |
|||||||
Issue descriptionLooks separate from issue 729160 . Ranges for new regression https://crosland.corp.google.com/log/9636.0.0..9637.0.0 https://chromium.googlesource.com/chromium/src/+log/61.0.3125.0..61.0.3126.0?pretty=fuller&n=10000
,
Jun 13 2017
,
Jun 13 2017
Robert is bisecting Chrome right now.
,
Jun 13 2017
commit 10b29d38bbd2eb90d7d93fb07214d468f7af7dde by Kai Ninomiya <kainino@chromium.org> breaks webgl on arm devices: "Workaround for Qualcomm renderbuffer resize bug Also re-enables the explicit resolve path on the Qualcomm devices which have EXT_multisampled_render_to_texture disabled." Will try to extract this commit from the top of the tree.
,
Jun 14 2017
,
Jun 14 2017
Not sure why this is P3, this is P1 since we can't ship with that.
,
Jun 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a2205e65faf366cf1748e6a72c4a41188b07e05b commit a2205e65faf366cf1748e6a72c4a41188b07e05b Author: marcheu <marcheu@google.com> Date: Wed Jun 14 05:03:30 2017 Revert "Workaround for Qualcomm renderbuffer resize bug" This reverts commit 10b29d38bbd2eb90d7d93fb07214d468f7af7dde. This breaks WebGL on ARM Chromebooks. BUG= chromium:732616 TEST=webgl works again on Chrome OS CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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/2936973003 Cr-Commit-Position: refs/heads/master@{#479289} [modify] https://crrev.com/a2205e65faf366cf1748e6a72c4a41188b07e05b/gpu/command_buffer/service/gles2_cmd_decoder.cc [modify] https://crrev.com/a2205e65faf366cf1748e6a72c4a41188b07e05b/gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.cc [modify] https://crrev.com/a2205e65faf366cf1748e6a72c4a41188b07e05b/gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.h [modify] https://crrev.com/a2205e65faf366cf1748e6a72c4a41188b07e05b/gpu/command_buffer/service/gles2_cmd_decoder_unittest_framebuffers.cc [modify] https://crrev.com/a2205e65faf366cf1748e6a72c4a41188b07e05b/gpu/command_buffer/service/renderbuffer_manager.cc [modify] https://crrev.com/a2205e65faf366cf1748e6a72c4a41188b07e05b/gpu/command_buffer/service/renderbuffer_manager.h [modify] https://crrev.com/a2205e65faf366cf1748e6a72c4a41188b07e05b/gpu/config/gpu_driver_bug_list.json [modify] https://crrev.com/a2205e65faf366cf1748e6a72c4a41188b07e05b/gpu/config/gpu_driver_bug_workaround_type.h
,
Jun 16 2017
,
Jun 16 2017
It's unfortunate that this was reverted. Could someone from the Chrome OS team work with me to debug why that driver bug workaround broke things on ARM Chromebooks? Certainly the workaround itself was not supposed to take effect on ARM GPUs.
,
Jun 19 2017
We debugged this together and the breaking commit has been reverted and the adjustment to the original code is understood.
,
Jun 29 2017
Verified. https://wmatrix.googleplex.com/unfiltered?hide_missing=True&releases=tot&tests=graphics_WebGLAquarium |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by gurcheta...@chromium.org
, Jun 13 2017