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

Unified Diff: base/memory/discardable_memory_mac.cc

Issue 114923005: base: Discardable memory types. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add missing include Created 7 years 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
Index: base/memory/discardable_memory_mac.cc
diff --git a/base/memory/discardable_memory_mac.cc b/base/memory/discardable_memory_mac.cc
index aa6823509d50f701ff6f07e6d3981e0df7226516..ae88c94cd9254dc25f12ed0d405138d9ffffbf6a 100644
--- a/base/memory/discardable_memory_mac.cc
+++ b/base/memory/discardable_memory_mac.cc
@@ -2,14 +2,14 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "base/memory/discardable_memory.h"
willchan no longer on Chromium 2013/12/24 01:21:06 I'd keep this first.
reveman 2013/12/26 11:46:08 Done.
-
#include <mach/mach.h>
#include <sys/mman.h>
#include "base/basictypes.h"
#include "base/compiler_specific.h"
#include "base/logging.h"
+#include "base/memory/discardable_memory.h"
+#include "base/memory/discardable_memory_emulated.h"
#include "base/memory/scoped_ptr.h"
namespace base {
@@ -22,51 +22,62 @@ const int kDiscardableMemoryTag = VM_MAKE_TAG(252);
class DiscardableMemoryMac : public DiscardableMemory {
public:
- DiscardableMemoryMac(void* memory, size_t size)
- : memory_(memory),
+ explicit DiscardableMemoryMac(size_t size)
+ : buffer_(0),
size_(size) {
- DCHECK(memory_);
+ }
+
+ bool Initialize() {
+ kern_return_t ret = vm_allocate(mach_task_self(),
+ &buffer_,
+ size_,
+ VM_FLAGS_PURGABLE |
+ VM_FLAGS_ANYWHERE |
+ kDiscardableMemoryTag);
+ if (ret != KERN_SUCCESS) {
+ DLOG(ERROR) << "vm_allocate() failed";
+ return false;
+ }
+
+ return true;
}
virtual ~DiscardableMemoryMac() {
- vm_deallocate(mach_task_self(),
- reinterpret_cast<vm_address_t>(memory_),
- size_);
+ if (buffer_)
+ vm_deallocate(mach_task_self(), buffer_, size_);
}
- virtual LockDiscardableMemoryStatus Lock() OVERRIDE {
- DCHECK_EQ(0, mprotect(memory_, size_, PROT_READ | PROT_WRITE));
+ virtual DiscardableMemoryLockStatus Lock() OVERRIDE {
+ DCHECK_EQ(0, mprotect(buffer_, size_, PROT_READ | PROT_WRITE));
int state = VM_PURGABLE_NONVOLATILE;
- kern_return_t ret = vm_purgable_control(
- mach_task_self(),
- reinterpret_cast<vm_address_t>(memory_),
- VM_PURGABLE_SET_STATE,
- &state);
+ kern_return_t ret = vm_purgable_control(mach_task_self(),
+ buffer_,
+ VM_PURGABLE_SET_STATE,
+ &state);
if (ret != KERN_SUCCESS)
return DISCARDABLE_MEMORY_FAILED;
- return state & VM_PURGABLE_EMPTY ? DISCARDABLE_MEMORY_PURGED
- : DISCARDABLE_MEMORY_SUCCESS;
+ return state & VM_PURGABLE_EMPTY ? DISCARDABLE_MEMORY_LOCK_STATUS_PURGED
+ : DISCARDABLE_MEMORY_LOCK_STATUS_SUCCESS;
}
virtual void Unlock() OVERRIDE {
int state = VM_PURGABLE_VOLATILE | VM_VOLATILE_GROUP_DEFAULT;
- kern_return_t ret = vm_purgable_control(
- mach_task_self(),
- reinterpret_cast<vm_address_t>(memory_),
- VM_PURGABLE_SET_STATE,
- &state);
- DCHECK_EQ(0, mprotect(memory_, size_, PROT_NONE));
+ kern_return_t ret = vm_purgable_control(mach_task_self(),
+ buffer_,
+ VM_PURGABLE_SET_STATE,
+ &state);
+ DCHECK_EQ(0, mprotect(buffer_, size_, PROT_NONE));
if (ret != KERN_SUCCESS)
DLOG(ERROR) << "Failed to unlock memory.";
}
virtual void* Memory() const OVERRIDE {
- return memory_;
+ return buffer_;
}
private:
- void* const memory_;
+ vm_address_t buffer_;
const size_t size_;
DISALLOW_COPY_AND_ASSIGN(DiscardableMemoryMac);
@@ -75,26 +86,38 @@ class DiscardableMemoryMac : public DiscardableMemory {
} // namespace
// static
-bool DiscardableMemory::SupportedNatively() {
- return true;
+void DiscardableMemory::GetSupportedTypes(
+ std::vector<DiscardableMemoryType>* types) {
+ types->push_back(DISCARDABLE_MEMORY_TYPE_MAC);
+ types->push_back(DISCARDABLE_MEMORY_TYPE_EMULATED);
}
// static
scoped_ptr<DiscardableMemory> DiscardableMemory::CreateLockedMemory(
size_t size) {
- vm_address_t buffer = 0;
- kern_return_t ret = vm_allocate(mach_task_self(),
- &buffer,
- size,
- VM_FLAGS_PURGABLE |
- VM_FLAGS_ANYWHERE |
- kDiscardableMemoryTag);
- if (ret != KERN_SUCCESS) {
- DLOG(ERROR) << "vm_allocate() failed";
- return scoped_ptr<DiscardableMemory>();
+ switch (GetType()) {
+ case DISCARDABLE_MEMORY_TYPE_NONE:
+ case DISCARDABLE_MEMORY_TYPE_ANDROID:
+ return scoped_ptr<DiscardableMemory>();
+ case DISCARDABLE_MEMORY_TYPE_MAC: {
+ scoped_ptr<DiscardableMemoryMac> memory(new DiscardableMemoryMac(size));
+ if (!memory->Initialize())
+ return scoped_ptr<DiscardableMemory>();
+
+ return memory.PassAs<DiscardableMemory>();
+ }
+ case DISCARDABLE_MEMORY_TYPE_EMULATED: {
+ scoped_ptr<internal::DiscardableMemoryEmulated> memory(
+ new internal::DiscardableMemoryEmulated(size));
+ if (!memory->Initialize())
+ return scoped_ptr<DiscardableMemory>();
+
+ return memory.PassAs<DiscardableMemory>();
+ }
}
- return scoped_ptr<DiscardableMemory>(
- new DiscardableMemoryMac(reinterpret_cast<void*>(buffer), size));
+
+ NOTREACHED();
+ return scoped_ptr<DiscardableMemory>();
}
// static
@@ -106,6 +129,7 @@ bool DiscardableMemory::PurgeForTestingSupported() {
void DiscardableMemory::PurgeForTesting() {
int state = 0;
vm_purgable_control(mach_task_self(), 0, VM_PURGABLE_PURGE_ALL, &state);
+ internal::DiscardableMemoryEmulated::PurgeForTesting();
}
} // namespace base

Powered by Google App Engine
This is Rietveld 408576698