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

content/common/sandbox_linux/bpf_renderer_policy_linux.cc

 // Copyright 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 "content/common/sandbox_linux/bpf_renderer_policy_linux.h"
 
 #include <errno.h>
 #include <sys/ioctl.h>
 
 #include "build/build_config.h"
 #include "content/common/sandbox_linux/sandbox_linux.h"
 #include "sandbox/linux/bpf_dsl/bpf_dsl.h"
 #include "sandbox/linux/seccomp-bpf-helpers/sigsys_handlers.h"
 #include "sandbox/linux/seccomp-bpf-helpers/syscall_parameters_restrictions.h"
 #include "sandbox/linux/seccomp-bpf-helpers/syscall_sets.h"
 #include "sandbox/linux/system_headers/linux_syscalls.h"
 
+#if defined(OS_CHROMEOS)
+// TODO(vignatti): replace the local definitions below with #include
+// <linux/dma-buf.h> once kernel version 4.6 becomes widely used.
+#include <linux/types.h>
+
+struct local_dma_buf_sync {
+  __u64 flags;
+};
+#define LOCAL_DMA_BUF_BASE 'b'

[rickyz (no longer on Chrome), 2016/04/06 02:04:48]

Not sure how problematic this is likely to be with ioctl req collisions, but it
looks like binder also uses 'b' :-/

[vignatti (out of this project), 2016/04/06 13:52:46]

https://github.com/torvalds/linux/blob/master/Documentation/ioctl/ioctl-numbe...

yup, we discussed this in dri-devel as well and seems alright, as noted in that
txt file: "Because of the large number of drivers, many drivers share a partial
letter with other drivers".

+#define LOCAL_DMA_BUF_IOCTL_SYNC \
+  _IOW(LOCAL_DMA_BUF_BASE, 0, struct local_dma_buf_sync)
+#endif
+
 using sandbox::SyscallSets;
 using sandbox::bpf_dsl::Allow;
 using sandbox::bpf_dsl::Arg;
 using sandbox::bpf_dsl::Error;
 using sandbox::bpf_dsl::ResultExpr;
 
 namespace content {
 
 namespace {
 
 ResultExpr RestrictIoctl() {
   const Arg<unsigned long> request(1);
   return Switch(request)
       .SANDBOX_BPF_DSL_CASES((static_cast<unsigned long>(TCGETS), FIONREAD),
                              Allow())
+#if defined(OS_CHROMEOS)
+      .SANDBOX_BPF_DSL_CASES(
+          (static_cast<unsigned long>(LOCAL_DMA_BUF_IOCTL_SYNC)), Allow())
+#endif
       .Default(sandbox::CrashSIGSYSIoctl());
 }
 
 }  // namespace
 
 RendererProcessPolicy::RendererProcessPolicy() {}
 RendererProcessPolicy::~RendererProcessPolicy() {}
 
 ResultExpr RendererProcessPolicy::EvaluateSyscall(int sysno) const {
   switch (sysno) {
@@ skipping 29 matching lines @@
       return sandbox::RestrictSchedTarget(GetPolicyPid(), sysno);
     case __NR_prlimit64:
       return Error(EPERM);  // See crbug.com/160157.
     default:
       // Default on the content baseline policy.
       return SandboxBPFBasePolicy::EvaluateSyscall(sysno);
   }
 }
 
 }  // namespace content