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

Side by Side Diff: base/memory/shared_memory_handle_mac.cc

Issue 2555483002: Add POSIX shared memory support for Mac (Closed)
Patch Set: Address erikchen@ nits Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/memory/shared_memory_handle.h" 5 #include "base/memory/shared_memory_handle.h"
6 6
7 #include <mach/mach_vm.h> 7 #include <mach/mach_vm.h>
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <sys/mman.h> 9 #include <sys/mman.h>
10 #include <unistd.h> 10 #include <unistd.h>
11 11
12 #include "base/mac/mac_util.h" 12 #include "base/mac/mac_util.h"
13 #include "base/mac/mach_logging.h"
13 #include "base/posix/eintr_wrapper.h" 14 #include "base/posix/eintr_wrapper.h"
14 15
15 namespace base { 16 namespace base {
16 17
17 SharedMemoryHandle::SharedMemoryHandle() {} 18 SharedMemoryHandle::SharedMemoryHandle()
19 : type_(MACH), memory_object_(MACH_PORT_NULL) {}
20
21 SharedMemoryHandle::SharedMemoryHandle(
22 const base::FileDescriptor& file_descriptor)
23 : type_(POSIX), file_descriptor_(file_descriptor) {}
18 24
19 SharedMemoryHandle::SharedMemoryHandle(mach_vm_size_t size) { 25 SharedMemoryHandle::SharedMemoryHandle(mach_vm_size_t size) {
26 type_ = MACH;
20 mach_port_t named_right; 27 mach_port_t named_right;
21 kern_return_t kr = mach_make_memory_entry_64( 28 kern_return_t kr = mach_make_memory_entry_64(
22 mach_task_self(), 29 mach_task_self(),
23 &size, 30 &size,
24 0, // Address. 31 0, // Address.
25 MAP_MEM_NAMED_CREATE | VM_PROT_READ | VM_PROT_WRITE, 32 MAP_MEM_NAMED_CREATE | VM_PROT_READ | VM_PROT_WRITE,
26 &named_right, 33 &named_right,
27 MACH_PORT_NULL); // Parent handle. 34 MACH_PORT_NULL); // Parent handle.
28 if (kr != KERN_SUCCESS) { 35 if (kr != KERN_SUCCESS) {
29 memory_object_ = MACH_PORT_NULL; 36 memory_object_ = MACH_PORT_NULL;
30 return; 37 return;
31 } 38 }
32 39
33 memory_object_ = named_right; 40 memory_object_ = named_right;
34 size_ = size; 41 size_ = size;
35 pid_ = GetCurrentProcId(); 42 pid_ = GetCurrentProcId();
36 ownership_passes_to_ipc_ = false; 43 ownership_passes_to_ipc_ = false;
37 } 44 }
38 45
39 SharedMemoryHandle::SharedMemoryHandle(mach_port_t memory_object, 46 SharedMemoryHandle::SharedMemoryHandle(mach_port_t memory_object,
40 mach_vm_size_t size, 47 mach_vm_size_t size,
41 base::ProcessId pid) 48 base::ProcessId pid)
42 : memory_object_(memory_object), 49 : type_(MACH),
50 memory_object_(memory_object),
43 size_(size), 51 size_(size),
44 pid_(pid), 52 pid_(pid),
45 ownership_passes_to_ipc_(false) {} 53 ownership_passes_to_ipc_(false) {}
46 54
47 SharedMemoryHandle::SharedMemoryHandle(const SharedMemoryHandle& handle) { 55 SharedMemoryHandle::SharedMemoryHandle(const SharedMemoryHandle& handle) {
48 CopyRelevantData(handle); 56 CopyRelevantData(handle);
49 } 57 }
50 58
51 SharedMemoryHandle& SharedMemoryHandle::operator=( 59 SharedMemoryHandle& SharedMemoryHandle::operator=(
52 const SharedMemoryHandle& handle) { 60 const SharedMemoryHandle& handle) {
53 if (this == &handle) 61 if (this == &handle)
54 return *this; 62 return *this;
55 63
64 type_ = handle.type_;
56 CopyRelevantData(handle); 65 CopyRelevantData(handle);
57 return *this; 66 return *this;
58 } 67 }
59 68
60 SharedMemoryHandle SharedMemoryHandle::Duplicate() const { 69 SharedMemoryHandle SharedMemoryHandle::Duplicate() const {
61 if (!IsValid()) 70 if (!IsValid())
62 return SharedMemoryHandle(MACH_PORT_NULL, 0, 0); 71 return SharedMemoryHandle(MACH_PORT_NULL, 0, 0);
63 72
64 // Increment the ref count. 73 // Increment the ref count.
65 kern_return_t kr = mach_port_mod_refs(mach_task_self(), memory_object_, 74 kern_return_t kr = mach_port_mod_refs(mach_task_self(), memory_object_,
66 MACH_PORT_RIGHT_SEND, 1); 75 MACH_PORT_RIGHT_SEND, 1);
67 DCHECK_EQ(kr, KERN_SUCCESS); 76 DCHECK_EQ(kr, KERN_SUCCESS);
77
Alexei Svitkine (slow) 2016/12/09 18:04:15 Nit: Remove extra diff.
lawrencewu 2016/12/09 18:33:56 Done.
68 SharedMemoryHandle handle(*this); 78 SharedMemoryHandle handle(*this);
69 handle.SetOwnershipPassesToIPC(true); 79 handle.SetOwnershipPassesToIPC(true);
70 return handle; 80 return handle;
71 } 81 }
72 82
73 bool SharedMemoryHandle::operator==(const SharedMemoryHandle& handle) const { 83 bool SharedMemoryHandle::operator==(const SharedMemoryHandle& handle) const {
74 if (!IsValid() && !handle.IsValid()) 84 if (!IsValid() && !handle.IsValid())
75 return true; 85 return true;
76 86
77 return memory_object_ == handle.memory_object_ && size_ == handle.size_ && 87 if (type_ != handle.type_)
78 pid_ == handle.pid_; 88 return false;
89
90 switch (type_) {
91 case POSIX:
92 return file_descriptor_.fd == handle.file_descriptor_.fd;
93 case MACH:
94 return memory_object_ == handle.memory_object_ && size_ == handle.size_ &&
95 pid_ == handle.pid_;
96 }
79 } 97 }
80 98
81 bool SharedMemoryHandle::operator!=(const SharedMemoryHandle& handle) const { 99 bool SharedMemoryHandle::operator!=(const SharedMemoryHandle& handle) const {
82 return !(*this == handle); 100 return !(*this == handle);
83 } 101 }
84 102
85 bool SharedMemoryHandle::IsValid() const { 103 bool SharedMemoryHandle::IsValid() const {
86 return memory_object_ != MACH_PORT_NULL; 104 switch (type_) {
105 case POSIX:
106 return file_descriptor_.fd >= 0;
107 case MACH:
108 return memory_object_ != MACH_PORT_NULL;
109 }
87 } 110 }
88 111
89 mach_port_t SharedMemoryHandle::GetMemoryObject() const { 112 mach_port_t SharedMemoryHandle::GetMemoryObject() const {
113 DCHECK_EQ(type_, MACH);
90 return memory_object_; 114 return memory_object_;
91 } 115 }
92 116
93 bool SharedMemoryHandle::GetSize(size_t* size) const { 117 bool SharedMemoryHandle::GetSize(size_t* size) const {
94 if (!IsValid()) { 118 if (!IsValid()) {
95 *size = 0; 119 *size = 0;
96 return true; 120 return true;
97 } 121 }
98 122
99 *size = size_; 123 switch (type_) {
100 return true; 124 case SharedMemoryHandle::POSIX:
125 struct stat st;
126 if (fstat(file_descriptor_.fd, &st) != 0)
127 return false;
128 if (st.st_size < 0)
129 return false;
130 *size = st.st_size;
131 return true;
132 case SharedMemoryHandle::MACH:
133 *size = size_;
134 return true;
135 }
101 } 136 }
102 137
103 bool SharedMemoryHandle::MapAt(off_t offset, 138 bool SharedMemoryHandle::MapAt(off_t offset,
104 size_t bytes, 139 size_t bytes,
105 void** memory, 140 void** memory,
106 bool read_only) { 141 bool read_only) {
107 DCHECK(IsValid()); 142 DCHECK(IsValid());
108 DCHECK_EQ(pid_, GetCurrentProcId()); 143 switch (type_) {
109 kern_return_t kr = mach_vm_map( 144 case SharedMemoryHandle::POSIX:
110 mach_task_self(), 145 *memory = mmap(nullptr, bytes, PROT_READ | (read_only ? 0 : PROT_WRITE),
111 reinterpret_cast<mach_vm_address_t*>(memory), // Output parameter 146 MAP_SHARED, file_descriptor_.fd, offset);
112 bytes, 147
113 0, // Alignment mask 148 return *memory && *memory != reinterpret_cast<void*>(-1);
114 VM_FLAGS_ANYWHERE, memory_object_, offset, 149 case SharedMemoryHandle::MACH:
115 FALSE, // Copy 150 DCHECK_EQ(pid_, GetCurrentProcId());
116 VM_PROT_READ | (read_only ? 0 : VM_PROT_WRITE), // Current protection 151 kern_return_t kr = mach_vm_map(
117 VM_PROT_WRITE | VM_PROT_READ | VM_PROT_IS_MASK, // Maximum protection 152 mach_task_self(),
118 VM_INHERIT_NONE); 153 reinterpret_cast<mach_vm_address_t*>(memory), // Output parameter
119 return kr == KERN_SUCCESS; 154 bytes,
155 0, // Alignment mask
156 VM_FLAGS_ANYWHERE,
157 memory_object_,
158 offset,
159 FALSE, // Copy
160 VM_PROT_READ | (read_only ? 0 : VM_PROT_WRITE), // Current protection
161 VM_PROT_WRITE | VM_PROT_READ | VM_PROT_IS_MASK, // Maximum protection
162 VM_INHERIT_NONE);
163 return kr == KERN_SUCCESS;
164 }
120 } 165 }
121 166
122 void SharedMemoryHandle::Close() const { 167 void SharedMemoryHandle::Close() const {
123 if (!IsValid()) 168 if (!IsValid())
124 return; 169 return;
125 170
126 kern_return_t kr = mach_port_deallocate(mach_task_self(), memory_object_); 171 switch (type_) {
127 if (kr != KERN_SUCCESS) 172 case POSIX:
128 DPLOG(ERROR) << "Error deallocating mach port: " << kr; 173 if (IGNORE_EINTR(close(file_descriptor_.fd)) < 0)
174 DPLOG(ERROR) << "Error closing fd.";
175 break;
176 case MACH:
177 kern_return_t kr = mach_port_deallocate(mach_task_self(), memory_object_);
178 if (kr != KERN_SUCCESS)
179 MACH_LOG(ERROR, kr) << "Error deallocating mach port";
180 break;
181 }
129 } 182 }
130 183
131 void SharedMemoryHandle::SetOwnershipPassesToIPC(bool ownership_passes) { 184 void SharedMemoryHandle::SetOwnershipPassesToIPC(bool ownership_passes) {
185 DCHECK_EQ(type_, MACH);
132 ownership_passes_to_ipc_ = ownership_passes; 186 ownership_passes_to_ipc_ = ownership_passes;
133 } 187 }
134 188
135 bool SharedMemoryHandle::OwnershipPassesToIPC() const { 189 bool SharedMemoryHandle::OwnershipPassesToIPC() const {
190 DCHECK_EQ(type_, MACH);
136 return ownership_passes_to_ipc_; 191 return ownership_passes_to_ipc_;
137 } 192 }
138 193
139 void SharedMemoryHandle::CopyRelevantData(const SharedMemoryHandle& handle) { 194 void SharedMemoryHandle::CopyRelevantData(const SharedMemoryHandle& handle) {
140 memory_object_ = handle.memory_object_; 195 type_ = handle.type_;
141 size_ = handle.size_; 196 switch (type_) {
142 pid_ = handle.pid_; 197 case POSIX:
143 ownership_passes_to_ipc_ = handle.ownership_passes_to_ipc_; 198 file_descriptor_ = handle.file_descriptor_;
199 break;
200 case MACH:
201 memory_object_ = handle.memory_object_;
202 size_ = handle.size_;
203 pid_ = handle.pid_;
204 ownership_passes_to_ipc_ = handle.ownership_passes_to_ipc_;
205 break;
206 }
144 } 207 }
145 208
146 } // namespace base 209 } // namespace base
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698