Add support for native ports in the vm.

runtime/vm/port.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/port.h"
 
 #include "platform/utils.h"
 #include "vm/dart_api_impl.h"
 #include "vm/isolate.h"
-#include "vm/message_queue.h"
+#include "vm/message.h"
 #include "vm/thread.h"
 
 namespace dart {
 
 DECLARE_FLAG(bool, trace_isolates);
 
+

[siva, 2012/01/28 00:21:05]

extra blank line?

[turnidge, 2012/01/31 20:04:31]

So when is it two and when is it one?  Functions get two, but variables do not? 
Or is this because I am following a DECLARE?  Just confused, that's all.

[siva, 2012/02/01 00:38:56]

I think functions get 2 and vars get 1.

On 2012/01/31 20:04:31, turnidge wrote:

[turnidge, 2012/02/01 18:51:26]

Done.

 Mutex* PortMap::mutex_ = NULL;
-
 PortMap::Entry* PortMap::map_ = NULL;
-Isolate* PortMap::deleted_entry_ = reinterpret_cast<Isolate*>(1);
+MessageHandler* PortMap::deleted_entry_ = reinterpret_cast<MessageHandler*>(1);
 intptr_t PortMap::capacity_ = 0;
 intptr_t PortMap::used_ = 0;
 intptr_t PortMap::deleted_ = 0;
-
 Dart_Port PortMap::next_port_ = 7111;
 
 
 intptr_t PortMap::FindPort(Dart_Port port) {
   intptr_t index = port % capacity_;
   intptr_t start_index = index;
   Entry entry = map_[index];
-  while (entry.isolate != NULL) {
+  while (entry.handler != NULL) {
     if (entry.port == port) {
       return index;
     }
     index = (index + 1) % capacity_;
     // Prevent endless loops.
     ASSERT(index != start_index);
     entry = map_[index];
   }
   return -1;
 }
@@ skipping 33 matching lines @@
   ASSERT(result != 0);
   return result;
 }
 
 
 void PortMap::SetLive(Dart_Port port) {
   MutexLocker ml(mutex_);
   intptr_t index = FindPort(port);
   ASSERT(index >= 0);
   map_[index].live = true;
-  map_[index].isolate->increment_live_ports();
+  map_[index].handler->increment_live_ports();
 }
 
 
 void PortMap::MaintainInvariants() {
   intptr_t empty = capacity_ - used_ - deleted_;
   if (used_ > ((capacity_ / 4) * 3)) {
     // Grow the port map.
     Rehash(capacity_ * 2);
   } else if (empty < deleted_) {
     // Rehash without growing the table to flush the deleted slots out of the
     // map.
     Rehash(capacity_);
   }
 }
 
 
-Dart_Port PortMap::CreatePort() {
-  Isolate* isolate = Isolate::Current();
-  MutexLocker ml(mutex_);
+Dart_Port PortMap::CreatePort(MessageHandler* handler) {
+  ASSERT(handler != NULL);

[siva, 2012/01/28 00:21:05]

#ifdef DEBUG
     handler->CheckAccess();
#endif

[turnidge, 2012/01/31 20:04:31]

Done.

+  mutex_->Lock();
 
   Entry entry;
   entry.port = AllocatePort();
-  entry.isolate = isolate;
+  entry.handler = handler;
   entry.live = false;
 
   // Search for the first unused slot. Make use of the knowledge that here is
   // currently no port with this id in the port map.
   ASSERT(FindPort(entry.port) < 0);
   intptr_t index = entry.port % capacity_;
   Entry cur = map_[index];
   // Stop the search at the first found unused (free or deleted) slot.
   while (cur.port != 0) {
     index = (index + 1) % capacity_;
     cur = map_[index];
   }
 
   // Insert the newly created port at the index.
   ASSERT(index >= 0);
   ASSERT(index < capacity_);
   ASSERT(map_[index].port == 0);
-  ASSERT((map_[index].isolate == NULL) ||
-         (map_[index].isolate == deleted_entry_));
-  if (map_[index].isolate == deleted_entry_) {
+  ASSERT((map_[index].handler == NULL) ||
+         (map_[index].handler == deleted_entry_));
+  if (map_[index].handler == deleted_entry_) {
     // Consuming a deleted entry.
     deleted_--;
   }
   map_[index] = entry;
-  isolate->increment_num_ports();
 
   // Increment number of used slots and grow if necessary.
   used_++;
   MaintainInvariants();
 
+  mutex_->Unlock();
   return entry.port;
 }
 
 
-void PortMap::ClosePort(Dart_Port port) {
-  Isolate* isolate = Isolate::Current();
+bool PortMap::ClosePort(Dart_Port port) {
+  MessageHandler* handler = NULL;
   {
-    MutexLocker ml(mutex_);
+    mutex_->Lock();
     intptr_t index = FindPort(port);
     if (index < 0) {
-      return;
+      mutex_->Unlock();
+      return false;
     }
     ASSERT(index < capacity_);
     ASSERT(map_[index].port != 0);
-    ASSERT(map_[index].isolate == isolate);
+    ASSERT(map_[index].handler != deleted_entry_);
+    ASSERT(map_[index].handler != NULL);
+
+    handler = map_[index].handler;
+#if defined(DEBUG)
+    handler->CheckAccess();
+#endif
     // Before releasing the lock mark the slot in the map as deleted. This makes
     // it possible to release the port map lock before flushing all of its
     // pending messages below.
     map_[index].port = 0;
-    map_[index].isolate = deleted_entry_;
-    isolate->decrement_num_ports();
+    map_[index].handler = deleted_entry_;
     if (map_[index].live) {
-      isolate->decrement_live_ports();
+      handler->decrement_live_ports();
     }
 
     used_--;
     deleted_++;
     MaintainInvariants();
+    mutex_->Unlock();
   }
-  isolate->ClosePort(port);
+  handler->ClosePort(port);
+  if (!handler->HasLivePorts() && handler->OwnedByPortMap()) {
+    delete handler;
+  }
+  return true;
 }
 
 
-void PortMap::ClosePorts() {
-  Isolate* isolate = Isolate::Current();
+void PortMap::ClosePorts(MessageHandler* handler) {
   {
     MutexLocker ml(mutex_);
     for (intptr_t i = 0; i < capacity_; i++) {
-      if (map_[i].isolate == isolate) {
+      if (map_[i].handler == handler) {
         // Mark the slot as deleted.
         map_[i].port = 0;
-        map_[i].isolate = deleted_entry_;
-        isolate->decrement_num_ports();
-
+        map_[i].handler = deleted_entry_;
+        if (map_[i].live) {
+          handler->decrement_live_ports();
+        }
         used_--;
         deleted_++;
       }
     }
     MaintainInvariants();
   }
-  isolate->CloseAllPorts();
+  handler->CloseAllPorts();
 }
 
 
 bool PortMap::PostMessage(Message* message) {
   // TODO(turnidge): Add a scoped locker for mutexes which is not a
   // stack resource.  This would probably be useful in the platform
   // headers.
   mutex_->Lock();
   intptr_t index = FindPort(message->dest_port());
   if (index < 0) {
     free(message);
     mutex_->Unlock();
     return false;
   }
   ASSERT(index >= 0);
   ASSERT(index < capacity_);
-  Isolate* isolate = map_[index].isolate;
+  MessageHandler* handler = map_[index].handler;
   ASSERT(map_[index].port != 0);
-  ASSERT((isolate != NULL) && (isolate != deleted_entry_));
-  isolate->PostMessage(message);
+  ASSERT((handler != NULL) && (handler != deleted_entry_));
+  handler->PostMessage(message);
   mutex_->Unlock();
   return true;
 }
 
 
 void PortMap::InitOnce() {
   mutex_ = new Mutex();
 
   static const intptr_t kInitialCapacity = 8;
   // TODO(iposva): Verify whether we want to keep exponentially growing.
   ASSERT(Utils::IsPowerOfTwo(kInitialCapacity));
   map_ = new Entry[kInitialCapacity];
   memset(map_, 0, kInitialCapacity * sizeof(Entry));
   capacity_ = kInitialCapacity;
   used_ = 0;
   deleted_ = 0;
 }
 
-
 }  // namespace dart