gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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

[reveman, 2017-05-04 21:39:23]

Description was changed from

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
==========

to

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
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
==========

[reveman, 2017-05-04 21:39:23]

Description was changed from

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
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
==========

to

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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
==========

[reveman, 2017-05-04 21:59:18]

reveman@chromium.org changed reviewers:
+ dcastagna@chromium.org

[reveman, 2017-05-22 20:00:14]

please take a look

[Daniele Castagna, 2017-05-22 20:04:08]

LGTM.

https://codereview.chromium.org/2864483007/diff/20001/components/exo/wayland/...
File components/exo/wayland/clients/client_base.cc (right):

https://codereview.chromium.org/2864483007/diff/20001/components/exo/wayland/...
components/exo/wayland/clients/client_base.cc:294: egl_sync_type_ =
EGL_SYNC_FENCE_KHR;
Should we DCHECK this is available?

[reveman, 2017-05-22 20:27:18]

reveman@chromium.org changed reviewers:
+ piman@chromium.org

[reveman, 2017-05-22 20:27:19]

+piman

fyi, I'll wait to land this until the required cros driver side changes that are
in the cq have landed.

https://codereview.chromium.org/2864483007/diff/20001/components/exo/wayland/...
File components/exo/wayland/clients/client_base.cc (right):

https://codereview.chromium.org/2864483007/diff/20001/components/exo/wayland/...
components/exo/wayland/clients/client_base.cc:294: egl_sync_type_ =
EGL_SYNC_FENCE_KHR;
On 2017/05/22 at 20:04:08, Daniele Castagna wrote:
> Should we DCHECK this is available?

Added a EGL_KHR_fence_sync DCHECK here. We should consider doing the same for
gbm_surfaceless in a separate patch if possible.

[reveman, 2017-05-22 20:29:20]

reveman@chromium.org changed reviewers:
+ dnicoara@chromium.org

[reveman, 2017-05-22 20:29:21]

+dnicoara for ui/ozone/platform/drm/gpu

[piman, 2017-05-22 20:30:54]

lgtm

[Julien Isorce, 2017-05-23 13:17:27]

https://codereview.chromium.org/2864483007/diff/40001/gpu/config/gpu_driver_b...
File gpu/config/gpu_driver_bug_list.json (left):

https://codereview.chromium.org/2864483007/diff/40001/gpu/config/gpu_driver_b...
gpu/config/gpu_driver_bug_list.json:846: },
Just in case, don't forget to bump "version" with "M.m+1" on top of the file
before landing it.

[dnicoara, 2017-05-23 14:41:22]

lgtm

[reveman, 2017-05-23 20:26:12]

https://codereview.chromium.org/2864483007/diff/40001/gpu/config/gpu_driver_b...
File gpu/config/gpu_driver_bug_list.json (left):

https://codereview.chromium.org/2864483007/diff/40001/gpu/config/gpu_driver_b...
gpu/config/gpu_driver_bug_list.json:846: },
On 2017/05/23 at 13:17:26, Julien Isorce wrote:
> Just in case, don't forget to bump "version" with "M.m+1" on top of the file
before landing it.

Thanks for the reminder! Done.

[reveman, 2017-05-27 17:14:29]

The CQ bit was checked by reveman@chromium.org

[reveman, 2017-05-27 17:14:30]

The patchset sent to the CQ was uploaded after l-g-t-m from
dcastagna@chromium.org, dnicoara@chromium.org, piman@chromium.org
Link to the patchset: https://codereview.chromium.org/2864483007/#ps60001
(title: "bump bug list version")

[commit-bot: I haz the power, 2017-05-27 17:14:39]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-27 17:20:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-27 17:20:28]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[reveman, 2017-05-27 20:21:25]

Description was changed from

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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
==========

to

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
TBR=dcheng@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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
==========

[reveman, 2017-05-27 20:21:44]

reveman@chromium.org changed reviewers:
+ dcheng@chromium.org

[reveman, 2017-05-27 20:21:44]

The CQ bit was checked by reveman@chromium.org

[reveman, 2017-05-27 20:21:45]

+dcheng, TBR for removal of 1 line in
gpu/ipc/common/gpu_command_buffer_traits_multi.h

[reveman, 2017-05-27 20:21:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
dnicoara@chromium.org, dcastagna@chromium.org, piman@chromium.org
Link to the patchset: https://codereview.chromium.org/2864483007/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2017-05-27 20:21:57]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-27 21:23:29]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1495916504806430,
"parent_rev": "d71bc7b31de04a31db7c360da95c5d1ff53b6784", "commit_rev":
"d2d8cbbe67801309f69be3c5c941efccb6fcbf6c"}

[commit-bot: I haz the power, 2017-05-27 21:23:41]

Description was changed from

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
TBR=dcheng@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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
==========

to

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
TBR=dcheng@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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/2864483007
Cr-Commit-Position: refs/heads/master@{#475250}
Committed:
https://chromium.googlesource.com/chromium/src/+/d2d8cbbe67801309f69be3c5c941...
==========

[commit-bot: I haz the power, 2017-05-27 21:23:43]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/d2d8cbbe67801309f69be3c5c941...

[dcheng, 2017-05-30 03:24:36]

LGTM

[Alexandre Courbot, 2017-05-30 05:54:43]

A revert of this CL (patchset #5 id:80001) has been created in
https://codereview.chromium.org/2908313002/ by acourbot@chromium.org.

The reason for reverting is: Breaks UI on peach_pit

http://crbug.com/727462.

[reveman, 2017-06-09 20:41:16]

Description was changed from

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
TBR=dcheng@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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/2864483007
Cr-Commit-Position: refs/heads/master@{#475250}
Committed:
https://chromium.googlesource.com/chromium/src/+/d2d8cbbe67801309f69be3c5c941...
==========

to

==========
gpu: Use ANDROID_native_fence_sync instead of ARM_implicit_external_sync.

This allows partial swaps to be enabled on Mali devices.

BUG=705290
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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
==========