ui/ozone: Flush CPU caches when mapping/unmapping prime pixmap

ui/ozone/platform/drm/common/client_native_pixmap_dmabuf.cc

 // Copyright 2016 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/ozone/platform/drm/common/client_native_pixmap_dmabuf.h"
 
+#include <errno.h>
 #include <fcntl.h>
+#include <linux/dma-buf.h>
 #include <stddef.h>
 #include <sys/mman.h>
 #include <xf86drm.h>
 
 #include "base/process/memory.h"
 #include "base/trace_event/trace_event.h"
 
 namespace ui {
 
+namespace {
+
+void PrimeSyncStart(int dmabuf_fd) {
+  struct dma_buf_sync sync_start = {0};
+
+  sync_start.flags = DMA_BUF_SYNC_START | DMA_BUF_SYNC_READ;
+  if (drmIoctl(dmabuf_fd, DMA_BUF_IOCTL_SYNC, &sync_start))
+    LOG(ERROR) << "fail to sync start dma buf fd, errno: " << errno;

[spang, 2016/03/30 00:19:00]

PLOG(ERROR)

[vignatti (out of this project), 2016/03/30 20:04:23]

Done.

+}
+
+void PrimeSyncEnd(int dmabuf_fd) {
+  struct dma_buf_sync sync_end = {0};
+
+  sync_end.flags = DMA_BUF_SYNC_END | DMA_BUF_SYNC_WRITE;
+  if (drmIoctl(dmabuf_fd, DMA_BUF_IOCTL_SYNC, &sync_end))
+    LOG(ERROR) << "fail to sync end dma buf fd, errno " << errno;

[spang, 2016/03/30 00:19:00]

PLOG(ERROR)

[vignatti (out of this project), 2016/03/30 20:04:23]

Done.

+}
+}

[spang, 2016/03/30 00:19:00]

Add a newline between the braces and say

}  // namespace

to close the anonymous namespace.

[vignatti (out of this project), 2016/03/30 20:04:23]

Done.

+
 // static
 scoped_ptr<ClientNativePixmap> ClientNativePixmapDmaBuf::ImportFromDmabuf(
     int dmabuf_fd,
     const gfx::Size& size,
     int stride) {
   DCHECK_GE(dmabuf_fd, 0);
   return make_scoped_ptr(new ClientNativePixmapDmaBuf(dmabuf_fd, size, stride));
 }
 
 ClientNativePixmapDmaBuf::ClientNativePixmapDmaBuf(int dmabuf_fd,
                                                    const gfx::Size& size,
                                                    int stride)
-    : size_(size), stride_(stride) {
+    : dmabuf_fd_(dmabuf_fd), size_(size), stride_(stride) {
   TRACE_EVENT0("drm", "ClientNativePixmapDmaBuf");
   size_t map_size = stride_ * size_.height();
   data_ = mmap(nullptr, map_size, (PROT_READ | PROT_WRITE), MAP_SHARED,
                dmabuf_fd, 0);
   CHECK_NE(data_, MAP_FAILED);
 }
 
 ClientNativePixmapDmaBuf::~ClientNativePixmapDmaBuf() {
   TRACE_EVENT0("drm", "~ClientNativePixmapDmaBuf");
+
   size_t size = stride_ * size_.height();
   int ret = munmap(data_, size);
   DCHECK(!ret);
 }
 
 void* ClientNativePixmapDmaBuf::Map() {
+  TRACE_EVENT0("nativepixmap", "DmaBuf:Map");

[spang, 2016/03/30 00:19:00]

Please, use "drm". This doesn't warrant its own tracing category.

[vignatti (out of this project), 2016/03/30 20:04:23]

Done.

+  PrimeSyncStart(dmabuf_fd_.get());
   return data_;
 }
 
-void ClientNativePixmapDmaBuf::Unmap() {}
+void ClientNativePixmapDmaBuf::Unmap() {
+  TRACE_EVENT0("nativepixmap", "DmaBuf:Unmap");
+  PrimeSyncEnd(dmabuf_fd_.get());
+}
 
 void ClientNativePixmapDmaBuf::GetStride(int* stride) const {
   *stride = stride_;
 }
 
 }  // namespace ui