merge for change to classState map definition and some debug logs (https://github.com/floodlight/floodlight/pull/294)

Update to attachment point port.  newAP is given preference if it is not a broadcast domain port and its active time is later than oldAP.

Update to attachment point port.  newAP is given preference if it is not a broadcast domain port and its active time is later than oldAP. (https://github.com/floodlight/floodlight/pull/298)

Update to attachment point logic. NewAP is always given preference if it is not broadcast domain port and its active time is later than oldAP.

Rewrite of reclassify, made learnDeviceByEntity the only place where devices get created or updated (through re-allocation).

Update to attachment point logic.  Ignore the attachment point if it is not picked up from oldAPlist and does not replace already known AP. (https://github.com/floodlight/floodlight/pull/297)

If an attachment point port from packet-in does not replace already known attachment point, put it in oldAP list only if it was picked up from oldAP list.  Otherwise, ignore the attachment point.

Bug fix in the attachment point learning, where previously known AP was incorrectly added to the oldAP list. (https://github.com/floodlight/floodlight/pull/296)

Bug fix: when an attachment point from packet-in (newAP) does not replace the already known attachment point (oldAP), then we need to put newAP in the oldAPlist for possibly flagging it as duplicate in the future.  Earlier, we were adding the already known attachment point in the list.

Clear out oldAPs whenever updateAttachmentPoint() is called as it is triggered due to topologyChange and device reclassification. (https://github.com/floodlight/floodlight/pull/295)

Clear out oldAPs whenever updateAttachmentPoint() is called as it is triggered due to topologyChange and device reclassification.

Changes to classState map to use IEntityClass name as key now. Also debug logs sprayed around some methods for more visibility if required while debugging.

Update to attachment point comptuation to flag devices flapping between two switch ports. Enable duplicate mac unit tests.

Update to attachment point comptuation to flag devices flapping between two switch ports.  Enable duplicate mac unit tests.

Bug fix in attachment point comparison. (https://github.com/floodlight/floodlight/pull/292)

merge for fix to entityclassChanged to reclassify devices that have null entity class as well. (https://github.com/floodlight/floodlight/pull/290)

MockDevice only returns valid attachmentPoints

Reclassify all devices that belong to changed entityclasses or null entity class. And also some fixes to prevent race conditions.

Firewall (https://github.com/floodlight/floodlight/pull/289)

BSC-2398 Add getServiceImpls() for MemoryStorageService (https://github.com/floodlight/floodlight/pull/288)

Order the OFMessageListener in mockFloodlightProvider

This reverts commit 75bc3dd9598efe061a37a579df5cb30c39d11b52.

merge for fixes to reclassify (https://github.com/floodlight/floodlight/pull/286)

When deleting or allocating a device as a part of reclassify update the device map, primary and class indices to reflect this change.

Rename writeLink to writeLinkToStorage.  Make these methods protected so others can extend them if needed. (https://github.com/floodlight/floodlight/pull/285)

rename writeLink to writeLinkToStorage.  Make these methods protected so others can extend them if needed.

Change time unit from sec to ms and make unit tests faster. (https://github.com/floodlight/floodlight/pull/284)