Introduce a method called isConsistent(..) that compares an old attachment and...

Introduce a method called isConsistent(..) that compares an old attachment and a new attachment point. If the new attachment point is not consistent with the old one, then it has to be learnt by the device manager.

src/main/java/net/floodlightcontroller/topology/ITopologyService.java

@@ -43,6 +43,8 @@ public interface ITopologyService extends IFloodlightService  {
 
     public boolean isAllowed(long sw, short portId);
 
+    public boolean isConsistent(long oldSw, short oldPort, long newSw,
+                                short newPort);
     /**
      * If the dst is not allowed by the higher-level topology, this method provides
      * the topologically equivalent broadcast port.  

src/main/java/net/floodlightcontroller/topology/TopologyInstance.java

@@ -524,6 +524,12 @@ public class TopologyInstance {
         return true;
     }
 
+    public boolean isConsistent(long oldSw, short oldPort, long newSw,
+                                short newPort) {
+        if (isInternal(newSw, newPort)) return true;
+        return (oldSw == newSw && oldPort == newPort);
+    }
+
     protected Set<NodePortTuple>
     getBroadcastNodePortsInCluster(long sw) {
         long clusterId = getSwitchClusterId(sw);

src/main/java/net/floodlightcontroller/topology/TopologyManager.java

@@ -435,5 +435,11 @@ IRoutingService, ILinkDiscoveryListener {
     public boolean isBroadcastDomainPort(long sw, short port) {
         return currentInstance.isBroadcastDomainPort(new NodePortTuple(sw, port));
     }
+
+    @Override
+    public boolean isConsistent(long oldSw, short oldPort, long newSw,
+                                short newPort) {
+        return currentInstance.isConsistent(oldSw, oldPort, newSw, newPort);
+    }
 }