exo: Implement zcr_linux_explicit_synchronization_v1

components/exo/wayland/server.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/exo/wayland/server.h"
 
 #include <grp.h>
 #include <linux/input.h>
 #include <stddef.h>
 #include <stdint.h>
 #include <viewporter-server-protocol.h>
 #include <wayland-server-core.h>
 #include <wayland-server-protocol-core.h>
 
 // Note: core wayland headers need to be included before protocol headers.
 #include <alpha-compositing-unstable-v1-server-protocol.h>  // NOLINT
+#include <explicit-fencing-unstable-v1-server-protocol.h>   // NOLINT
 #include <gaming-input-unstable-v1-server-protocol.h>       // NOLINT
 #include <remote-shell-unstable-v1-server-protocol.h>       // NOLINT
 #include <secure-output-unstable-v1-server-protocol.h>      // NOLINT
 #include <stylus-unstable-v1-server-protocol.h>             // NOLINT
 #include <xdg-shell-unstable-v5-server-protocol.h>          // NOLINT
 #include <vsync-feedback-unstable-v1-server-protocol.h>     // NOLINT
 
 #include <algorithm>
 #include <cstdlib>
 #include <iterator>
@@ skipping 26 matching lines @@
 #include "components/exo/pointer_delegate.h"
 #include "components/exo/pointer_stylus_delegate.h"
 #include "components/exo/shared_memory.h"
 #include "components/exo/shell_surface.h"
 #include "components/exo/sub_surface.h"
 #include "components/exo/surface.h"
 #include "components/exo/surface_property.h"
 #include "components/exo/touch.h"
 #include "components/exo/touch_delegate.h"
 #include "components/exo/wm_helper.h"
+#include "gpu/command_buffer/client/gpu_memory_buffer_manager.h"
 #include "ipc/unix_domain_socket_util.h"
 #include "third_party/skia/include/core/SkRegion.h"
+#include "ui/aura/env.h"
 #include "ui/aura/window_property.h"
 #include "ui/base/hit_test.h"
 #include "ui/compositor/compositor_vsync_manager.h"
 #include "ui/display/display_observer.h"
 #include "ui/display/manager/managed_display_info.h"
 #include "ui/display/screen.h"
 #include "ui/events/keycodes/dom/keycode_converter.h"
 #include "ui/gfx/buffer_format_util.h"
 #include "ui/gfx/buffer_types.h"
 #include "ui/views/widget/widget.h"
