diff --git a/src/main/java/net/floodlightcontroller/devicemanager/DeviceAttachmentPoint.java b/src/main/java/net/floodlightcontroller/devicemanager/DeviceAttachmentPoint.java index 9571c8f3b4f0b3d13a254eb7113b423cfcdb7915..1d95244151398735c08e50d7bad26cd37aae7da7 100644 --- a/src/main/java/net/floodlightcontroller/devicemanager/DeviceAttachmentPoint.java +++ b/src/main/java/net/floodlightcontroller/devicemanager/DeviceAttachmentPoint.java @@ -46,8 +46,7 @@ public class DeviceAttachmentPoint { this.lastSeenInStorage = null; this.lastWrittenToStorage = null; - this.lastConflict = null; - this.conflictFrequency = 0; + resetConflictState(); } public SwitchPortTuple getSwitchPort() { @@ -138,6 +137,12 @@ public class DeviceAttachmentPoint { conflictFrequency += 100; } + public void resetConflictState() { + lastConflict = null; + conflictFrequency = 0; + setBlocked(false); + } + public boolean isInConflict() { return lastConflict != null; } diff --git a/src/main/java/net/floodlightcontroller/devicemanager/internal/DeviceManagerImpl.java b/src/main/java/net/floodlightcontroller/devicemanager/internal/DeviceManagerImpl.java index 390a7affcb4984478ab0d1e9670e1bc3be4673c6..7bc23bd23cd1c0b7226d60f828db481b190d4381 100755 --- a/src/main/java/net/floodlightcontroller/devicemanager/internal/DeviceManagerImpl.java +++ b/src/main/java/net/floodlightcontroller/devicemanager/internal/DeviceManagerImpl.java @@ -1670,11 +1670,11 @@ public class DeviceManagerImpl implements IDeviceManagerService, IOFMessageListe * Also make sure the attachmentPoints are in non-blocked state */ for (DeviceAttachmentPoint dap: tempAPMap.values()) { - dap.setBlocked(false); + dap.resetConflictState(); } d.setAttachmentPoints(tempAPMap.values()); for (DeviceAttachmentPoint dap : tempOldAPMap.values()) { - dap.setBlocked(false); + dap.resetConflictState(); d.addOldAttachmentPoint(dap); }