commit fe611e412740e45ad2ce4af207fa08c17c84cea2
author Carmelo Cascone <carmelo@opennetworking.org> Wed Apr 11 13:46:25 2018 -0700
committer Thomas Vachuska <tom@opennetworking.org> Wed Apr 11 21:13:31 2018 +0000
tree 4adfc0c1ead4497409e40939cbe40dc90ae8904b
parent fef5c664caff190c18646088a9a48354d4844fc0

Avoid installing ARP packet requests in ReactiveForwarding

The same packet request is installed by proxyarp which is needed by
reactive forwarding to work.

Change-Id: I2c92e8e3eb8eb2515585fbfbcb2633cae9bf0777

apps/fwd/src/main/java/org/onosproject/fwd/ReactiveForwarding.java

diff --git a/apps/fwd/src/main/java/org/onosproject/fwd/ReactiveForwarding.java b/apps/fwd/src/main/java/org/onosproject/fwd/ReactiveForwarding.java
index 09b8533..9917a85 100644
--- a/apps/fwd/src/main/java/org/onosproject/fwd/ReactiveForwarding.java
+++ b/apps/fwd/src/main/java/org/onosproject/fwd/ReactiveForwarding.java
@@ -259,8 +259,6 @@
         TrafficSelector.Builder selector = DefaultTrafficSelector.builder();
         selector.matchEthType(Ethernet.TYPE_IPV4);
         packetService.requestPackets(selector.build(), PacketPriority.REACTIVE, appId);
-        selector.matchEthType(Ethernet.TYPE_ARP);
-        packetService.requestPackets(selector.build(), PacketPriority.REACTIVE, appId);
 
         selector.matchEthType(Ethernet.TYPE_IPV6);
         if (ipv6Forwarding) {
@@ -277,8 +275,6 @@
         TrafficSelector.Builder selector = DefaultTrafficSelector.builder();
         selector.matchEthType(Ethernet.TYPE_IPV4);
         packetService.cancelPackets(selector.build(), PacketPriority.REACTIVE, appId);
-        selector.matchEthType(Ethernet.TYPE_ARP);
-        packetService.cancelPackets(selector.build(), PacketPriority.REACTIVE, appId);
         selector.matchEthType(Ethernet.TYPE_IPV6);
         packetService.cancelPackets(selector.build(), PacketPriority.REACTIVE, appId);
     }