@@ skipping 15 matching lines @@
 
 namespace exo {
 namespace wayland {
 namespace {
 
 // We don't send configure immediately after tablet mode switch
 // because layout can change due to orientation lock state or accelerometer.
 const int kConfigureDelayAfterLayoutSwitchMs = 300;
 
 // Default wayland socket name.
-const base::FilePath::CharType kSocketName[] = FILE_PATH_LITERAL("wayland-0");
+const base::FilePath::CharType kSocketName[] = FILE_PATH_LITERAL("exosphere-0");

[reveman, 2016/10/09 19:03:17]

We'll have to do this in a separate CL. Maybe try wayland-0 first and move to
wayland-1, etc. if already in use?

[fooishbar, 2016/10/12 15:41:56]

Yeah, this really shouldn't be in here; was a local change which slipped in!

 
 // Group used for wayland socket.
 const char kWaylandSocketGroup[] = "wayland";
 
 template <class T>
 T* GetUserDataAs(wl_resource* resource) {
   return static_cast<T*>(wl_resource_get_user_data(resource));
 }
 
 template <class T>
@@ skipping 2848 matching lines @@
 const struct zcr_stylus_v1_interface stylus_implementation = {
     stylus_get_pointer_stylus};
 
 void bind_stylus(wl_client* client, void* data, uint32_t version, uint32_t id) {
   wl_resource* resource =
       wl_resource_create(client, &zcr_stylus_v1_interface, version, id);
   wl_resource_set_implementation(resource, &stylus_implementation, data,
                                  nullptr);
 }
 
+////////////////////////////////////////////////////////////////////////////////
+// per-surface explict-fencing interface:

[reveman, 2016/10/09 19:03:17]

nit: for consistency "INTERFACENAME interface:"

[fooishbar, 2016/10/12 15:41:56]

Done.

+
+void fenced_surface_destroy(wl_client* client, wl_resource* resource) {
+  wl_resource_destroy(resource);
+}
+
+void fenced_surface_set_acquire_fence(wl_client* client,
+                                      wl_resource* resource,
+                                      int fence_fd)
+{
+  Surface* surface = GetUserDataAs<Surface>(resource);
+  gfx::GpuFenceHandle handle;
+  std::unique_ptr<gfx::GpuFence> fence;
+
+  handle.fd.fd = fence_fd;
+
+  fence = aura::Env::GetInstance()

[reveman, 2016/10/09 19:03:17]

How about we move this to exo::Display by adding a Display::CreateLinuxFence in
a way that's consistent with CreateLinuxDMABufBuffer? A basic
DisplatTest.CreateLinuxFence unittest added to display_unittest.cc would be nice
too.

[fooishbar, 2016/10/12 15:41:56]

Done.

+            ->context_factory()
+            ->GetGpuMemoryBufferManager()
+            ->CreateGpuFenceFromHandle(handle);
+
+  if (!fence) {
+    DLOG(WARNING) << "failed to create GpuFence!";
+    return;
+  }
+
+  surface->FenceAcquire(std::move(fence));
+}
+
+const struct zcr_explicit_fenced_surface_v1_interface
+    fenced_surface_implementation = {
+    fenced_surface_destroy,
+    fenced_surface_set_acquire_fence};
+
+
+////////////////////////////////////////////////////////////////////////////////
+// explict-fencing interface:
+
+void fencing_destroy(wl_client* client, wl_resource* resource) {
+  wl_resource_destroy(resource);
+}
+
+void fencing_get_surface(wl_client* client, wl_resource* resource, uint32_t id,

[reveman, 2016/10/09 19:03:17]

Can we make this consistent with the alpha_compositing interface and have it
fail if a fencing interface already exists for this surface?

[fooishbar, 2016/10/12 15:41:56]

Yep, but let me do that in v3 so I can sort the rest first.

[fooishbar, 2016/10/12 15:45:19]

Done.

+                         wl_resource* surface_resource) {
+  Surface* surface = GetUserDataAs<Surface>(surface_resource);
+  wl_resource* fenced_surface_resource =
+      wl_resource_create(client, &zcr_explicit_fenced_surface_v1_interface, 1,
+                         id);
+
+  wl_resource_set_implementation(fenced_surface_resource,
+                                 &fenced_surface_implementation,
+                                 surface,
+                                 nullptr);
+}
+
+const struct zcr_linux_explicit_fencing_v1_interface
+  fencing_implementation = {
+    fencing_destroy,
+    fencing_get_surface};
+
+void bind_fencing(wl_client* client, void* data, uint32_t version,
+    uint32_t id) {
+  wl_resource* resource =
+      wl_resource_create(client, &zcr_linux_explicit_fencing_v1_interface, 1,
+                         id);
+  wl_resource_set_implementation(resource, &fencing_implementation, data,
+                                 nullptr);
+}
+
 }  // namespace
 
 ////////////////////////////////////////////////////////////////////////////////
 // Server, public:
 
 Server::Server(Display* display)
     : display_(display), wl_display_(wl_display_create()) {
   wl_global_create(wl_display_.get(), &wl_compositor_interface,
                    compositor_version, display_, bind_compositor);
   wl_global_create(wl_display_.get(), &wl_shm_interface, 1, display_, bind_shm);
@@ skipping 22 matching lines @@
   wl_global_create(wl_display_.get(), &zcr_secure_output_v1_interface, 1,
                    display_, bind_secure_output);
   wl_global_create(wl_display_.get(), &zcr_alpha_compositing_v1_interface, 1,
                    display_, bind_alpha_compositing);
   wl_global_create(wl_display_.get(), &zcr_remote_shell_v1_interface,
                    remote_shell_version, display_, bind_remote_shell);
   wl_global_create(wl_display_.get(), &zcr_gaming_input_v1_interface, 1,
                    display_, bind_gaming_input);
   wl_global_create(wl_display_.get(), &zcr_stylus_v1_interface, 1, display_,
                    bind_stylus);
+  wl_global_create(wl_display_.get(),

[reveman, 2016/10/09 19:03:17]

We currently don't make the assumption that all this exo code is only going to
be used on linux even though that might be the only platform we build it on
today. Can you move this up into the USE_OZONE ifdef and protect all the code
added in the same way? Consistent with wl_drm and linux dmabuf interface.

[fooishbar, 2016/10/12 15:41:56]

Right, makes sense. Remembered that I'd only done that so I could test it on a
generic target_os='chromeos' build, not targeted at actual CrOS.

+                   &zcr_linux_explicit_fencing_v1_interface, 1, display_,
+                   bind_fencing);
 }
 
 Server::~Server() {}
 
 // static
 std::unique_ptr<Server> Server::Create(Display* display) {
   std::unique_ptr<Server> server(new Server(display));
 
   char* runtime_dir = getenv("XDG_RUNTIME_DIR");
   if (!runtime_dir) {
@@ skipping 51 matching lines @@
   DCHECK(event_loop);
   wl_event_loop_dispatch(event_loop, timeout.InMilliseconds());
 }
 
 void Server::Flush() {
   wl_display_flush_clients(wl_display_.get());
 }
 
 }  // namespace wayland
 }  // namespace exo