Bug fix: Whenever a device with new entity was created that causes a change in attachment point, deviceMoved notification should be sent after the indices are updated.

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).

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.

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

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

Reclassify devices in entityClassChanged only for devices that belong to entity classes that got changed.

FL-81 - Don't trust the buffer ID of the PacketIn if the switch has 0 buffers according to it's features reply.

method to delete a device removing all entities, updating secondary map indices first and then delete from devicemap.

BSC-2131

Send port status update from link discovery manager whenever a port status MODIFY message is received. Minor changes to printing of link discovery updates and addition of some trace logs.

Update attachmentpoint whenever topologyChanged() notification is received.  It is not necessary to process every event; just one update call per topologyChanged() should take care of fixing all attachment point changes.

Updates to attachment point learning and udpating.  DeviceManager listens to topologyChanged() events.  Whenever a port goes down or a switch is removed, attachment points are updated.  Cleanup of link discovery manager updates; cleanup also includes port up/down switch update/removed messages.

portEnabled(short) should return false if a port is not present.  TopologyManager checks for "Special" ports and port status as part of "isAttachmentPointPort".  The checking of this logic in devicemanager is remoed.

Rewrite of attachment point logic.  Attachmetn points are maintained independent of entities.  Update test cases.  Duplicate Mac test is disabled for the time being.

During flow reconciliation, device manager should not call learnDeviceByEntity, which would incorrectly update timestamps of entities existing in flow cache. It should call findDeviceByEntity instead.

If we cannot find an entity in the primary index we need to
classify it into an IEntityClass and the search in the class index.