Implement missing SysInfo methods on OpenBSD

base/sys_info_openbsd.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 <sys/param.h>
 #include <sys/shm.h>
 #include <sys/sysctl.h>
 
 #include "base/logging.h"
 
+namespace {
+
+int64 AmountOfMemory(int pages_name) {
+  long pages = sysconf(pages_name);
+  long page_size = sysconf(_SC_PAGESIZE);
+  if (pages == -1 || page_size == -1) {
+    NOTREACHED();
+    return 0;
+  }
+  return static_cast<int64>(pages) * page_size;
+}
+
+}  // namespace
+
 namespace base {
 
+// static
 int SysInfo::NumberOfProcessors() {
   int mib[] = { CTL_HW, HW_NCPU };
   int ncpu;
   size_t size = sizeof(ncpu);
   if (sysctl(mib, arraysize(mib), &ncpu, &size, NULL, 0) == -1) {
     NOTREACHED();
     return 1;
   }
   return ncpu;
 }
 
+// static
 int64 SysInfo::AmountOfPhysicalMemory() {
-  long pages = sysconf(_SC_PHYS_PAGES);
-  long page_size = sysconf(_SC_PAGESIZE);
-  if (pages == -1 || page_size == -1) {
-    NOTREACHED();
-    return 0;
-  }
-
-  return static_cast<int64>(pages) * page_size;
+  return AmountOfMemory(_SC_PHYS_PAGES);
 }
 
+// static
+int64 SysInfo::AmountOfAvailablePhysicalMemory() {
+  return AmountOfMemory(_SC_AVPHYS_PAGES);
+}
+
+// static
 size_t SysInfo::MaxSharedMemorySize() {
   int mib[] = { CTL_KERN, KERN_SHMINFO, KERN_SHMINFO_SHMMAX };
   size_t limit;
   size_t size = sizeof(limit);
-  if (sysctl(mib, arraysize(mib), &limit, &size, NULL, 0) < 0) {
+  if (sysctl(mib, arraysize(mib), &limit, &size, NULL, 0) == -1) {

[Mark Mentovai, 2013/04/09 22:14:21]

I wouldn’t normally pick on this, but I can’t see a reason for this change, so…

For instances where you’re looking at syscall return values, where -1 is the
failure return value and don’t expect any other negative numbers, the |< 0| form
that was here before generally produces slightly better (and smaller) object
code. Same in the new code on line 68.

[mdempsky_google, 2013/04/09 23:06:32]

No problem.  Your project, your rules. :)

     NOTREACHED();
     return 0;
   }
   return limit;
 }
 
+// static
+std::string SysInfo::CPUModelName() {
+  int mib[] = { CTL_HW, HW_MODEL };
+  char name[256];
+  size_t len = arraysize(name);
+  if (sysctl(mib, arraysize(mib), name, &len, NULL, 0) == -1) {
+    NOTREACHED();
+    return std::string();
+  }
+  return name;
+}
+
 }  // namespace base