net: openvswitch: IPv6: Add IPv6 extension header support
Subsystems (3)
PATCH v4 1/1 net: openvswitch: IPv6: Add IPv6 extension header support
This change adds a new OpenFlow field OFPXMT_OFB_IPV6_EXTHDR and packets can be filtered using ipv6_ext flag.
--- include/uapi/linux/openvswitch.h | 12 +++ net/openvswitch/flow.c | 140 +++++++++++++++++++++++++++++++ net/openvswitch/flow.h | 14 ++++ net/openvswitch/flow_netlink.c | 24 +++++- 4 files changed, 189 insertions(+), 1 deletion(-)
include/uapi/linux/openvswitch.h +12 -0
@@ -346,6 +346,13 @@ enum ovs_key_attr {#ifdef __KERNEL__ OVS_KEY_ATTR_TUNNEL_INFO, /* struct ip_tunnel_info */#endif + +#ifndef __KERNEL__ +PADDING ,/* Padding so kernel and non kernel field count would match */ +#endif + +OVS_KEY_ATTR_IPV6_EXTHDRS ,/* struct ovs_key_ipv6_exthdr */ + __OVS_KEY_ATTR_MAX };
@@ -421,6 +428,11 @@ struct ovs_key_ipv6 {__u8 ipv6_frag;/* One of OVS_FRAG_TYPE_*. */ }; +/* separate structure to support backward compatibility with older user space */ +struct ovs_key_ipv6_exthdrs { +__u16 hdrs; +}; +struct ovs_key_tcp {__be16 tcp_src;__be16 tcp_dst;
net/openvswitch/flow.c +140 -0
152 lines not on this page · raw message
net/openvswitch/flow.h +14 -0
@@ -32,6 +32,19 @@ enum sw_flow_mac_proto {#define SW_FLOW_KEY_INVALID 0x80#define MPLS_LABEL_DEPTH 3 +/* Bit definitions for IPv6 Extension Header pseudo-field. */ +enum ofp12_ipv6exthdr_flags { +OFPIEH12_NONEXT =1 <<0 ,/* "No next header" encountered. */ +OFPIEH12_ESP =1 <<1 ,/* Encrypted Sec Payload header present. */ +OFPIEH12_AUTH =1 <<2 ,/* Authentication header present. */ +OFPIEH12_DEST =1 <<3 ,/* 1 or 2 dest headers present. */ +OFPIEH12_FRAG =1 <<4 ,/* Fragment header present. */ +OFPIEH12_ROUTER =1 <<5 ,/* Router header present. */ +OFPIEH12_HOP =1 <<6 ,/* Hop-by-hop header present. */ +OFPIEH12_UNREP =1 <<7 ,/* Unexpected repeats encountered. */ +OFPIEH12_UNSEQ =1 <<8 /* Unexpected sequencing encountered. */ +}; +/* Store options at the end of the array if they are less than the * maximum size. This allows us to get the benefits of variable length * matching for small options.
@@ -121,6 +134,7 @@ struct sw_flow_key {struct in6_addr dst;/* IPv6 destination address. */ } addr;__be32 label;/* IPv6 flow label. */ +u16 exthdrs;/* IPv6 extension header flags */ union { struct { struct in6_addr src;
net/openvswitch/flow_netlink.c +23 -1
@@ -367,7 +367,8 @@ size_t ovs_key_attr_size(void) +nla_total_size (2 )/* OVS_KEY_ATTR_ETHERTYPE */ +nla_total_size (40 )/* OVS_KEY_ATTR_IPV6 */ +nla_total_size (2 )/* OVS_KEY_ATTR_ICMPV6 */ - +nla_total_size (28 );/* OVS_KEY_ATTR_ND */ + +nla_total_size (28 )/* OVS_KEY_ATTR_ND */ + +nla_total_size (2 );/* OVS_KEY_ATTR_IPV6_EXTHDRS */ }static const struct ovs_len_tbl ovs_vxlan_ext_key_lens[OVS_VXLAN_EXT_MAX +1 ] = {
@@ -435,6 +436,8 @@ static const struct ovs_len_tbl ovs_key_lens[OVS_KEY_ATTR_MAX + 1] = { .len = sizeof(struct ovs_key_ct_tuple_ipv6) }, [OVS_KEY_ATTR_NSH] = { .len = OVS_ATTR_NESTED, .next = ovs_nsh_key_attr_lens, }, + [OVS_KEY_ATTR_IPV6_EXTHDRS] = { + .len = sizeof(struct ovs_key_ipv6_exthdrs) }, };static bool check_attr_len (unsigned int attr_len,unsigned int expected_len)
@@ -1595,6 +1598,17 @@ static int ovs_key_from_nlattrs(struct net *net, struct sw_flow_match *match, attrs &= ~(1 <<OVS_KEY_ATTR_IPV6 ); } +if (attrs & (1ULL <<OVS_KEY_ATTR_IPV6_EXTHDRS )) { +const struct ovs_key_ipv6_exthdrs *ipv6_exthdrs_key; + + ipv6_exthdrs_key =nla_data (a[OVS_KEY_ATTR_IPV6_EXTHDRS ]); + +SW_FLOW_KEY_PUT (match, ipv6.exthdrs, + ipv6_exthdrs_key->hdrs, is_mask); + + attrs &= ~(1ULL <<OVS_KEY_ATTR_IPV6_EXTHDRS ); + } +if (attrs & (1 <<OVS_KEY_ATTR_ARP )) {const struct ovs_key_arp *arp_key;
@@ -2097,6 +2111,7 @@ static int __ovs_nla_put_key(const struct sw_flow_key *swkey, ipv4_key->ipv4_frag = output->ip.frag; }else if (swkey->eth.type ==htons (ETH_P_IPV6 )) {struct ovs_key_ipv6 *ipv6_key; +struct ovs_key_ipv6_exthdrs *ipv6_exthdrs_key; nla =nla_reserve (skb,OVS_KEY_ATTR_IPV6 ,sizeof (*ipv6_key));if (!nla)
@@ -2111,6 +2126,13 @@ static int __ovs_nla_put_key(const struct sw_flow_key *swkey, ipv6_key->ipv6_tclass = output->ip.tos; ipv6_key->ipv6_hlimit = output->ip.ttl; ipv6_key->ipv6_frag = output->ip.frag; + + nla =nla_reserve (skb,OVS_KEY_ATTR_IPV6_EXTHDRS , +sizeof (*ipv6_exthdrs_key)); +if (!nla) + goto nla_put_failure; + ipv6_exthdrs_key =nla_data (nla); + ipv6_exthdrs_key->hdrs = output->ipv6.exthdrs; }else if (swkey->eth.type ==htons (ETH_P_NSH )) {if (nsh_key_to_nlattr (&output->nsh, is_mask, skb)) goto nla_put_failure;
Signature · 1 line
2.25.1
Jakub Kicinski
maintainerQuoted 25 lines · On Thu, 16 Sep 2021 13:44:09 -0700 Toms Atteka wrote:
+/**
+ * Parses packet and sets IPv6 extension header flags.
+ *+ * skb buffer where extension header data starts in packet + * nh ipv6 header + * ext_hdrs flags are stored here+ *
+ *OFPIEH12_UNREPis set if more than one of a given IPv6 extension header
+ * is unexpectedly encountered. (Two destination options headers may be
+ * expected and would not cause this bit to be set.)
+ *
+ *OFPIEH12_UNSEQis set if IPv6 extension headers were not in the order
+ * preferred (but not required) by RFC 2460:
+ *
+ * When more than one extension header is used in the same packet, it is
+ * recommended that those headers appear in the following order:+ * IPv6 header + * Hop-by-Hop Options header + * Destination Options header + * Routing header + * Fragment header + * Authentication header + * Encapsulating Security Payload header + * Destination Options header + * upper-layer header
Quoted 1 line
+ */
This is not a valid kdoc format, please read Documentation/doc-guide/kernel-doc.rst and double check for warnings with scripts/kernel-doc -none.
Quoted 3 lines
net/openvswitch/flow.c +3
-0+static void get_ipv6_ext_hdrs (struct sk_buff *skb,struct ipv6hdr *nh, +u16 *ext_hdrs) +{
Jakub Kicinski
maintainerQuoted 28 lines · On Thu, 16 Sep 2021 19:14:49 -0700 Jakub Kicinski wrote:
Quoted 25 lines · On Thu, 16 Sep 2021 13:44:09 -0700 Toms Atteka wrote:
+/**
+ * Parses packet and sets IPv6 extension header flags.
+ *+ * skb buffer where extension header data starts in packet + * nh ipv6 header + * ext_hdrs flags are stored here+ *
+ *OFPIEH12_UNREPis set if more than one of a given IPv6 extension header
+ * is unexpectedly encountered. (Two destination options headers may be
+ * expected and would not cause this bit to be set.)
+ *
+ *OFPIEH12_UNSEQis set if IPv6 extension headers were not in the order
+ * preferred (but not required) by RFC 2460:
+ *
+ * When more than one extension header is used in the same packet, it is
+ * recommended that those headers appear in the following order:+ * IPv6 header + * Hop-by-Hop Options header + * Destination Options header + * Routing header + * Fragment header + * Authentication header + * Encapsulating Security Payload header + * Destination Options header + * upper-layer headerQuoted 1 line
+ */
Quoted 3 lines
This is not a valid kdoc format, please read
Documentation/doc-guide/kernel-doc.rstand double check for warnings withscripts/kernel-doc-none.
Also please make sure to CC appropriate maintainers on v5. scripts/get_maintainers.pl is your friend there.