Changeset 3067 for source/ariba/overlay

Timestamp:

Apr 24, 2009, 10:04:11 AM (16 years ago)

Author:

Christoph Mayer

Message:

-fixed #30 (stress test pingpong, has some errors), #24 (autolinks for easy message sending)

Location:

source/ariba/overlay

Files:

• BaseOverlay.cpp (modified) (11 diffs)
• BaseOverlay.h (modified) (1 diff)
• modules/OverlayInterface.h (modified) (1 diff)
• modules/onehop/OneHop.cpp (modified) (3 diffs)

source/ariba/overlay/BaseOverlay.cpp

@@ -123 +123 @@
         logging_debug( "dropping all auto-links ..." );
 
+        // now we start leaving the spovnet: fist delete all links
+        // that we still have in the baseoverlay initiated by
+        // some services, the leave the actual overlay structure,
+        // then leave the spovnet
+
+        // --> drop all service links
+
+        vector<LinkID> servicelinks;
         BOOST_FOREACH( LinkPair item, linkMapping ){
-                if( item.second.autolink )
-                        dropLink( item.first );
-        }
+                if( item.second.service != OverlayInterface::OVERLAY_SERVICE_ID )
+                        servicelinks.push_back( item.first );
+        }
+        BOOST_FOREACH( LinkID lnk, servicelinks ){
+                // the dropLink function will remove
+                // the item from the linkMapping
+                dropLink( lnk );
+        }
+
+        // --> leave overlay structure
 
         logging_debug( "leaving overlay" );
         // first, leave the overlay interface
         overlayInterface->leaveOverlay();
+
+        // --> leave spovnet
 
         if( state != BaseOverlayStateInitiator ){
@@ -185 +202 @@
 }
 
-const EndpointDescriptor& BaseOverlay::getEndpointDescriptor(const LinkID& link) const {
-
-        return bc->getEndpointDescriptor( link );
-}
-
-const EndpointDescriptor& BaseOverlay::getEndpointDescriptor(const NodeID& node) const {
-
-        if( node == nodeId || node == NodeID::UNSPECIFIED )
-                return bc->getEndpointDescriptor();
-
-        if( overlayInterface == NULL ){
-                logging_error( "overlay interface not set, cannot resolve endpoint" );
-                return EndpointDescriptor::UNSPECIFIED;
-        }
-
-        // TODO: if this is not a onehop overlay the operation will go asynchronously
-        return overlayInterface->resolveNode( node );
-}
-
 const LinkID BaseOverlay::establishLink(const NodeID& node, const ServiceID& service){
 
@@ -227 +225 @@
         }
 
+        const LinkID link = bc->establishLink( ep );
+
         CommunicationListener* receiver = communicationListeners.get( service );
-        const LinkID link = bc->establishLink( ep );
-
         LinkItem item (link, NodeID::UNSPECIFIED, service, receiver);
         linkMapping.insert( make_pair(link, item) );
@@ -292 +290 @@
                                 ". creating auto link ...");
 
-                const LinkID link = establishLink( node, service );
+                link = establishLink( node, service );
                 LinkMapping::iterator i = linkMapping.find( link );
 
-                if( i == linkMapping.end() ){
+                if( i == linkMapping.end() || link == LinkID::UNSPECIFIED ){
                         logging_error( "failed to establish auto link to node " << node.toString() <<
                                         " for service " << service.toString() );
@@ -303 +301 @@
                 i->second.autolink = true;
 
+                logging_debug( "establishing autolink in progress to node "
+                                << node.toString() << " with new link-id " << link.toString() );
+
         } // if( link != LinkID::UNSPECIFIED )
 
+        assert( link != LinkID::UNSPECIFIED );
+
+        // mark the link as used, as we
+        // now send a message through it
         i->second.markused();
+
+        // send the message through the new link. the link may not be functional,
+        // but for us there is a link-id so we can send messages through it. if
+        // the link is not yet up and the message needs to be cached, this is the
+        // task of the BaseCommunication, it will cache and send it later.
         return sendMessage( message, link );
 }
 
