[Ozone] Adding a software rendering implementation of SurfaceFactoryOzone

ui/gfx/ozone/impl/software_surface_factory_ozone.cc

+// Copyright (c) 2013 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 "ui/gfx/ozone/impl/software_surface_factory_ozone.h"
+
+#include <drm.h>
+#include <errno.h>
+#include <xf86drm.h>
+
+#include "base/message_loop/message_loop.h"
+#include "third_party/skia/include/core/SkBitmap.h"
+#include "third_party/skia/include/core/SkDevice.h"
+#include "ui/gfx/native_widget_types.h"
+#include "ui/gfx/ozone/impl/drm_skbitmap_ozone.h"
+#include "ui/gfx/ozone/impl/drm_wrapper_ozone.h"
+#include "ui/gfx/ozone/impl/hardware_display_controller_ozone.h"
+#include "ui/gfx/ozone/impl/software_surface_ozone.h"
+
+namespace gfx {
+
+namespace {
+
+const char kDefaultGraphicsCardPath[] = "/dev/dri/card0";
+
+const gfx::AcceleratedWidget kDefaultWidgetHandle = 1;
+
+// DRM callback on page flip events. This callback is triggered after the
+// page flip has happened and the backbuffer is now the new frontbuffer
+// The old frontbuffer is no longer used by the hardware and can be used for
+// future draw operations.
+//
+// |device| will contain a reference to the |SoftwareSurfaceOzone| object which
+// the event belongs to.
+//
+// TODO(dnicoara) When we have a FD handler for the DRM calls in the message
+// loop, we can move this function in the handler.
+void HandlePageFlipEvent(int fd,
+                         unsigned int frame,
+                         unsigned int seconds,
+                         unsigned int useconds,
+                         void* controller) {
+  static_cast<HardwareDisplayControllerOzone*>(controller)->get_surface()
+      ->SwapBuffers();
+}
+
+uint32_t GetCrtc(int fd, drmModeRes* resources, drmModeConnector* connector) {
+  // If the connector already has an encoder try to re-use.
+  if (connector->encoder_id) {
+    drmModeEncoder* encoder = drmModeGetEncoder(fd, connector->encoder_id);
+    if (encoder) {
+      if (encoder->crtc_id) {
+        uint32_t crtc = encoder->crtc_id;
+        drmModeFreeEncoder(encoder);
+        return crtc;
+      }
+      drmModeFreeEncoder(encoder);
+    }
+  }
+
+  // Try to find an encoder for the connector.
+  for (int i = 0; i < connector->count_encoders; ++i) {
+    drmModeEncoder* encoder = drmModeGetEncoder(fd, connector->encoders[i]);
+    if (!encoder)
+      continue;
+
+    for (int j = 0; j < resources->count_crtcs; ++j) {
+      // Check if the encoder is compatible with this CRTC
+      if (!(encoder->possible_crtcs & (1 << j)))
+        continue;
+
+      drmModeFreeEncoder(encoder);
+      return resources->crtcs[j];
+    }
+  }
+
+  return 0;
+}
+
+}  // namespace
+
+SoftwareSurfaceFactoryOzone::SoftwareSurfaceFactoryOzone()
+    : drm_(),
+      state_(UNINITIALIZED),
+      controller_() {
+}
+
+SoftwareSurfaceFactoryOzone::~SoftwareSurfaceFactoryOzone() {
+  if (state_ == INITIALIZED)
+    ShutdownHardware();
+}
+
+SurfaceFactoryOzone::HardwareState
+SoftwareSurfaceFactoryOzone::InitializeHardware() {
+  CHECK(state_ == UNINITIALIZED);
+
+  // TODO(dnicoara): Short-cut right now. What we want is to look at all the
+  // graphics devices available and select the primary one.
+  drm_.reset(CreateWrapper());
+  if (drm_->get_fd() < 0) {
+    LOG(ERROR) << "Cannot open graphics card '"
+               << kDefaultGraphicsCardPath << "': " << strerror(errno);
+    state_ = FAILED;
+    return state_;
+  }
+
+  state_ = INITIALIZED;
+  return state_;
+}
+
+void SoftwareSurfaceFactoryOzone::ShutdownHardware() {
+  CHECK(state_ == INITIALIZED);
+
+  controller_.reset();
+  drm_.reset();
+
+  state_ = UNINITIALIZED;
+}
+
+gfx::AcceleratedWidget SoftwareSurfaceFactoryOzone::GetAcceleratedWidget() {
+  CHECK(state_ != FAILED);
+
+  // TODO(dnicoara) When there's more information on which display we want,
+  // then we can return the widget associated with the display.
+  // For now just assume we have 1 display device and return it.
+  if (!controller_.get())
+    controller_.reset(new HardwareDisplayControllerOzone());
+
+  // TODO(dnicoara) We only have 1 display for now, so only 1 AcceleratedWidget.
+  // When we'll support multiple displays this needs to be changed to return a
+  // different handle for every display.
+  return kDefaultWidgetHandle;
+}
+
+gfx::AcceleratedWidget SoftwareSurfaceFactoryOzone::RealizeAcceleratedWidget(
+    gfx::AcceleratedWidget w) {
+  CHECK(state_ == INITIALIZED);
+  // TODO(dnicoara) Once we can handle multiple displays this needs to be
+  // changed.
+  CHECK(w == kDefaultWidgetHandle);
+
+  CHECK(controller_->get_state() ==
+        HardwareDisplayControllerOzone::UNASSOCIATED);
+
+  // Until now the controller is just a stub. Initializing it will link it to a
+  // hardware display.
+  if (!InitializeControllerForPrimaryDisplay(drm_.get(), controller_.get())) {
+    LOG(ERROR) << "Failed to initialize controller";
+    return gfx::kNullAcceleratedWidget;
+  }
+
+  // Create a surface suitable for the current controller.
+  SoftwareSurfaceOzone* surface = CreateSurface(controller_.get());

[rjkroege, 2013/10/16 19:03:36]

who tracks ownership of surface? it would seem to leak here

[dnicoara, 2013/10/16 20:27:29]

You are right, there is one case where it would leak. I also added comment to
indicate that the controller takes ownership of the surface.

+
+  if (!surface->Initialize()) {
+    LOG(ERROR) << "Failed to initialize surface";
+    return gfx::kNullAcceleratedWidget;
+  }
+
+  // Bind the surface to the controller. This will register the backing buffers
+  // with the hardware CRTC such that we can show the buffers.
+  if (!controller_->BindSurfaceToController(surface)) {
+    delete surface;
+    LOG(ERROR) << "Failed to bind surface to controller";
+    return gfx::kNullAcceleratedWidget;
+  }
+
+  return reinterpret_cast<gfx::AcceleratedWidget>(
+      surface->GetDrawableForWidget());
+}
+
+bool SoftwareSurfaceFactoryOzone::LoadEGLGLES2Bindings() {
+  return false;
+}
+
+bool SoftwareSurfaceFactoryOzone::AttemptToResizeAcceleratedWidget(
+    gfx::AcceleratedWidget w,
+    const gfx::Rect& bounds) {
+  return false;
+}
+
+bool SoftwareSurfaceFactoryOzone::SchedulePageFlip(gfx::AcceleratedWidget w) {
+  CHECK(state_ == INITIALIZED);
+  // TODO(dnicoara) Change this CHECK once we're running with the threaded
+  // compositor.
+  CHECK(base::MessageLoop::current()->type() == base::MessageLoop::TYPE_UI);
+
+  // TODO(dnicoara) Once we can handle multiple displays this needs to be
+  // changed.
+  CHECK(w == kDefaultWidgetHandle);
+
+  if (!controller_->SchedulePageFlip())

[rjkroege, 2013/10/16 19:03:36]

The comments imply that we must always return false because otherwise, we have
called WaitForPageFlipEvent from the UI thread which you say is forbidden. Can
you clarify?

[dnicoara, 2013/10/16 20:27:29]

I have updated the comments. Hopefully they are clearer now.

+    return false;
+
+  // TODO(dnicoara) This requires the threaded compositor.
+  // This should happen in the message loop such that we don't
+  // block. However we should be able to signal the drawing thread when the page
+  // flip finished such that it can start a new frame. This would typically be
+  // done using the VSyncProvider, however at this point the
+  // SoftwareOutputDevice doesn't provide a way to use a custom VSyncProvider to
+  // update its timer or state.
+  WaitForPageFlipEvent(drm_->get_fd());
+
+  return true;
+}
+
+gfx::VSyncProvider* SoftwareSurfaceFactoryOzone::GetVSyncProvider(
+    gfx::AcceleratedWidget w) {
+  return NULL;
+}
+
+////////////////////////////////////////////////////////////////////////////////
+// SoftwareSurfaceFactoryOzone private
+
+SoftwareSurfaceOzone* SoftwareSurfaceFactoryOzone::CreateSurface(
+    HardwareDisplayControllerOzone* controller) {
+  return new SoftwareSurfaceOzone(controller);
+}
+
+DrmWrapperOzone* SoftwareSurfaceFactoryOzone::CreateWrapper() {
+  return new DrmWrapperOzone(kDefaultGraphicsCardPath);
+}
+
+bool SoftwareSurfaceFactoryOzone::InitializeControllerForPrimaryDisplay(
+    DrmWrapperOzone* drm,
+    HardwareDisplayControllerOzone* controller) {
+  CHECK(state_ == SurfaceFactoryOzone::INITIALIZED);
+
+  drmModeRes* resources = drmModeGetResources(drm->get_fd());
+
+  // Search for an active connector.
+  for (int i = 0; i < resources->count_connectors; ++i) {
+    drmModeConnector* connector = drmModeGetConnector(
+        drm->get_fd(),
+        resources->connectors[i]);
+
+    if (!connector)
+      continue;
+
+    if (connector->connection != DRM_MODE_CONNECTED ||
+        connector->count_modes == 0) {
+      drmModeFreeConnector(connector);
+      continue;
+    }
+
+    uint32_t crtc = GetCrtc(drm->get_fd(), resources, connector);
+
+    if (!crtc)
+      continue;
+
+    // TODO(dnicoara) Select one mode for now. In the future we may need to
+    // save all the modes and allow the user to choose a specific mode. Or
+    // even some fullscreen applications may need to change the mode.
+    controller->SetControllerInfo(
+        drm,
+        connector->connector_id,
+        crtc,
+        connector->modes[0]);
+
+    drmModeFreeConnector(connector);
+
+    return true;
+  }
+
+  return false;
+}
+
+void SoftwareSurfaceFactoryOzone::WaitForPageFlipEvent(int fd) {
+  drmEventContext drm_event;
+  drm_event.version = DRM_EVENT_CONTEXT_VERSION;
+  drm_event.page_flip_handler = HandlePageFlipEvent;
+  drm_event.vblank_handler = NULL;
+
+  // Wait for the page-flip to complete.
+  drmHandleEvent(fd, &drm_event);
+}
+
+}  // namespace gfx