owenh / rpms / openvswitch

Forked from rpms/openvswitch a month ago
Clone
Open vSwitch CI a00584
diff --git a/lib/dp-packet.h b/lib/dp-packet.h
Open vSwitch CI a00584
index a75b1c5cdb..e816b9f20b 100644
Open vSwitch CI a00584
--- a/lib/dp-packet.h
Open vSwitch CI a00584
+++ b/lib/dp-packet.h
Open vSwitch CI a00584
@@ -529,6 +529,16 @@ dp_packet_inner_l3(const struct dp_packet *b)
Open vSwitch CI a00584
            : NULL;
Open vSwitch CI a00584
 }
Open vSwitch CI a00584
 
Open vSwitch CI a00584
+static inline size_t
Open vSwitch CI a00584
+dp_packet_inner_l3_size(const struct dp_packet *b)
Open vSwitch CI a00584
+{
Open vSwitch CI a00584
+    return OVS_LIKELY(b->inner_l3_ofs != UINT16_MAX)
Open vSwitch CI a00584
+           ? (const char *) dp_packet_tail(b)
Open vSwitch CI a00584
+           - (const char *) dp_packet_inner_l3(b)
Open vSwitch CI a00584
+           - dp_packet_l2_pad_size(b)
Open vSwitch CI a00584
+           : 0;
Open vSwitch CI a00584
+}
Open vSwitch CI a00584
+
Open vSwitch CI a00584
 static inline void *
Open vSwitch CI a00584
 dp_packet_inner_l4(const struct dp_packet *b)
Open vSwitch CI a00584
 {
Open vSwitch CI a00584
@@ -1390,11 +1400,26 @@ dp_packet_hwol_l3_ipv4(const struct dp_packet *b)
Open vSwitch CI a00584
 static inline void
Open vSwitch CI a00584
 dp_packet_ip_set_header_csum(struct dp_packet *p, bool inner)
Open vSwitch CI a00584
 {
Open vSwitch CI a00584
-    struct ip_header *ip = (inner) ? dp_packet_inner_l3(p) : dp_packet_l3(p);
Open vSwitch CI a00584
+    struct ip_header *ip;
Open vSwitch CI a00584
+    size_t l3_size;
Open vSwitch CI a00584
+    size_t ip_len;
Open vSwitch CI a00584
+
Open vSwitch CI a00584
+    if (inner) {
Open vSwitch CI a00584
+        ip = dp_packet_inner_l3(p);
Open vSwitch CI a00584
+        l3_size = dp_packet_inner_l3_size(p);
Open vSwitch CI a00584
+    } else {
Open vSwitch CI a00584
+        ip = dp_packet_l3(p);
Open vSwitch CI a00584
+        l3_size = dp_packet_l3_size(p);
Open vSwitch CI a00584
+    }
Open vSwitch CI a00584
 
Open vSwitch CI a00584
     ovs_assert(ip);
Open vSwitch CI a00584
-    ip->ip_csum = 0;
Open vSwitch CI a00584
-    ip->ip_csum = csum(ip, sizeof *ip);
Open vSwitch CI a00584
+
Open vSwitch CI a00584
+    ip_len = IP_IHL(ip->ip_ihl_ver) * 4;
Open vSwitch CI a00584
+
Open vSwitch CI a00584
+    if (OVS_LIKELY(ip_len >= IP_HEADER_LEN && ip_len < l3_size)) {
Open vSwitch CI a00584
+        ip->ip_csum = 0;
Open vSwitch CI a00584
+        ip->ip_csum = csum(ip, ip_len);
Open vSwitch CI a00584
+    }
Open vSwitch CI a00584
 }
Open vSwitch CI a00584
 
Open vSwitch CI a00584
 /* Returns 'true' if the packet 'p' has good integrity and the
Open vSwitch CI e9b567
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
Open vSwitch CI e9b567
index 02cef6e451..7cced0f226 100644
Open vSwitch CI e9b567
--- a/lib/netdev-dpdk.c
Open vSwitch CI e9b567
+++ b/lib/netdev-dpdk.c
Open vSwitch CI e9b567
@@ -4637,10 +4637,11 @@ netdev_dpdk_get_mempool_info(struct unixctl_conn *conn,
Open vSwitch CI e9b567
                              int argc, const char *argv[],
Open vSwitch CI e9b567
                              void *aux OVS_UNUSED)
Open vSwitch CI e9b567
 {
Open vSwitch CI e9b567
-    size_t size;
Open vSwitch CI e9b567
-    FILE *stream;
Open vSwitch CI e9b567
-    char *response = NULL;
Open vSwitch CI e9b567
     struct netdev *netdev = NULL;
Open vSwitch CI e9b567
+    const char *error = NULL;
Open vSwitch CI e9b567
+    char *response = NULL;
Open vSwitch CI e9b567
+    FILE *stream;
Open vSwitch CI e9b567
+    size_t size;
Open vSwitch CI e9b567
 
Open vSwitch CI e9b567
     if (argc == 2) {
Open vSwitch CI e9b567
         netdev = netdev_from_name(argv[1]);
Open vSwitch CI e9b567
@@ -4664,10 +4665,14 @@ netdev_dpdk_get_mempool_info(struct unixctl_conn *conn,
Open vSwitch CI e9b567
         ovs_mutex_lock(&dev->mutex);
Open vSwitch CI e9b567
         ovs_mutex_lock(&dpdk_mp_mutex);
Open vSwitch CI e9b567
 
Open vSwitch CI e9b567
-        rte_mempool_dump(stream, dev->dpdk_mp->mp);
Open vSwitch CI e9b567
-        fprintf(stream, "    count: avail (%u), in use (%u)\n",
Open vSwitch CI e9b567
-                rte_mempool_avail_count(dev->dpdk_mp->mp),
Open vSwitch CI e9b567
-                rte_mempool_in_use_count(dev->dpdk_mp->mp));
Open vSwitch CI e9b567
+        if (dev->dpdk_mp) {
Open vSwitch CI e9b567
+            rte_mempool_dump(stream, dev->dpdk_mp->mp);
Open vSwitch CI e9b567
+            fprintf(stream, "    count: avail (%u), in use (%u)\n",
Open vSwitch CI e9b567
+                    rte_mempool_avail_count(dev->dpdk_mp->mp),
Open vSwitch CI e9b567
+                    rte_mempool_in_use_count(dev->dpdk_mp->mp));
Open vSwitch CI e9b567
+        } else {
Open vSwitch CI e9b567
+            error = "Not allocated";
Open vSwitch CI e9b567
+        }
Open vSwitch CI e9b567
 
Open vSwitch CI e9b567
         ovs_mutex_unlock(&dpdk_mp_mutex);
Open vSwitch CI e9b567
         ovs_mutex_unlock(&dev->mutex);
Open vSwitch CI e9b567
@@ -4679,7 +4684,11 @@ netdev_dpdk_get_mempool_info(struct unixctl_conn *conn,
Open vSwitch CI e9b567
 
Open vSwitch CI e9b567
     fclose(stream);
Open vSwitch CI e9b567
 
Open vSwitch CI e9b567
-    unixctl_command_reply(conn, response);
Open vSwitch CI e9b567
+    if (error) {
Open vSwitch CI e9b567
+        unixctl_command_reply_error(conn, error);
Open vSwitch CI e9b567
+    } else {
Open vSwitch CI e9b567
+        unixctl_command_reply(conn, response);
Open vSwitch CI e9b567
+    }
Open vSwitch CI e9b567
 out:
Open vSwitch CI e9b567
     free(response);
Open vSwitch CI e9b567
     netdev_close(netdev);
Open vSwitch CI c553f2
diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c
Open vSwitch CI 396a46
index c316238cd5..0cd0850a31 100644
Open vSwitch CI c553f2
--- a/lib/netdev-linux.c
Open vSwitch CI c553f2
+++ b/lib/netdev-linux.c
Open vSwitch CI c553f2
@@ -1061,8 +1061,7 @@ netdev_linux_construct_tap(struct netdev *netdev_)
Open vSwitch CI c553f2
 
Open vSwitch CI c553f2
     if (tap_supports_vnet_hdr
Open vSwitch CI c553f2
         && ioctl(netdev->tap_fd, TUNSETOFFLOAD, oflags) == 0) {
Open vSwitch CI c553f2
-        netdev_->ol_flags |= (NETDEV_TX_OFFLOAD_IPV4_CKSUM
Open vSwitch CI c553f2
-                              | NETDEV_TX_OFFLOAD_TCP_CKSUM
Open vSwitch CI c553f2
+        netdev_->ol_flags |= (NETDEV_TX_OFFLOAD_TCP_CKSUM
Open vSwitch CI c553f2
                               | NETDEV_TX_OFFLOAD_UDP_CKSUM);
Open vSwitch CI c553f2
 
Open vSwitch CI c553f2
         if (userspace_tso_enabled()) {
Open vSwitch CI c553f2
@@ -2510,13 +2509,11 @@ netdev_linux_set_ol(struct netdev *netdev_)
Open vSwitch CI c553f2
         char *string;
Open vSwitch CI c553f2
         uint32_t value;
Open vSwitch CI c553f2
     } t_list[] = {
Open vSwitch CI c553f2
-        {"tx-checksum-ipv4", NETDEV_TX_OFFLOAD_IPV4_CKSUM |
Open vSwitch CI c553f2
-                             NETDEV_TX_OFFLOAD_TCP_CKSUM |
Open vSwitch CI c553f2
+        {"tx-checksum-ipv4", NETDEV_TX_OFFLOAD_TCP_CKSUM |
Open vSwitch CI c553f2
                              NETDEV_TX_OFFLOAD_UDP_CKSUM},
Open vSwitch CI c553f2
         {"tx-checksum-ipv6", NETDEV_TX_OFFLOAD_TCP_CKSUM |
Open vSwitch CI c553f2
                              NETDEV_TX_OFFLOAD_UDP_CKSUM},
Open vSwitch CI c553f2
-        {"tx-checksum-ip-generic", NETDEV_TX_OFFLOAD_IPV4_CKSUM |
Open vSwitch CI c553f2
-                                   NETDEV_TX_OFFLOAD_TCP_CKSUM |
Open vSwitch CI c553f2
+        {"tx-checksum-ip-generic", NETDEV_TX_OFFLOAD_TCP_CKSUM |
Open vSwitch CI c553f2
                                    NETDEV_TX_OFFLOAD_UDP_CKSUM},
Open vSwitch CI c553f2
         {"tx-checksum-sctp", NETDEV_TX_OFFLOAD_SCTP_CKSUM},
Open vSwitch CI c553f2
         {"tx-tcp-segmentation", NETDEV_TX_OFFLOAD_TCP_TSO},
Open vSwitch CI 396a46
@@ -6738,7 +6735,8 @@ get_stats_via_netlink(const struct netdev *netdev_, struct netdev_stats *stats)
Open vSwitch CI 396a46
             struct rtnl_link_stats64 aligned_lstats;
Open vSwitch CI 396a46
 
Open vSwitch CI 396a46
             if (!IS_PTR_ALIGNED(lstats)) {
Open vSwitch CI 396a46
-                memcpy(&aligned_lstats, lstats, sizeof aligned_lstats);
Open vSwitch CI 396a46
+                memcpy(&aligned_lstats, (void *) lstats,
Open vSwitch CI 396a46
+                       sizeof aligned_lstats);
Open vSwitch CI 396a46
                 lstats = &aligned_lstats;
Open vSwitch CI 396a46
             }
Open vSwitch CI 396a46
             netdev_stats_from_rtnl_link_stats64(stats, lstats);
Open vSwitch CI 396a46
@@ -7203,13 +7201,6 @@ netdev_linux_prepend_vnet_hdr(struct dp_packet *b, int mtu)
Open vSwitch CI c553f2
         /* The packet has good L4 checksum. No need to validate again. */
Open vSwitch CI c553f2
         vnet->csum_start = vnet->csum_offset = (OVS_FORCE __virtio16) 0;
Open vSwitch CI c553f2
         vnet->flags = VIRTIO_NET_HDR_F_DATA_VALID;
Open vSwitch CI c553f2
-
Open vSwitch CI c553f2
-        /* It is possible that L4 is good but the IPv4 checksum isn't
Open vSwitch CI c553f2
-         * complete. For example in the case of UDP encapsulation of an ARP
Open vSwitch CI c553f2
-         * packet where the UDP checksum is 0. */
Open vSwitch CI c553f2
-        if (dp_packet_hwol_l3_csum_ipv4_ol(b)) {
Open vSwitch CI c553f2
-            dp_packet_ip_set_header_csum(b, false);
Open vSwitch CI c553f2
-        }
Open vSwitch CI c553f2
     } else if (dp_packet_hwol_tx_l4_checksum(b)) {
Open vSwitch CI c553f2
         /* The csum calculation is offloaded. */
Open vSwitch CI c553f2
         if (dp_packet_hwol_l4_is_tcp(b)) {
Open vSwitch CI 396a46
diff --git a/lib/route-table.c b/lib/route-table.c
Open vSwitch CI 396a46
index f1fe32714e..c6cb21394a 100644
Open vSwitch CI 396a46
--- a/lib/route-table.c
Open vSwitch CI 396a46
+++ b/lib/route-table.c
Open vSwitch CI 396a46
@@ -85,7 +85,7 @@ static bool route_table_valid = false;
Open vSwitch CI 396a46
 
Open vSwitch CI 396a46
 static void route_table_reset(void);
Open vSwitch CI 396a46
 static void route_table_handle_msg(const struct route_table_msg *);
Open vSwitch CI 396a46
-static int route_table_parse(struct ofpbuf *, struct route_table_msg *);
Open vSwitch CI 396a46
+static int route_table_parse(struct ofpbuf *, void *change);
Open vSwitch CI 396a46
 static void route_table_change(const struct route_table_msg *, void *);
Open vSwitch CI 396a46
 static void route_map_clear(void);
Open vSwitch CI 396a46
 
Open vSwitch CI 396a46
@@ -110,8 +110,7 @@ route_table_init(void)
Open vSwitch CI 396a46
     ovs_assert(!route6_notifier);
Open vSwitch CI 396a46
 
Open vSwitch CI 396a46
     ovs_router_init();
Open vSwitch CI 396a46
-    nln = nln_create(NETLINK_ROUTE, (nln_parse_func *) route_table_parse,
Open vSwitch CI 396a46
-                     &rtmsg);
Open vSwitch CI 396a46
+    nln = nln_create(NETLINK_ROUTE, route_table_parse, &rtmsg);
Open vSwitch CI 396a46
 
Open vSwitch CI 396a46
     route_notifier =
Open vSwitch CI 396a46
         nln_notifier_create(nln, RTNLGRP_IPV4_ROUTE,
Open vSwitch CI 396a46
@@ -223,8 +222,9 @@ route_table_reset(void)
Open vSwitch CI 396a46
 /* Return RTNLGRP_IPV4_ROUTE or RTNLGRP_IPV6_ROUTE on success, 0 on parse
Open vSwitch CI 396a46
  * error. */
Open vSwitch CI 396a46
 static int
Open vSwitch CI 396a46
-route_table_parse(struct ofpbuf *buf, struct route_table_msg *change)
Open vSwitch CI 396a46
+route_table_parse(struct ofpbuf *buf, void *change_)
Open vSwitch CI 396a46
 {
Open vSwitch CI 396a46
+    struct route_table_msg *change = change_;
Open vSwitch CI 396a46
     bool parsed, ipv4 = false;
Open vSwitch CI 396a46
 
Open vSwitch CI 396a46
     static const struct nl_policy policy[] = {
Open vSwitch CI b3666f
diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c
Open vSwitch CI b3666f
index 850597b3a4..7506ab5371 100644
Open vSwitch CI b3666f
--- a/ofproto/ofproto-dpif-xlate.c
Open vSwitch CI b3666f
+++ b/ofproto/ofproto-dpif-xlate.c
Open vSwitch CI b3666f
@@ -679,6 +679,7 @@ static size_t count_skb_priorities(const struct xport *);
Open vSwitch CI b3666f
 static bool dscp_from_skb_priority(const struct xport *, uint32_t skb_priority,
Open vSwitch CI b3666f
                                    uint8_t *dscp);
Open vSwitch CI b3666f
 
Open vSwitch CI b3666f
+static bool xlate_resubmit_resource_check(struct xlate_ctx *);
Open vSwitch CI b3666f
 static void xlate_xbridge_init(struct xlate_cfg *, struct xbridge *);
Open vSwitch CI b3666f
 static void xlate_xbundle_init(struct xlate_cfg *, struct xbundle *);
Open vSwitch CI b3666f
 static void xlate_xport_init(struct xlate_cfg *, struct xport *);
Open vSwitch CI b3666f
@@ -3723,6 +3724,10 @@ compose_table_xlate(struct xlate_ctx *ctx, const struct xport *out_dev,
Open vSwitch CI b3666f
     struct ofpact_output output;
Open vSwitch CI b3666f
     struct flow flow;
Open vSwitch CI b3666f
 
Open vSwitch CI b3666f
+    if (!xlate_resubmit_resource_check(ctx)) {
Open vSwitch CI b3666f
+        return 0;
Open vSwitch CI b3666f
+    }
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
     ofpact_init(&output.ofpact, OFPACT_OUTPUT, sizeof output);
Open vSwitch CI b3666f
     flow_extract(packet, &flow);
Open vSwitch CI b3666f
     flow.in_port.ofp_port = out_dev->ofp_port;
Open vSwitch CI b3666f
@@ -3731,7 +3736,8 @@ compose_table_xlate(struct xlate_ctx *ctx, const struct xport *out_dev,
Open vSwitch CI b3666f
 
Open vSwitch CI b3666f
     return ofproto_dpif_execute_actions__(xbridge->ofproto, version, &flow,
Open vSwitch CI b3666f
                                           NULL, &output.ofpact, sizeof output,
Open vSwitch CI b3666f
-                                          ctx->depth, ctx->resubmits, packet);
Open vSwitch CI b3666f
+                                          ctx->depth + 1, ctx->resubmits,
Open vSwitch CI b3666f
+                                          packet);
Open vSwitch CI b3666f
 }
Open vSwitch CI b3666f
 
Open vSwitch CI b3666f
 static void
Open vSwitch CI e9b567
diff --git a/ovsdb/transaction.c b/ovsdb/transaction.c
Open vSwitch CI e9b567
index 65eca64783..98fff1a744 100644
Open vSwitch CI e9b567
--- a/ovsdb/transaction.c
Open vSwitch CI e9b567
+++ b/ovsdb/transaction.c
Open vSwitch CI e9b567
@@ -1090,7 +1090,6 @@ ovsdb_txn_precommit(struct ovsdb_txn *txn)
Open vSwitch CI e9b567
      * was really a no-op. */
Open vSwitch CI e9b567
     error = for_each_txn_row(txn, determine_changes);
Open vSwitch CI e9b567
     if (error) {
Open vSwitch CI e9b567
-        ovsdb_txn_abort(txn);
Open vSwitch CI e9b567
         return OVSDB_WRAP_BUG("can't happen", error);
Open vSwitch CI e9b567
     }
Open vSwitch CI e9b567
     if (ovs_list_is_empty(&txn->txn_tables)) {
Open vSwitch CI a00584
diff --git a/tests/dpif-netdev.at b/tests/dpif-netdev.at
Open vSwitch CI a00584
index bdc24cc307..c16bdd0326 100644
Open vSwitch CI a00584
--- a/tests/dpif-netdev.at
Open vSwitch CI a00584
+++ b/tests/dpif-netdev.at
Open vSwitch CI a00584
@@ -807,6 +807,41 @@ AT_CHECK([ovs-appctl netdev-dummy/receive p1 ${bad_frame}])
Open vSwitch CI a00584
 AT_CHECK([ovs-pcap p2.pcap > p2.pcap.txt 2>&1])
Open vSwitch CI a00584
 AT_CHECK_UNQUOTED([tail -n 1 p2.pcap.txt], [0], [${good_expected}
Open vSwitch CI a00584
 ])
Open vSwitch CI a00584
+
Open vSwitch CI a00584
+dnl Test with IP optional fields in a valid packet.  Note that neither this
Open vSwitch CI a00584
+dnl packet nor the following one contain a correct checksum.  OVS is
Open vSwitch CI a00584
+dnl expected to replace this dummy checksum with a valid one if possible.
Open vSwitch CI a00584
+m4_define([OPT_PKT], m4_join([],
Open vSwitch CI a00584
+dnl eth(dst=aa:aa:aa:aa:aa:aa,src=bb:bb:bb:bb:bb:bb,type=0x0800)
Open vSwitch CI a00584
+[aaaaaaaaaaaabbbbbbbbbbbb0800],
Open vSwitch CI a00584
+dnl ipv4(dst=10.0.0.2,src=10.0.0.1,proto=1,len=60,tot_len=68,csum=0xeeee)
Open vSwitch CI a00584
+[4f000044abab00004001eeee0a0000010a000002],
Open vSwitch CI a00584
+dnl IPv4 Opt: type 7 (Record Route) len 39 + type 0 (EOL).
Open vSwitch CI a00584
+[07270c010203040a000003000000000000000000],
Open vSwitch CI a00584
+[0000000000000000000000000000000000000000],
Open vSwitch CI a00584
+dnl icmp(type=8,code=0), csum 0x3e2f incorrect, should be 0x412f.
Open vSwitch CI a00584
+[08003e2fb6d00000]))
Open vSwitch CI a00584
+
Open vSwitch CI a00584
+dnl IP header indicates optional fields but doesn't contain any.
Open vSwitch CI a00584
+m4_define([MICROGRAM], m4_join([],
Open vSwitch CI a00584
+dnl eth(dst=aa:aa:aa:aa:aa:aa,src=bb:bb:bb:bb:bb:bb,type=0x0800)
Open vSwitch CI a00584
+[aaaaaaaaaaaabbbbbbbbbbbb0800],
Open vSwitch CI a00584
+dnl ipv4(dst=10.0.0.2,src=10.0.0.1,proto=1,len=60,tot_len=68,csum=0xeeee)
Open vSwitch CI a00584
+[4f000044abab00004001eeee0a0000010a000002]))
Open vSwitch CI a00584
+
Open vSwitch CI a00584
+AT_CHECK([ovs-vsctl set Interface p1 options:ol_ip_csum=true])
Open vSwitch CI a00584
+AT_CHECK([ovs-vsctl set Interface p1 options:ol_ip_csum_set_good=true])
Open vSwitch CI a00584
+AT_CHECK([ovs-appctl netdev-dummy/receive p1 OPT_PKT])
Open vSwitch CI a00584
+AT_CHECK([ovs-appctl netdev-dummy/receive p1 MICROGRAM])
Open vSwitch CI a00584
+AT_CHECK([ovs-pcap p2.pcap > p2.pcap.txt 2>&1])
Open vSwitch CI a00584
+
Open vSwitch CI a00584
+dnl Build the expected modified packets.  The first packet has a valid IPv4
Open vSwitch CI a00584
+dnl checksum and modified destination IP address.  The second packet isn't
Open vSwitch CI a00584
+dnl expected to change.
Open vSwitch CI a00584
+AT_CHECK([echo "OPT_PKT" | sed -e "s/0a000002/c0a80101/" -e "s/eeee/dd2e/" > expout])
Open vSwitch CI a00584
+AT_CHECK([echo "MICROGRAM" >> expout])
Open vSwitch CI a00584
+AT_CHECK([tail -n 2 p2.pcap.txt], [0], [expout])
Open vSwitch CI a00584
+
Open vSwitch CI a00584
 OVS_VSWITCHD_STOP
Open vSwitch CI a00584
 AT_CLEANUP
Open vSwitch CI a00584
 
Open vSwitch CI e9b567
diff --git a/tests/system-dpdk.at b/tests/system-dpdk.at
Open vSwitch CI e9b567
index 1c97bf7772..e79c755657 100644
Open vSwitch CI e9b567
--- a/tests/system-dpdk.at
Open vSwitch CI e9b567
+++ b/tests/system-dpdk.at
Open vSwitch CI e9b567
@@ -88,6 +88,12 @@ ADD_VHOST_USER_CLIENT_PORT([br10], [dpdkvhostuserclient0], [$OVS_RUNDIR/dpdkvhos
Open vSwitch CI e9b567
 AT_CHECK([ovs-vsctl show], [], [stdout])
Open vSwitch CI e9b567
 sleep 2
Open vSwitch CI e9b567
 
Open vSwitch CI e9b567
+dnl Check that no mempool was allocated.
Open vSwitch CI e9b567
+AT_CHECK([ovs-appctl netdev-dpdk/get-mempool-info dpdkvhostuserclient0], [2], [], [dnl
Open vSwitch CI e9b567
+Not allocated
Open vSwitch CI e9b567
+ovs-appctl: ovs-vswitchd: server returned an error
Open vSwitch CI e9b567
+])
Open vSwitch CI e9b567
+
Open vSwitch CI e9b567
 dnl Clean up
Open vSwitch CI e9b567
 AT_CHECK([ovs-vsctl del-port br10 dpdkvhostuserclient0], [], [stdout], [stderr])
Open vSwitch CI e9b567
 OVS_DPDK_STOP_VSWITCHD(["dnl
Open vSwitch CI b3666f
diff --git a/tests/tunnel-push-pop-ipv6.at b/tests/tunnel-push-pop-ipv6.at
Open vSwitch CI b3666f
index 3edec5fbca..824a226b21 100644
Open vSwitch CI b3666f
--- a/tests/tunnel-push-pop-ipv6.at
Open vSwitch CI b3666f
+++ b/tests/tunnel-push-pop-ipv6.at
Open vSwitch CI b3666f
@@ -825,3 +825,86 @@ hash(l4(0)),recirc(0x2)
Open vSwitch CI b3666f
 
Open vSwitch CI b3666f
 OVS_VSWITCHD_STOP
Open vSwitch CI b3666f
 AT_CLEANUP
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+AT_SETUP([tunnel_push_pop_ipv6 - Mirror over tunnels])
Open vSwitch CI b3666f
+OVS_VSWITCHD_START([dnl
Open vSwitch CI b3666f
+    add-br br-ext -- set bridge br-ext datapath_type=dummy \
Open vSwitch CI b3666f
+        other-config:hwaddr=aa:55:aa:55:00:00 \
Open vSwitch CI b3666f
+    -- add-port br0 t1 -- set Interface t1 type=geneve \
Open vSwitch CI b3666f
+        options:remote_ip=2001:cafe::91 \
Open vSwitch CI b3666f
+    -- add-port br0 t2 -- set Interface t2 type=erspan \
Open vSwitch CI b3666f
+        options:remote_ip=2001:cafe::92 options:key=flow \
Open vSwitch CI b3666f
+        options:erspan_ver=1 options:erspan_idx=flow \
Open vSwitch CI b3666f
+    -- add-port br0 p0 -- set Interface p0 type=dummy \
Open vSwitch CI b3666f
+    -- add-port br0 p1 -- set Interface p1 type=dummy \
Open vSwitch CI b3666f
+    -- add-port br-ext p-ext -- set Interface p-ext type=dummy \
Open vSwitch CI b3666f
+        options:pcap=ext.pcap])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Configure mirroring over the UDP and ERSPAN tunnels.
Open vSwitch CI b3666f
+AT_CHECK([dnl
Open vSwitch CI b3666f
+    ovs-vsctl \
Open vSwitch CI b3666f
+        set Bridge br0 mirrors=@m1,@m2 -- \
Open vSwitch CI b3666f
+        --id=@t1 get Port t1 -- \
Open vSwitch CI b3666f
+        --id=@t2 get Port t2 -- \
Open vSwitch CI b3666f
+        --id=@m1 create Mirror name=vxlan select_all=true output_port=@t1 -- \
Open vSwitch CI b3666f
+        --id=@m2 create Mirror name=erspan select_all=true output_port=@t2],
Open vSwitch CI b3666f
+    [0], [stdout])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+AT_CHECK([ovs-ofctl add-flow br-ext actions=normal])
Open vSwitch CI b3666f
+AT_CHECK([ovs-ofctl add-flow br0 actions=normal])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Make sure ephemeral ports stay static across tests.
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl tnl/egress_port_range 35190 35190], [0], [OK
Open vSwitch CI b3666f
+])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Setup an IP address.
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl netdev-dummy/ip6addr br-ext 2001:cafe::90/64], [0], [OK
Open vSwitch CI b3666f
+])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Send two ND packets to set the tunnel's port and mac address.
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl netdev-dummy/receive p-ext dnl
Open vSwitch CI b3666f
+ 'eth(src=f8:bc:12:44:34:b3,dst=aa:55:aa:55:00:00),eth_type(0x86dd),dnl
Open vSwitch CI b3666f
+  ipv6(src=2001:cafe::91,dst=2001:cafe::90,label=0,proto=58,tclass=0,hlimit=255,frag=no),dnl
Open vSwitch CI b3666f
+  icmpv6(type=136,code=0),dnl
Open vSwitch CI b3666f
+  nd(target=2001:cafe::91,sll=00:00:00:00:00:00,tll=f8:bc:12:44:34:b3)'
Open vSwitch CI b3666f
+])
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl netdev-dummy/receive p-ext dnl
Open vSwitch CI b3666f
+ 'eth(src=f8:bc:12:44:34:b6,dst=aa:55:aa:55:00:00),eth_type(0x86dd),dnl
Open vSwitch CI b3666f
+  ipv6(src=2001:cafe::92,dst=2001:cafe::90,label=0,proto=58,tclass=0,hlimit=255,frag=no),dnl
Open vSwitch CI b3666f
+  icmpv6(type=136,code=0),dnl
Open vSwitch CI b3666f
+  nd(target=2001:cafe::92,sll=00:00:00:00:00:00,tll=f8:bc:12:44:34:b6)'
Open vSwitch CI b3666f
+])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+m4_define([FLOW], [m4_join([,],
Open vSwitch CI b3666f
+  [in_port(p1)],
Open vSwitch CI b3666f
+  [eth(src=50:54:00:00:00:05,dst=50:54:00:00:00:07),eth_type(0x0800)],
Open vSwitch CI b3666f
+  [ipv4(src=192.168.0.1,dst=192.168.0.2,proto=1,tos=0,ttl=128,frag=no)],
Open vSwitch CI b3666f
+  [icmp(type=8,code=0)])])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+m4_define([ERSPAN_ACT], [m4_join([,],
Open vSwitch CI b3666f
+  [clone(tnl_push(tnl_port(erspan_sys)],
Open vSwitch CI b3666f
+           [header(size=70,type=108],
Open vSwitch CI b3666f
+                  [eth(dst=f8:bc:12:44:34:b6,src=aa:55:aa:55:00:00,dl_type=0x86dd)],
Open vSwitch CI b3666f
+                  [ipv6(src=2001:cafe::90,dst=2001:cafe::92,label=0,proto=47,tclass=0x0,hlimit=64)],
Open vSwitch CI b3666f
+                  [erspan(ver=1,sid=0x0,idx=0x0))],
Open vSwitch CI b3666f
+           [out_port(br-ext))],
Open vSwitch CI b3666f
+         [p-ext)])])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+m4_define([GENEVE_ACT], [m4_join([,],
Open vSwitch CI b3666f
+  [clone(tnl_push(tnl_port(genev_sys_6081)],
Open vSwitch CI b3666f
+           [header(size=70,type=5],
Open vSwitch CI b3666f
+                   [eth(dst=f8:bc:12:44:34:b3,src=aa:55:aa:55:00:00,dl_type=0x86dd)],
Open vSwitch CI b3666f
+                   [ipv6(src=2001:cafe::90,dst=2001:cafe::91,label=0,proto=17,tclass=0x0,hlimit=64)],
Open vSwitch CI b3666f
+                   [udp(src=0,dst=6081,csum=0xffff)],
Open vSwitch CI b3666f
+                   [geneve(vni=0x0))],
Open vSwitch CI b3666f
+           [out_port(br-ext))],
Open vSwitch CI b3666f
+         [p-ext)])])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Verify packet is mirrored to both tunnels.  Tunnel actions may happen
Open vSwitch CI b3666f
+dnl in any order.
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl ofproto/trace --names ovs-dummy "FLOW"], [0], [stdout])
Open vSwitch CI b3666f
+AT_CHECK([grep -q "ERSPAN_ACT" stdout])
Open vSwitch CI b3666f
+AT_CHECK([grep -q "GENEVE_ACT" stdout])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+OVS_VSWITCHD_STOP
Open vSwitch CI b3666f
+AT_CLEANUP
Open vSwitch CI b3666f
diff --git a/tests/tunnel-push-pop.at b/tests/tunnel-push-pop.at
Open vSwitch CI b3666f
index 7ec4c31ab2..99b1b02bf9 100644
Open vSwitch CI b3666f
--- a/tests/tunnel-push-pop.at
Open vSwitch CI b3666f
+++ b/tests/tunnel-push-pop.at
Open vSwitch CI b3666f
@@ -1259,3 +1259,82 @@ hash(l4(0)),recirc(0x2)
Open vSwitch CI b3666f
 
