Commit 00e84568 authored by Kramer Ge's avatar Kramer Ge Committed by Chromium LUCI CQ

Revert "[LaCrOS]Disable usage of linux_explicit_synchronization for now"

This reverts commit 500376ce.

Reason for revert: crbug.com/1144179 has been fixed, we no longer make consecutive subsurface commits with fences.

Original change's description:
> [LaCrOS]Disable usage of linux_explicit_synchronization for now
>
> linux_explicit_synchronization can cause crash due to a bug in earlier
> exo version. Underlying issue can be fixed from lacros side. Re-enable
> once fix is landed.
>
> Change-Id: I54c9b07d48d29e5ed091831483152e2e4e6e5b38
> Bug: 1151012
>
> TBR=rjkroege@chromium.org
>
> Change-Id: I54c9b07d48d29e5ed091831483152e2e4e6e5b38
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2552854
> Reviewed-by: Kramer Ge <fangzhoug@chromium.org>
> Reviewed-by: Robert Kroeger <rjkroege@chromium.org>
> Commit-Queue: Kramer Ge <fangzhoug@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#829874}

TBR=rjkroege@chromium.org,fangzhoug@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1151012
Change-Id: I932daf018ac1e48898221f849652b50d65f1b0eb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580280Reviewed-by: default avatarKramer Ge <fangzhoug@chromium.org>
Reviewed-by: default avatarRobert Kroeger <rjkroege@chromium.org>
Commit-Queue: Kramer Ge <fangzhoug@chromium.org>
Cr-Commit-Position: refs/heads/master@{#835845}
parent ea36a1c1
...@@ -790,9 +790,7 @@ bool WaylandBufferManagerHost::SupportsDmabuf() const { ...@@ -790,9 +790,7 @@ bool WaylandBufferManagerHost::SupportsDmabuf() const {
} }
bool WaylandBufferManagerHost::SupportsAcquireFence() const { bool WaylandBufferManagerHost::SupportsAcquireFence() const {
// TODO(fangzhoug@): Re-enable usage of linux_explicit_synchronization once return !!connection_->linux_explicit_synchronization_v1();
// crbug.com/1144179 is fixed.
return !!connection_->linux_explicit_synchronization_v1() && false;
} }
void WaylandBufferManagerHost::SetWaylandBufferManagerGpu( void WaylandBufferManagerHost::SetWaylandBufferManagerGpu(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment