[ovs-dev] [PATCH V2 25/41] userspace: add gre sequence number support.
Greg Rose
gvrose8192 at gmail.com
Fri May 18 01:57:38 UTC 2018
From: William Tu <u9012063 at gmail.com>
The patch adds support for gre sequence number.
Default is disable. When enable with 'options:seq=true',
the outgoing gre packet will have its sequence number
incremented by one.
Signed-off-by: William Tu <u9012063 at gmail.com>
---
lib/netdev-native-tnl.c | 19 ++++++++++++++++++-
lib/netdev-vport.c | 11 ++++++++++-
lib/netdev.h | 3 +++
tests/tunnel-push-pop.at | 8 ++++----
vswitchd/vswitch.xml | 7 +++++++
5 files changed, 42 insertions(+), 6 deletions(-)
diff --git a/lib/netdev-native-tnl.c b/lib/netdev-native-tnl.c
index 5eb3d22..876718a 100644
--- a/lib/netdev-native-tnl.c
+++ b/lib/netdev-native-tnl.c
@@ -436,10 +436,12 @@ err:
}
void
-netdev_gre_push_header(const struct netdev *netdev OVS_UNUSED,
+netdev_gre_push_header(const struct netdev *netdev,
struct dp_packet *packet,
const struct ovs_action_push_tnl *data)
{
+ struct netdev_vport *dev = netdev_vport_cast(netdev);
+ struct netdev_tunnel_config *tnl_cfg;
struct gre_base_hdr *greh;
int ip_tot_size;
@@ -449,6 +451,15 @@ netdev_gre_push_header(const struct netdev *netdev OVS_UNUSED,
ovs_be16 *csum_opt = (ovs_be16 *) (greh + 1);
*csum_opt = csum(greh, ip_tot_size);
}
+
+ if (greh->flags & htons(GRE_SEQ)) {
+ /* Last 4 byte is GRE seqno */
+ int seq_ofs = gre_header_len(greh->flags) - 4;
+ ovs_16aligned_be32 *seq_opt =
+ ALIGNED_CAST(ovs_16aligned_be32 *, (char *)greh + seq_ofs);
+ tnl_cfg = &dev->tnl_cfg;
+ put_16aligned_be32(seq_opt, htonl(tnl_cfg->seqno++));
+ }
}
int
@@ -491,6 +502,12 @@ netdev_gre_build_header(const struct netdev *netdev,
options++;
}
+ if (tnl_cfg->set_seq) {
+ greh->flags |= htons(GRE_SEQ);
+ /* seqno is updated at push header */
+ options++;
+ }
+
ovs_mutex_unlock(&dev->mutex);
hlen = (uint8_t *) options - (uint8_t *) greh;
diff --git a/lib/netdev-vport.c b/lib/netdev-vport.c
index 52aa12d..cc50aa5 100644
--- a/lib/netdev-vport.c
+++ b/lib/netdev-vport.c
@@ -428,7 +428,7 @@ set_tunnel_config(struct netdev *dev_, const struct smap *args, char **errp)
const char *name = netdev_get_name(dev_);
const char *type = netdev_get_type(dev_);
struct ds errors = DS_EMPTY_INITIALIZER;
- bool needs_dst_port, has_csum;
+ bool needs_dst_port, has_csum, has_seq;
uint16_t dst_proto = 0, src_proto = 0;
struct netdev_tunnel_config tnl_cfg;
struct smap_node *node;
@@ -436,6 +436,7 @@ set_tunnel_config(struct netdev *dev_, const struct smap *args, char **errp)
has_csum = strstr(type, "gre") || strstr(type, "geneve") ||
strstr(type, "stt") || strstr(type, "vxlan");
+ has_seq = strstr(type, "gre");
memset(&tnl_cfg, 0, sizeof tnl_cfg);
/* Add a default destination port for tunnel ports if none specified. */
@@ -506,6 +507,10 @@ set_tunnel_config(struct netdev *dev_, const struct smap *args, char **errp)
if (!strcmp(node->value, "true")) {
tnl_cfg.csum = true;
}
+ } else if (!strcmp(node->key, "seq") && has_seq) {
+ if (!strcmp(node->value, "true")) {
+ tnl_cfg.set_seq = true;
+ }
} else if (!strcmp(node->key, "df_default")) {
if (!strcmp(node->value, "false")) {
tnl_cfg.dont_fragment = false;
@@ -709,6 +714,10 @@ get_tunnel_config(const struct netdev *dev, struct smap *args)
smap_add(args, "csum", "true");
}
+ if (tnl_cfg.set_seq) {
+ smap_add(args, "seq", "true");
+ }
+
enum tunnel_layers layers = tunnel_supported_layers(type, &tnl_cfg);
if (tnl_cfg.pt_mode != default_pt_mode(layers)) {
smap_add(args, "packet_type",
diff --git a/lib/netdev.h b/lib/netdev.h
index bd6f45a..e331cac 100644
--- a/lib/netdev.h
+++ b/lib/netdev.h
@@ -127,6 +127,9 @@ struct netdev_tunnel_config {
bool csum;
bool dont_fragment;
enum netdev_pt_mode pt_mode;
+
+ bool set_seq;
+ uint32_t seqno;
};
void netdev_run(void);
diff --git a/tests/tunnel-push-pop.at b/tests/tunnel-push-pop.at
index 10f98c6..4bd412e 100644
--- a/tests/tunnel-push-pop.at
+++ b/tests/tunnel-push-pop.at
@@ -358,7 +358,7 @@ AT_SETUP([tunnel_push_pop - underlay bridge match])
OVS_VSWITCHD_START([add-port br0 p0 -- set Interface p0 type=dummy ofport_request=1 other-config:hwaddr=aa:55:aa:55:00:00])
AT_CHECK([ovs-vsctl add-br int-br -- set bridge int-br datapath_type=dummy], [0])
AT_CHECK([ovs-vsctl add-port int-br t1 -- set Interface t1 type=gre \
- options:remote_ip=1.1.2.92 options:key=456 ofport_request=3], [0])
+ options:remote_ip=1.1.2.92 options:key=456 options:seq=true ofport_request=3], [0])
AT_CHECK([ovs-appctl dpif/show], [0], [dnl
dummy at ovs-dummy: hit:0 missed:0
@@ -367,7 +367,7 @@ dummy at ovs-dummy: hit:0 missed:0
p0 1/1: (dummy)
int-br:
int-br 65534/2: (dummy-internal)
- t1 3/3: (gre: key=456, remote_ip=1.1.2.92)
+ t1 3/3: (gre: key=456, remote_ip=1.1.2.92, seq=true)
])
AT_CHECK([ovs-appctl netdev-dummy/ip4addr br0 1.1.2.88/24], [0], [OK
@@ -390,14 +390,14 @@ AT_CHECK([ovs-ofctl add-flow int-br action=3])
AT_CHECK([ovs-appctl ofproto/trace ovs-dummy 'in_port(2),eth(src=f8:bc:12:44:34:b6,dst=aa:55:aa:55:00:00),eth_type(0x0800),ipv4(src=1.1.3.88,dst=1.1.3.112,proto=17,tos=0,ttl=64,frag=no),udp(src=51283,dst=4789)'], [0], [stdout])
AT_CHECK([tail -1 stdout], [0],
- [Datapath actions: clone(tnl_push(tnl_port(3),header(size=42,type=3,eth(dst=f8:bc:12:44:34:b6,src=aa:55:aa:55:00:00,dl_type=0x0800),ipv4(src=1.1.2.88,dst=1.1.2.92,proto=47,tos=0,ttl=64,frag=0x4000),gre((flags=0x2000,proto=0x6558),key=0x1c8)),out_port(100)),1)
+ [Datapath actions: tnl_push(tnl_port(3),header(size=46,type=3,eth(dst=f8:bc:12:44:34:b6,src=aa:55:aa:55:00:00,dl_type=0x0800),ipv4(src=1.1.2.88,dst=1.1.2.92,proto=47,tos=0,ttl=64,frag=0x4000),gre((flags=0x3000,proto=0x6558),key=0x1c8,seq=0x0)),out_port(100))
])
dnl Verify outer L2 and L3 header flow fields can be matched in the underlay bridge
AT_CHECK([ovs-appctl netdev-dummy/receive int-br '50540000000a5054000000091234'])
AT_CHECK([ovs-ofctl dump-flows br0 | ofctl_strip | sort], [0], [dnl)
n_packets=1, n_bytes=42, priority=1,arp actions=NORMAL
- n_packets=1, n_bytes=56, priority=99,ip,dl_src=aa:55:aa:55:00:00,dl_dst=f8:bc:12:44:34:b6,nw_src=1.1.2.88,nw_dst=1.1.2.92,nw_proto=47,nw_tos=0 actions=NORMAL
+ n_packets=1, n_bytes=60, priority=99,ip,dl_src=aa:55:aa:55:00:00,dl_dst=f8:bc:12:44:34:b6,nw_src=1.1.2.88,nw_dst=1.1.2.92,nw_proto=47,nw_tos=0 actions=NORMAL
NXST_FLOW reply:
])
diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
index 7ab90d5..acc1b03 100644
--- a/vswitchd/vswitch.xml
+++ b/vswitchd/vswitch.xml
@@ -2596,6 +2596,13 @@
</li>
</ul>
</column>
+ <column name="options" key="seq" type='{"type": "boolean"}'>
+ <p>
+ Optional. A 4-byte sequence number field for GRE tunnel only.
+ Default is disabled, set to <code>true</code> to enable.
+ Sequence number is incremented by one on each outgoing packet.
+ </p>
+ </column>
</group>
<group title="Tunnel Options: gre, geneve, and vxlan">
--
1.8.3.1
More information about the dev
mailing list