Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4174)

Unified Diff: base/sys_info_linux.cc

Issue 55913004: Reland "Enable SysInfo::AmountOfPhysicalMemory to be called from within the Linux Sandbox" (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Seperate MaxSharedMemorySize and AmountOfPhysicalMemory initialization. Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/sys_info.h ('k') | content/zygote/zygote_main_linux.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/sys_info_linux.cc
diff --git a/base/sys_info_linux.cc b/base/sys_info_linux.cc
index acc477134cd38bed2ba37e6add9f84370bfc87ca..d7c9f1032ff5470100f81ce350e34bc69b0396ce 100644
--- a/base/sys_info_linux.cc
+++ b/base/sys_info_linux.cc
@@ -7,6 +7,7 @@
#include <limits>
#include "base/file_util.h"
+#include "base/lazy_instance.h"
#include "base/logging.h"
#include "base/strings/string_number_conversions.h"
@@ -22,41 +23,68 @@ int64 AmountOfMemory(int pages_name) {
return static_cast<int64>(pages) * page_size;
}
+int64 AmountOfPhysicalMemory() {
+ return AmountOfMemory(_SC_PHYS_PAGES);
+}
+
+size_t MaxSharedMemorySize() {
+ std::string contents;
+ base::ReadFileToString(base::FilePath("/proc/sys/kernel/shmmax"), &contents);
+ DCHECK(!contents.empty());
+ if (!contents.empty() && contents[contents.length() - 1] == '\n') {
+ contents.erase(contents.length() - 1);
+ }
+
+ int64 limit;
+ if (!base::StringToInt64(contents, &limit)) {
+ limit = 0;
+ }
+ if (limit < 0 ||
+ static_cast<uint64>(limit) > std::numeric_limits<size_t>::max()) {
+ limit = 0;
+ }
+ DCHECK(limit > 0);
+ return static_cast<size_t>(limit);
+}
+
+template<typename T, T (*F)(void)>
+class LazySysInfoValue {
+ public:
+ LazySysInfoValue()
+ : value_(F()) { }
+
+ ~LazySysInfoValue() { }
+
+ T value() { return value_; }
+
+ private:
+ const T value_;
+
+ DISALLOW_COPY_AND_ASSIGN(LazySysInfoValue);
+};
+
+base::LazyInstance<LazySysInfoValue<int64, AmountOfPhysicalMemory> >::Leaky
+ g_lazy_physical_memory = LAZY_INSTANCE_INITIALIZER;
+base::LazyInstance<LazySysInfoValue<size_t, MaxSharedMemorySize> >::Leaky
+ g_lazy_max_shared_memory = LAZY_INSTANCE_INITIALIZER;
+
} // namespace
namespace base {
// static
-int64 SysInfo::AmountOfPhysicalMemory() {
- return AmountOfMemory(_SC_PHYS_PAGES);
+int64 SysInfo::AmountOfAvailablePhysicalMemory() {
+ return AmountOfMemory(_SC_AVPHYS_PAGES);
}
// static
-int64 SysInfo::AmountOfAvailablePhysicalMemory() {
- return AmountOfMemory(_SC_AVPHYS_PAGES);
+int64 SysInfo::AmountOfPhysicalMemory() {
+ return g_lazy_physical_memory.Get().value();
}
// static
size_t SysInfo::MaxSharedMemorySize() {
- static int64 limit;
- static bool limit_valid = false;
- if (!limit_valid) {
- std::string contents;
- ReadFileToString(FilePath("/proc/sys/kernel/shmmax"), &contents);
- DCHECK(!contents.empty());
- if (!contents.empty() && contents[contents.length() - 1] == '\n') {
- contents.erase(contents.length() - 1);
- }
- if (base::StringToInt64(contents, &limit)) {
- DCHECK(limit >= 0);
- DCHECK(static_cast<uint64>(limit) <= std::numeric_limits<size_t>::max());
- limit_valid = true;
- } else {
- NOTREACHED();
- return 0;
- }
- }
- return static_cast<size_t>(limit);
+ return g_lazy_max_shared_memory.Get().value();
}
// static
« no previous file with comments | « base/sys_info.h ('k') | content/zygote/zygote_main_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698