Merge into master from pull request #1636: Open source stuffff (https://github.com/bigswitch/bigswitchcontroller/pull/1636)

Change getFeaturesReply method to 'query' so it's clear this is an external query, and avoids calling it when serializing to JSON.

* Must not call clearAllFlowMods() on slave -> master transition.
* Hack to solve immediate perf problem on switch handshake: only call
  readSwitchPortStateFromStorage() on role change. Switch handshake
  is now fast again but role change is slow (which is has been before)

BSC-2695
BSC-2727

Bug fix

Changes to REST API for links.  http://<controller>:8080/wm/topology/links/json would now return only direct and tunnel links (LinkType.DIRECT_LINK or LinkType.TUNNEL).  To get external links, use http://<controller>:8080/wm/topology/external-links/json.

Add isTunnelPort() to ILinkDiscoveryService.  A tunnel port is not an attachment point port even if no links are found through it.

Fix mismatch between Javadoc comment and method argument

Updates

BDDP packets are now transmitted with BSN Eth-type. Minor refactoring of the code inclues: (1) Removal of BDDP class; (2) Using LLDP encapsulated inside BSN header for BDDP functionality.

Fix an issue in the REST API where setting datalayer MAC addresses in OFActions was not getting displayed properly.

Merge into master from pull request #1515: TopologyManager should not process switch removed and port down messages explicitly as they are processed by link discovery module and generate corresponding link removed events. (https://github.com/bigswitch/bigswitchcontroller/pull/1515)

Merge into master from pull request #1514: Add RARP ethertype (https://github.com/bigswitch/bigswitchcontroller/pull/1514)

TopologyManager should not process switch removed and port down messages explicitly as they are processed by link discovery module and generate corresponding link removed events.

Changes to autoportfast working.  Quarantining a fast port does not depend on whether the autoportfast feature is enabled or not.

Rename isLLDPSuppressed to isLinkDiscoverySuppressed.  Rewrite of autoPortFast feature and how the ports are added to quarantine and maintenance.

Some (broken!) switches send BAD_REQUEST:EPERM or BAD_REQUEST:BAD_TYPE
errors in reply to our HA role request.  We can ignore the type in
an effort to be more liberal in what we receive.