Add a new driver bug workaround SANDBOX_START_EARLY

base/sys_info_posix.cc

 // Copyright (c) 2011 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 "base/sys_info.h"
 
 #include <errno.h>
 #include <stddef.h>
 #include <stdint.h>
 #include <string.h>
@@ skipping 95 matching lines @@
     NOTREACHED();
     return std::string();
   }
   return std::string(info.sysname);
 }
 #endif
 
 #if !defined(OS_MACOSX) && !defined(OS_ANDROID)
 // static
 std::string SysInfo::OperatingSystemVersion() {
-  struct utsname info;
-  if (uname(&info) < 0) {
+  static struct utsname info = {};
+  if (!strlen(info.release) && uname(&info) < 0) {

[piman, 2016/05/06 19:53:11]

This is not thread safe. Why the change?

[Julien Isorce Samsung, 2016/05/07 08:18:13]

Txh for pointing that, well I made this change because uname is used from
GpuControlList when calling gpu::ApplyGpuDriverBugWorkarounds. This is a problem
when switches::kGpuSandboxStartEarly is set. On arm when kGpuSandboxStartEarly
is set they solve that by enabling uname from the sandbox policy directly since
they do not use the common GpuProcessPolicy but they use
CrosArmGpuProcessPolicy::EvaluateSyscall instead.

     NOTREACHED();
     return std::string();
   }
   return std::string(info.release);
 }
 #endif
 
 // static
 std::string SysInfo::OperatingSystemArchitecture() {
   struct utsname info;
   if (uname(&info) < 0) {
     NOTREACHED();
     return std::string();
   }
   std::string arch(info.machine);
   if (arch == "i386" || arch == "i486" || arch == "i586" || arch == "i686") {
     arch = "x86";
   } else if (arch == "amd64") {
     arch = "x86_64";
   }
   return arch;
 }
 
 // static
 size_t SysInfo::VMAllocationGranularity() {
   return getpagesize();
 }
 
 }  // namespace base