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) {

[Mark Mentovai, 2013/04/09 21:25:28]

This is good. You can do Linux this way too, and Windows a similar way.

[mdempsky_google, 2013/04/09 21:45:09]

Done.  (Not sure if the way I did the Windows code is idiomatic Chromium code
though.)

+  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 };
+  static const int mib[] = { CTL_HW, HW_NCPU };
   int ncpu;
   size_t size = sizeof(ncpu);
-  if (sysctl(mib, arraysize(mib), &ncpu, &size, NULL, 0) == -1) {
+  if (sysctl(const_cast<int*>(mib), arraysize(mib), &ncpu, &size, NULL, 0)

[Mark Mentovai, 2013/04/09 21:25:28]

No no, don’t const_cast it, that’s even worse. Just go back to using int. Broken
system headers are a reasonable reason to have to do that. Same on lines 54/57
and 67/70.

[mdempsky_google, 2013/04/09 21:45:09]

Okay, reverted.

+      == -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 };
+  static const 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(const_cast<int*>(mib), arraysize(mib), &limit, &size, NULL, 0)
+      == -1) {
     NOTREACHED();
     return 0;
   }
   return limit;
 }
 
+// static
+std::string SysInfo::CPUModelName() {
+  static const int mib[] = { CTL_HW, HW_MODEL };
+  char name[256];
+  size_t len = arraysize(name);
+  if (sysctl(const_cast<int*>(mib), arraysize(mib), name, &len, NULL, 0)
+      == -1) {
+    NOTREACHED();
+    return std::string();
+  }
+  return name;
+}
+
 }  // namespace base