ofproto: Use RCU to protect rule_actions.
[sliver-openvswitch.git] / ofproto / tunnel.c
1 /* Copyright (c) 2013, 2014 Nicira, Inc.
2  *
3  * Licensed under the Apache License, Version 2.0 (the "License");
4  * you may not use this file except in compliance with the License.
5  * You may obtain a copy of the License at:
6  *
7  *     http://www.apache.org/licenses/LICENSE-2.0
8  *
9  * Unless required by applicable law or agreed to in writing, software
10  * distributed under the License is distributed on an "AS IS" BASIS,
11  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12  * See the License for the specific language governing permissions and
13  * limitations under the License. */
14
15 #include <config.h>
16 #include "tunnel.h"
17
18 #include <errno.h>
19
20 #include "byte-order.h"
21 #include "connectivity.h"
22 #include "dynamic-string.h"
23 #include "hash.h"
24 #include "hmap.h"
25 #include "netdev.h"
26 #include "odp-util.h"
27 #include "packets.h"
28 #include "seq.h"
29 #include "smap.h"
30 #include "socket-util.h"
31 #include "tunnel.h"
32 #include "vlog.h"
33
34 VLOG_DEFINE_THIS_MODULE(tunnel);
35
36 /* skb mark used for IPsec tunnel packets */
37 #define IPSEC_MARK 1
38
39 struct tnl_match {
40     ovs_be64 in_key;
41     ovs_be32 ip_src;
42     ovs_be32 ip_dst;
43     odp_port_t odp_port;
44     uint32_t pkt_mark;
45     bool in_key_flow;
46     bool ip_src_flow;
47     bool ip_dst_flow;
48 };
49
50 struct tnl_port {
51     struct hmap_node ofport_node;
52     struct hmap_node match_node;
53
54     const struct ofport_dpif *ofport;
55     unsigned int change_seq;
56     struct netdev *netdev;
57
58     struct tnl_match match;
59 };
60
61 static struct ovs_rwlock rwlock = OVS_RWLOCK_INITIALIZER;
62
63 /* Tunnel matches.
64  *
65  * This module maps packets received over tunnel protocols to vports.  The
66  * tunnel protocol and, for some protocols, tunnel-specific information (e.g.,
67  * for VXLAN, the UDP destination port number) are always use as part of the
68  * mapping.  Which other fields are used for the mapping depends on the vports
69  * themselves (the parenthesized notations refer to "struct tnl_match" fields):
70  *
71  *     - in_key: A vport may match a specific tunnel ID (in_key_flow == false)
72  *       or arrange for the tunnel ID to be matched as tunnel.tun_id in the
73  *       OpenFlow flow (in_key_flow == true).
74  *
75  *     - ip_dst: A vport may match a specific destination IP address
76  *       (ip_dst_flow == false) or arrange for the destination IP to be matched
77  *       as tunnel.ip_dst in the OpenFlow flow (ip_dst_flow == true).
78  *
79  *     - ip_src: A vport may match a specific IP source address (ip_src_flow ==
80  *       false, ip_src != 0), wildcard all source addresses (ip_src_flow ==
81  *       false, ip_src == 0), or arrange for the IP source address to be
82  *       handled in the OpenFlow flow table (ip_src_flow == true).
83  *
84  * Thus, there are 2 * 2 * 3 == 12 possible ways a vport can match against a
85  * tunnel packet.  We number the possibilities for each field in increasing
86  * order as listed in each bullet above.  We order the 12 overall combinations
87  * in lexicographic order considering in_key first, then ip_dst, then
88  * ip_src. */
89 #define N_MATCH_TYPES (2 * 2 * 3)
90
91 /* The three possibilities (see above) for vport ip_src matches. */
92 enum ip_src_type {
93     IP_SRC_CFG,             /* ip_src must equal configured address. */
94     IP_SRC_ANY,             /* Any ip_src is acceptable. */
95     IP_SRC_FLOW             /* ip_src is handled in flow table. */
96 };
97
98 /* Each hmap contains "struct tnl_port"s.
99  * The index is a combination of how each of the fields listed under "Tunnel
100  * matches" above matches, see the final paragraph for ordering. */
101 static struct hmap *tnl_match_maps[N_MATCH_TYPES] OVS_GUARDED_BY(rwlock);
102 static struct hmap **tnl_match_map(const struct tnl_match *);
103
104 static struct hmap ofport_map__ = HMAP_INITIALIZER(&ofport_map__);
105 static struct hmap *ofport_map OVS_GUARDED_BY(rwlock) = &ofport_map__;
106
107 static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
108 static struct vlog_rate_limit dbg_rl = VLOG_RATE_LIMIT_INIT(60, 60);
109
110 static struct tnl_port *tnl_find(const struct flow *) OVS_REQ_RDLOCK(rwlock);
111 static struct tnl_port *tnl_find_exact(struct tnl_match *, struct hmap *)
112     OVS_REQ_RDLOCK(rwlock);
113 static struct tnl_port *tnl_find_ofport(const struct ofport_dpif *)
114     OVS_REQ_RDLOCK(rwlock);
115
116 static uint32_t tnl_hash(struct tnl_match *);
117 static void tnl_match_fmt(const struct tnl_match *, struct ds *);
118 static char *tnl_port_fmt(const struct tnl_port *) OVS_REQ_RDLOCK(rwlock);
119 static void tnl_port_mod_log(const struct tnl_port *, const char *action)
120     OVS_REQ_RDLOCK(rwlock);
121 static const char *tnl_port_get_name(const struct tnl_port *)
122     OVS_REQ_RDLOCK(rwlock);
123 static void tnl_port_del__(const struct ofport_dpif *) OVS_REQ_WRLOCK(rwlock);
124
125 static bool
126 tnl_port_add__(const struct ofport_dpif *ofport, const struct netdev *netdev,
127                odp_port_t odp_port, bool warn)
128     OVS_REQ_WRLOCK(rwlock)
129 {
130     const struct netdev_tunnel_config *cfg;
131     struct tnl_port *existing_port;
132     struct tnl_port *tnl_port;
133     struct hmap **map;
134
135     cfg = netdev_get_tunnel_config(netdev);
136     ovs_assert(cfg);
137
138     tnl_port = xzalloc(sizeof *tnl_port);
139     tnl_port->ofport = ofport;
140     tnl_port->netdev = netdev_ref(netdev);
141     tnl_port->change_seq = seq_read(connectivity_seq_get());
142
143     tnl_port->match.in_key = cfg->in_key;
144     tnl_port->match.ip_src = cfg->ip_src;
145     tnl_port->match.ip_dst = cfg->ip_dst;
146     tnl_port->match.ip_src_flow = cfg->ip_src_flow;
147     tnl_port->match.ip_dst_flow = cfg->ip_dst_flow;
148     tnl_port->match.pkt_mark = cfg->ipsec ? IPSEC_MARK : 0;
149     tnl_port->match.in_key_flow = cfg->in_key_flow;
150     tnl_port->match.odp_port = odp_port;
151
152     map = tnl_match_map(&tnl_port->match);
153     existing_port = tnl_find_exact(&tnl_port->match, *map);
154     if (existing_port) {
155         if (warn) {
156             struct ds ds = DS_EMPTY_INITIALIZER;
157             tnl_match_fmt(&tnl_port->match, &ds);
158             VLOG_WARN("%s: attempting to add tunnel port with same config as "
159                       "port '%s' (%s)", tnl_port_get_name(tnl_port),
160                       tnl_port_get_name(existing_port), ds_cstr(&ds));
161             ds_destroy(&ds);
162             free(tnl_port);
163         }
164         return false;
165     }
166
167     hmap_insert(ofport_map, &tnl_port->ofport_node, hash_pointer(ofport, 0));
168
169     if (!*map) {
170         *map = xmalloc(sizeof **map);
171         hmap_init(*map);
172     }
173     hmap_insert(*map, &tnl_port->match_node, tnl_hash(&tnl_port->match));
174     tnl_port_mod_log(tnl_port, "adding");
175     return true;
176 }
177
178 /* Adds 'ofport' to the module with datapath port number 'odp_port'. 'ofport's
179  * must be added before they can be used by the module. 'ofport' must be a
180  * tunnel. */
181 void
182 tnl_port_add(const struct ofport_dpif *ofport, const struct netdev *netdev,
183              odp_port_t odp_port) OVS_EXCLUDED(rwlock)
184 {
185     ovs_rwlock_wrlock(&rwlock);
186     tnl_port_add__(ofport, netdev, odp_port, true);
187     ovs_rwlock_unlock(&rwlock);
188 }
189
190 /* Checks if the tunnel represented by 'ofport' reconfiguration due to changes
191  * in its netdev_tunnel_config.  If it does, returns true. Otherwise, returns
192  * false.  'ofport' and 'odp_port' should be the same as would be passed to
193  * tnl_port_add(). */
194 bool
195 tnl_port_reconfigure(const struct ofport_dpif *ofport,
196                      const struct netdev *netdev, odp_port_t odp_port)
197     OVS_EXCLUDED(rwlock)
198 {
199     struct tnl_port *tnl_port;
200     bool changed = false;
201
202     ovs_rwlock_wrlock(&rwlock);
203     tnl_port = tnl_find_ofport(ofport);
204     if (!tnl_port) {
205         changed = tnl_port_add__(ofport, netdev, odp_port, false);
206     } else if (tnl_port->netdev != netdev
207                || tnl_port->match.odp_port != odp_port
208                || tnl_port->change_seq != seq_read(connectivity_seq_get())) {
209         VLOG_DBG("reconfiguring %s", tnl_port_get_name(tnl_port));
210         tnl_port_del__(ofport);
211         tnl_port_add__(ofport, netdev, odp_port, true);
212         changed = true;
213     }
214     ovs_rwlock_unlock(&rwlock);
215     return changed;
216 }
217
218 static void
219 tnl_port_del__(const struct ofport_dpif *ofport) OVS_REQ_WRLOCK(rwlock)
220 {
221     struct tnl_port *tnl_port;
222
223     if (!ofport) {
224         return;
225     }
226
227     tnl_port = tnl_find_ofport(ofport);
228     if (tnl_port) {
229         struct hmap **map;
230
231         tnl_port_mod_log(tnl_port, "removing");
232         map = tnl_match_map(&tnl_port->match);
233         hmap_remove(*map, &tnl_port->match_node);
234         if (hmap_is_empty(*map)) {
235             hmap_destroy(*map);
236             free(*map);
237             *map = NULL;
238         }
239         hmap_remove(ofport_map, &tnl_port->ofport_node);
240         netdev_close(tnl_port->netdev);
241         free(tnl_port);
242     }
243 }
244
245 /* Removes 'ofport' from the module. */
246 void
247 tnl_port_del(const struct ofport_dpif *ofport) OVS_EXCLUDED(rwlock)
248 {
249     ovs_rwlock_wrlock(&rwlock);
250     tnl_port_del__(ofport);
251     ovs_rwlock_unlock(&rwlock);
252 }
253
254 /* Looks in the table of tunnels for a tunnel matching the metadata in 'flow'.
255  * Returns the 'ofport' corresponding to the new in_port, or a null pointer if
256  * none is found.
257  *
258  * Callers should verify that 'flow' needs to be received by calling
259  * tnl_port_should_receive() before this function. */
260 const struct ofport_dpif *
261 tnl_port_receive(const struct flow *flow) OVS_EXCLUDED(rwlock)
262 {
263     char *pre_flow_str = NULL;
264     const struct ofport_dpif *ofport;
265     struct tnl_port *tnl_port;
266
267     ovs_rwlock_rdlock(&rwlock);
268     tnl_port = tnl_find(flow);
269     ofport = tnl_port ? tnl_port->ofport : NULL;
270     if (!tnl_port) {
271         char *flow_str = flow_to_string(flow);
272
273         VLOG_WARN_RL(&rl, "receive tunnel port not found (%s)", flow_str);
274         free(flow_str);
275         goto out;
276     }
277
278     if (!VLOG_DROP_DBG(&dbg_rl)) {
279         pre_flow_str = flow_to_string(flow);
280     }
281
282     if (pre_flow_str) {
283         char *post_flow_str = flow_to_string(flow);
284         char *tnl_str = tnl_port_fmt(tnl_port);
285         VLOG_DBG("flow received\n"
286                  "%s"
287                  " pre: %s\n"
288                  "post: %s",
289                  tnl_str, pre_flow_str, post_flow_str);
290         free(tnl_str);
291         free(pre_flow_str);
292         free(post_flow_str);
293     }
294
295 out:
296     ovs_rwlock_unlock(&rwlock);
297     return ofport;
298 }
299
300 static bool
301 tnl_ecn_ok(const struct flow *base_flow, struct flow *flow)
302 {
303     if (is_ip_any(base_flow)
304         && (flow->tunnel.ip_tos & IP_ECN_MASK) == IP_ECN_CE) {
305         if ((base_flow->nw_tos & IP_ECN_MASK) == IP_ECN_NOT_ECT) {
306             VLOG_WARN_RL(&rl, "dropping tunnel packet marked ECN CE"
307                          " but is not ECN capable");
308             return false;
309         } else {
310             /* Set the ECN CE value in the tunneled packet. */
311             flow->nw_tos |= IP_ECN_CE;
312         }
313     }
314
315     return true;
316 }
317
318 /* Should be called at the beginning of action translation to initialize
319  * wildcards and perform any actions based on receiving on tunnel port.
320  *
321  * Returns false if the packet must be dropped. */
322 bool
323 tnl_xlate_init(const struct flow *base_flow, struct flow *flow,
324                struct flow_wildcards *wc)
325 {
326     if (tnl_port_should_receive(flow)) {
327         wc->masks.tunnel.tun_id = OVS_BE64_MAX;
328         wc->masks.tunnel.ip_src = OVS_BE32_MAX;
329         wc->masks.tunnel.ip_dst = OVS_BE32_MAX;
330         wc->masks.tunnel.flags = (FLOW_TNL_F_DONT_FRAGMENT |
331                                   FLOW_TNL_F_CSUM |
332                                   FLOW_TNL_F_KEY);
333         wc->masks.tunnel.ip_tos = UINT8_MAX;
334         wc->masks.tunnel.ip_ttl = UINT8_MAX;
335
336         memset(&wc->masks.pkt_mark, 0xff, sizeof wc->masks.pkt_mark);
337
338         if (!tnl_ecn_ok(base_flow, flow)) {
339             return false;
340         }
341
342         flow->pkt_mark &= ~IPSEC_MARK;
343     }
344
345     return true;
346 }
347
348 /* Given that 'flow' should be output to the ofport corresponding to
349  * 'tnl_port', updates 'flow''s tunnel headers and returns the actual datapath
350  * port that the output should happen on.  May return ODPP_NONE if the output
351  * shouldn't occur. */
352 odp_port_t
353 tnl_port_send(const struct ofport_dpif *ofport, struct flow *flow,
354               struct flow_wildcards *wc) OVS_EXCLUDED(rwlock)
355 {
356     const struct netdev_tunnel_config *cfg;
357     struct tnl_port *tnl_port;
358     char *pre_flow_str = NULL;
359     odp_port_t out_port;
360
361     ovs_rwlock_rdlock(&rwlock);
362     tnl_port = tnl_find_ofport(ofport);
363     out_port = tnl_port ? tnl_port->match.odp_port : ODPP_NONE;
364     if (!tnl_port) {
365         goto out;
366     }
367
368     cfg = netdev_get_tunnel_config(tnl_port->netdev);
369     ovs_assert(cfg);
370
371     if (!VLOG_DROP_DBG(&dbg_rl)) {
372         pre_flow_str = flow_to_string(flow);
373     }
374
375     if (!cfg->ip_src_flow) {
376         flow->tunnel.ip_src = tnl_port->match.ip_src;
377     }
378     if (!cfg->ip_dst_flow) {
379         flow->tunnel.ip_dst = tnl_port->match.ip_dst;
380     }
381     flow->pkt_mark = tnl_port->match.pkt_mark;
382
383     if (!cfg->out_key_flow) {
384         flow->tunnel.tun_id = cfg->out_key;
385     }
386
387     if (cfg->ttl_inherit && is_ip_any(flow)) {
388         wc->masks.nw_ttl = 0xff;
389         flow->tunnel.ip_ttl = flow->nw_ttl;
390     } else {
391         flow->tunnel.ip_ttl = cfg->ttl;
392     }
393
394     if (cfg->tos_inherit && is_ip_any(flow)) {
395         wc->masks.nw_tos = 0xff;
396         flow->tunnel.ip_tos = flow->nw_tos & IP_DSCP_MASK;
397     } else {
398         flow->tunnel.ip_tos = cfg->tos;
399     }
400
401     /* ECN fields are always inherited. */
402     if (is_ip_any(flow)) {
403         wc->masks.nw_tos |= IP_ECN_MASK;
404     }
405
406     if ((flow->nw_tos & IP_ECN_MASK) == IP_ECN_CE) {
407         flow->tunnel.ip_tos |= IP_ECN_ECT_0;
408     } else {
409         flow->tunnel.ip_tos |= flow->nw_tos & IP_ECN_MASK;
410     }
411
412     flow->tunnel.flags = (cfg->dont_fragment ? FLOW_TNL_F_DONT_FRAGMENT : 0)
413         | (cfg->csum ? FLOW_TNL_F_CSUM : 0)
414         | (cfg->out_key_present ? FLOW_TNL_F_KEY : 0);
415
416     if (pre_flow_str) {
417         char *post_flow_str = flow_to_string(flow);
418         char *tnl_str = tnl_port_fmt(tnl_port);
419         VLOG_DBG("flow sent\n"
420                  "%s"
421                  " pre: %s\n"
422                  "post: %s",
423                  tnl_str, pre_flow_str, post_flow_str);
424         free(tnl_str);
425         free(pre_flow_str);
426         free(post_flow_str);
427     }
428
429 out:
430     ovs_rwlock_unlock(&rwlock);
431     return out_port;
432 }
433
434 static uint32_t
435 tnl_hash(struct tnl_match *match)
436 {
437     BUILD_ASSERT_DECL(sizeof *match % sizeof(uint32_t) == 0);
438     return hash_words((uint32_t *) match, sizeof *match / sizeof(uint32_t), 0);
439 }
440
441 static struct tnl_port *
442 tnl_find_ofport(const struct ofport_dpif *ofport) OVS_REQ_RDLOCK(rwlock)
443 {
444     struct tnl_port *tnl_port;
445
446     HMAP_FOR_EACH_IN_BUCKET (tnl_port, ofport_node, hash_pointer(ofport, 0),
447                              ofport_map) {
448         if (tnl_port->ofport == ofport) {
449             return tnl_port;
450         }
451     }
452     return NULL;
453 }
454
455 static struct tnl_port *
456 tnl_find_exact(struct tnl_match *match, struct hmap *map)
457     OVS_REQ_RDLOCK(rwlock)
458 {
459     if (map) {
460         struct tnl_port *tnl_port;
461
462         HMAP_FOR_EACH_WITH_HASH (tnl_port, match_node, tnl_hash(match), map) {
463             if (!memcmp(match, &tnl_port->match, sizeof *match)) {
464                 return tnl_port;
465             }
466         }
467     }
468     return NULL;
469 }
470
471 /* Returns the tnl_port that is the best match for the tunnel data in 'flow',
472  * or NULL if no tnl_port matches 'flow'. */
473 static struct tnl_port *
474 tnl_find(const struct flow *flow) OVS_REQ_RDLOCK(rwlock)
475 {
476     enum ip_src_type ip_src;
477     int in_key_flow;
478     int ip_dst_flow;
479     int i;
480
481     i = 0;
482     for (in_key_flow = 0; in_key_flow < 2; in_key_flow++) {
483         for (ip_dst_flow = 0; ip_dst_flow < 2; ip_dst_flow++) {
484             for (ip_src = 0; ip_src < 3; ip_src++) {
485                 struct hmap *map = tnl_match_maps[i];
486
487                 if (map) {
488                     struct tnl_port *tnl_port;
489                     struct tnl_match match;
490
491                     memset(&match, 0, sizeof match);
492
493                     /* The apparent mix-up of 'ip_dst' and 'ip_src' below is
494                      * correct, because "struct tnl_match" is expressed in
495                      * terms of packets being sent out, but we are using it
496                      * here as a description of how to treat received
497                      * packets. */
498                     match.in_key = in_key_flow ? 0 : flow->tunnel.tun_id;
499                     match.ip_src = (ip_src == IP_SRC_CFG
500                                     ? flow->tunnel.ip_dst
501                                     : 0);
502                     match.ip_dst = ip_dst_flow ? 0 : flow->tunnel.ip_src;
503                     match.odp_port = flow->in_port.odp_port;
504                     match.pkt_mark = flow->pkt_mark;
505                     match.in_key_flow = in_key_flow;
506                     match.ip_dst_flow = ip_dst_flow;
507                     match.ip_src_flow = ip_src == IP_SRC_FLOW;
508
509                     tnl_port = tnl_find_exact(&match, map);
510                     if (tnl_port) {
511                         return tnl_port;
512                     }
513                 }
514
515                 i++;
516             }
517         }
518     }
519
520     return NULL;
521 }
522
523 /* Returns a pointer to the 'tnl_match_maps' element corresponding to 'm''s
524  * matching criteria. */
525 static struct hmap **
526 tnl_match_map(const struct tnl_match *m)
527 {
528     enum ip_src_type ip_src;
529
530     ip_src = (m->ip_src_flow ? IP_SRC_FLOW
531               : m->ip_src ? IP_SRC_CFG
532               : IP_SRC_ANY);
533
534     return &tnl_match_maps[6 * m->in_key_flow + 3 * m->ip_dst_flow + ip_src];
535 }
536
537 static void
538 tnl_match_fmt(const struct tnl_match *match, struct ds *ds)
539     OVS_REQ_RDLOCK(rwlock)
540 {
541     if (!match->ip_dst_flow) {
542         ds_put_format(ds, IP_FMT"->"IP_FMT, IP_ARGS(match->ip_src),
543                       IP_ARGS(match->ip_dst));
544     } else if (!match->ip_src_flow) {
545         ds_put_format(ds, IP_FMT"->flow", IP_ARGS(match->ip_src));
546     } else {
547         ds_put_cstr(ds, "flow->flow");
548     }
549
550     if (match->in_key_flow) {
551         ds_put_cstr(ds, ", key=flow");
552     } else {
553         ds_put_format(ds, ", key=%#"PRIx64, ntohll(match->in_key));
554     }
555
556     ds_put_format(ds, ", dp port=%"PRIu32, match->odp_port);
557     ds_put_format(ds, ", pkt mark=%"PRIu32, match->pkt_mark);
558 }
559
560 static void
561 tnl_port_mod_log(const struct tnl_port *tnl_port, const char *action)
562     OVS_REQ_RDLOCK(rwlock)
563 {
564     if (VLOG_IS_DBG_ENABLED()) {
565         struct ds ds = DS_EMPTY_INITIALIZER;
566
567         tnl_match_fmt(&tnl_port->match, &ds);
568         VLOG_INFO("%s tunnel port %s (%s)", action,
569                   tnl_port_get_name(tnl_port), ds_cstr(&ds));
570         ds_destroy(&ds);
571     }
572 }
573
574 static char *
575 tnl_port_fmt(const struct tnl_port *tnl_port) OVS_REQ_RDLOCK(rwlock)
576 {
577     const struct netdev_tunnel_config *cfg =
578         netdev_get_tunnel_config(tnl_port->netdev);
579     struct ds ds = DS_EMPTY_INITIALIZER;
580
581     ds_put_format(&ds, "port %"PRIu32": %s (%s: ", tnl_port->match.odp_port,
582                   tnl_port_get_name(tnl_port),
583                   netdev_get_type(tnl_port->netdev));
584     tnl_match_fmt(&tnl_port->match, &ds);
585
586     if (cfg->out_key != cfg->in_key ||
587         cfg->out_key_present != cfg->in_key_present ||
588         cfg->out_key_flow != cfg->in_key_flow) {
589         ds_put_cstr(&ds, ", out_key=");
590         if (!cfg->out_key_present) {
591             ds_put_cstr(&ds, "none");
592         } else if (cfg->out_key_flow) {
593             ds_put_cstr(&ds, "flow");
594         } else {
595             ds_put_format(&ds, "%#"PRIx64, ntohll(cfg->out_key));
596         }
597     }
598
599     if (cfg->ttl_inherit) {
600         ds_put_cstr(&ds, ", ttl=inherit");
601     } else {
602         ds_put_format(&ds, ", ttl=%"PRIu8, cfg->ttl);
603     }
604
605     if (cfg->tos_inherit) {
606         ds_put_cstr(&ds, ", tos=inherit");
607     } else if (cfg->tos) {
608         ds_put_format(&ds, ", tos=%#"PRIx8, cfg->tos);
609     }
610
611     if (!cfg->dont_fragment) {
612         ds_put_cstr(&ds, ", df=false");
613     }
614
615     if (cfg->csum) {
616         ds_put_cstr(&ds, ", csum=true");
617     }
618
619     ds_put_cstr(&ds, ")\n");
620
621     return ds_steal_cstr(&ds);
622 }
623
624 static const char *
625 tnl_port_get_name(const struct tnl_port *tnl_port) OVS_REQ_RDLOCK(rwlock)
626 {
627     return netdev_get_name(tnl_port->netdev);
628 }