Open vSwitch CI b3666f
 OVS_VSWITCHD_STOP
Open vSwitch CI b3666f
 AT_CLEANUP
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+AT_SETUP([tunnel_push_pop - Mirror over tunnels])
Open vSwitch CI b3666f
+OVS_VSWITCHD_START([dnl
Open vSwitch CI b3666f
+    add-br br-ext -- set bridge br-ext datapath_type=dummy \
Open vSwitch CI b3666f
+        other-config:hwaddr=aa:55:aa:55:00:00 \
Open vSwitch CI b3666f
+    -- add-port br0 t1 -- set Interface t1 type=geneve \
Open vSwitch CI b3666f
+        options:remote_ip=1.1.1.1 \
Open vSwitch CI b3666f
+    -- add-port br0 t2 -- set Interface t2 type=erspan \
Open vSwitch CI b3666f
+        options:remote_ip=1.1.1.2 options:key=flow options:erspan_ver=1 \
Open vSwitch CI b3666f
+        options:erspan_idx=flow \
Open vSwitch CI b3666f
+    -- add-port br0 p0 -- set Interface p0 type=dummy \
Open vSwitch CI b3666f
+    -- add-port br0 p1 -- set Interface p1 type=dummy \
Open vSwitch CI b3666f
+    -- add-port br-ext p-ext -- set Interface p-ext type=dummy \
Open vSwitch CI b3666f
+        options:pcap=ext.pcap])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Configure mirroring over the UDP and ERSPAN tunnels.
Open vSwitch CI b3666f
+AT_CHECK([dnl
Open vSwitch CI b3666f
+    ovs-vsctl \
Open vSwitch CI b3666f
+        set Bridge br0 mirrors=@m1,@m2 -- \
Open vSwitch CI b3666f
+        --id=@t1 get Port t1 -- \
Open vSwitch CI b3666f
+        --id=@t2 get Port t2 -- \
Open vSwitch CI b3666f
+        --id=@m1 create Mirror name=vxlan select_all=true output_port=@t1 -- \
Open vSwitch CI b3666f
+        --id=@m2 create Mirror name=erspan select_all=true output_port=@t2],
Open vSwitch CI b3666f
+    [0], [stdout])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+AT_CHECK([ovs-ofctl add-flow br-ext actions=normal])
Open vSwitch CI b3666f
+AT_CHECK([ovs-ofctl add-flow br0 actions=normal])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Make sure ephemeral ports stay static across tests.
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl tnl/egress_port_range 35190 35190], [0], [OK
Open vSwitch CI b3666f
+])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Setup an IP address for the local side of the tunnel.
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl netdev-dummy/ip4addr br-ext 1.1.1.3/24], [0], [OK
Open vSwitch CI b3666f
+])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Send two arp replies to populate arp table with tunnel remote endpoints.
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl netdev-dummy/receive p-ext dnl
Open vSwitch CI b3666f
+ 'eth(src=f8:bc:12:44:34:b6,dst=ff:ff:ff:ff:ff:ff),eth_type(0x0806),dnl
Open vSwitch CI b3666f
+  arp(sip=1.1.1.1,tip=1.1.1.3,op=2,sha=f8:bc:12:44:34:b6,tha=00:00:00:00:00:00)'
Open vSwitch CI b3666f
+])
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl netdev-dummy/receive p-ext dnl
Open vSwitch CI b3666f
+ 'eth(src=f8:bc:12:44:34:b3,dst=ff:ff:ff:ff:ff:ff),eth_type(0x0806),dnl
Open vSwitch CI b3666f
+  arp(sip=1.1.1.2,tip=1.1.1.3,op=2,sha=f8:bc:12:44:34:b3,tha=00:00:00:00:00:00)'
Open vSwitch CI b3666f
+])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+m4_define([FLOW], [m4_join([,],
Open vSwitch CI b3666f
+  [in_port(p1)],
Open vSwitch CI b3666f
+  [eth(src=50:54:00:00:00:05,dst=50:54:00:00:00:07),eth_type(0x0800)],
Open vSwitch CI b3666f
+  [ipv4(src=192.168.0.1,dst=192.168.0.2,proto=1,tos=0,ttl=128,frag=no)],
Open vSwitch CI b3666f
+  [icmp(type=8,code=0)])])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+m4_define([ERSPAN_ACT], [m4_join([,],
Open vSwitch CI b3666f
+  [clone(tnl_push(tnl_port(erspan_sys)],
Open vSwitch CI b3666f
+           [header(size=50,type=107],
Open vSwitch CI b3666f
+                  [eth(dst=f8:bc:12:44:34:b3,src=aa:55:aa:55:00:00,dl_type=0x0800)],
Open vSwitch CI b3666f
+                  [ipv4(src=1.1.1.3,dst=1.1.1.2,proto=47,tos=0,ttl=64,frag=0x4000)],
Open vSwitch CI b3666f
+                  [erspan(ver=1,sid=0x0,idx=0x0))],
Open vSwitch CI b3666f
+           [out_port(br-ext))],
Open vSwitch CI b3666f
+         [p-ext)])])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+m4_define([GENEVE_ACT], [m4_join([,],
Open vSwitch CI b3666f
+  [clone(tnl_push(tnl_port(genev_sys_6081)],
Open vSwitch CI b3666f
+           [header(size=50,type=5],
Open vSwitch CI b3666f
+                   [eth(dst=f8:bc:12:44:34:b6,src=aa:55:aa:55:00:00,dl_type=0x0800)],
Open vSwitch CI b3666f
+                   [ipv4(src=1.1.1.3,dst=1.1.1.1,proto=17,tos=0,ttl=64,frag=0x4000)],
Open vSwitch CI b3666f
+                   [udp(src=0,dst=6081,csum=0x0)],
Open vSwitch CI b3666f
+                   [geneve(vni=0x0))],
Open vSwitch CI b3666f
+           [out_port(br-ext))],
Open vSwitch CI b3666f
+         [p-ext)])])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+dnl Verify packet is mirrored to both tunnels.  Tunnel actions may happen
Open vSwitch CI b3666f
+dnl in any order.
Open vSwitch CI b3666f
+AT_CHECK([ovs-appctl ofproto/trace --names ovs-dummy "FLOW"], [0], [stdout])
Open vSwitch CI b3666f
+AT_CHECK([grep -q "ERSPAN_ACT" stdout])
Open vSwitch CI b3666f
+AT_CHECK([grep -q "GENEVE_ACT" stdout])
Open vSwitch CI b3666f
+
Open vSwitch CI b3666f
+OVS_VSWITCHD_STOP
Open vSwitch CI b3666f
+AT_CLEANUP