in-band: Use OFPP_NORMAL instead of OFPP_LOCAL for traffic to local port.

If we use OFPP_LOCAL then the code that implements the normal action won't
get a chance to learn from these packets.

This change seems somewhat risky, since in-band control is so touchy, which
is why it is broken out as a separate commit that can easily be reverted.

diff --git a/secchan/in-band.c b/secchan/in-band.c
index b84a775110a39381c46b57bd732f052d57427901..b6b51d33fce93289f8a3ab561ab26911891b97c8 100644 (file)
--- a/secchan/in-band.c
+++ b/secchan/in-band.c
@@ -256,7 +256,7 @@ in_band_run(struct in_band *in_band)
         memset(&flow, 0, sizeof flow);
         memcpy(flow.dl_dst, local_mac, ETH_ADDR_LEN);
         setup_flow(in_band, IBR_TO_LOCAL_PORT, &flow, OFPFW_DL_DST,
-                   OFPP_LOCAL);
+                   OFPP_NORMAL);
     } else {
         drop_flow(in_band, IBR_TO_LOCAL_PORT);
     }