-void BaseOverlay::bind(CommunicationListener* listener, const ServiceID& sid){
+const EndpointDescriptor& BaseOverlay::getEndpointDescriptor(const LinkID& link) const {
+
+        return bc->getEndpointDescriptor( link );
+}
+
+const EndpointDescriptor& BaseOverlay::getEndpointDescriptor(const NodeID& node) const {
+
+        if( node == nodeId || node == NodeID::UNSPECIFIED )
+                return bc->getEndpointDescriptor();
+
+        if( overlayInterface == NULL ){
+                logging_error( "overlay interface not set, cannot resolve endpoint" );
+                return EndpointDescriptor::UNSPECIFIED;
+        }
+
+        // TODO: if this is not a onehop overlay the operation will go asynchronously
+        return overlayInterface->resolveNode( node );
+}
+
+bool BaseOverlay::bind(CommunicationListener* listener, const ServiceID& sid){
         logging_debug( "binding communication listener " << listener
                 << " on serviceid " << sid.toString() );
@@ -316 +345 @@
                 logging_error( "some listener already registered for service id "
                         << sid.toString() );
-                return;
+                return false;
         }
 
         communicationListeners.registerItem( listener, sid );
-}
-
-void BaseOverlay::unbind(CommunicationListener* listener, const ServiceID& sid){
+        return true;
+}
+
+bool BaseOverlay::unbind(CommunicationListener* listener, const ServiceID& sid){
         logging_debug( "unbinding listener " << listener
                 << " from serviceid " << sid.toString() );
@@ -328 +358 @@
         if( !communicationListeners.contains( sid ) ){
                 logging_warn( "cannot unbind listener. no listener registered on service id " << sid.toString() );
-                return;
+                return false;
         }
 
@@ -334 +364 @@
                 logging_warn( "listener bound to service id " << sid.toString()
                         << " is different than listener trying to unbind" );
-                return;
+                return false;
         }
 
         communicationListeners.unregisterItem( sid );
-}
-
-void BaseOverlay::bind(NodeListener* listener){
+        return true;
+}
+
+bool BaseOverlay::bind(NodeListener* listener){
         logging_debug( "binding node listener " << listener );
 
@@ -346 +377 @@
         if( i != nodeListeners.end() ){
                 logging_warn( "node listener " << listener << " is already bound, cannot bind" );
-                return;
+                return false;
         }
 
         nodeListeners.push_back( listener );
-}
-
-void BaseOverlay::unbind(NodeListener* listener){
+        return true;
+}
+
+bool BaseOverlay::unbind(NodeListener* listener){
         logging_debug( "unbinding node listener " << listener );
 
@@ -358 +390 @@
         if( i == nodeListeners.end() ){
                 logging_warn( "node listener " << listener << " is not bound, cannot unbind" );
-                return;
+                return false;
         }
 
         nodeListeners.erase( i );
+        return true;
 }
 
@@ -445 +478 @@
         if( i == linkMapping.end() ) {
                 // this can also be one of the baseoverlay links that
-                // no mapping is stored for. therefore we issue no warning
+                // no mapping is stored for. therefore we issue no warning.
+                // it can also be a link that has been dropped and the
+                // mapping is already deleted in the dropLink function.
+                // also, the service notification is issued then in dropLink
                 return;
         }

source/ariba/overlay/BaseOverlay.h

@@ -202 +202 @@
          * TODO
          */
-        void bind(CommunicationListener* listener, const ServiceID& sid);
-
-        /**
-         * TODO
-         */
-        void unbind(CommunicationListener* listener, const ServiceID& sid);
-
-        /**
-         * TODO
-         */
-        void bind(NodeListener* listener);
-
-        /**
-         * TODO
-         */
-        void unbind(NodeListener* listener);
+        bool bind(CommunicationListener* listener, const ServiceID& sid);
+
+        /**
+         * TODO
+         */
+        bool unbind(CommunicationListener* listener, const ServiceID& sid);
+
+        /**
+         * TODO
+         */
+        bool bind(NodeListener* listener);
+
+        /**
+         * TODO
+         */
+        bool unbind(NodeListener* listener);
 
         /**

source/ariba/overlay/modules/OverlayInterface.h

@@ -58 +58 @@
 
 class OverlayInterface : public CommunicationListener {
+        friend class BaseOverlay;
 public:
         OverlayInterface(

source/ariba/overlay/modules/onehop/OneHop.cpp

@@ -146 +146 @@
         logging_info( "leaving onehop overlay structure" );
 
+        // set the state to invalid, this will prevent from
+        // handling onLinkDown events, as we are traversing the
+        // overlayNodes map and the onLinkDown function is called
+        // from the BaseOverlay and OneHop::onLinkDown will also
+        // try to access the overlayNodes structure.
+        state = OneHopStateInvalid;
+
         //
         // close all links, this will indicate the other nodes that we left
@@ -158 +165 @@
         }
 
-        state = OneHopStateInvalid;
         pendingLinks = 0;
 }
@@ -164 +170 @@
 
 void OneHop::onLinkDown(const LinkID& lnk, const NodeID& remote){
+
+        // don't handle when we are in state-invalid,
+        // see comment in OneHop::leaveOverlay
+        if( state == OneHopStateInvalid ) return;
 
         //