diff options
author | Jesse Gross <jesse@nicira.com> | 2013-07-09 14:21:26 -0700 |
---|---|---|
committer | Jesse Gross <jesse@nicira.com> | 2013-07-09 15:31:52 -0700 |
commit | 8bf118c212fed261aeeb33ee8900b423aee31ed8 (patch) | |
tree | dcbf3680083d837d2c76c56cc808893e84519c4e /datapath | |
parent | 7bebb62c6766bf7d7bb95c2b31d0e528d95ed88c (diff) |
datapath: Use DP_MAX_PORTS when no IN_PORT attribute is present.
To indicate that a flow is not associated with any particular in port,
userspace may omit the IN_PORT attribute, which the kernel translates
internally to the special value DP_MAX_PORTS. After the megaflows
changes, this was no longer being done, resulting in it using port 0
(the internal port).
This also adopts a wildcarding scheme similar to 802.2 packets where
a mask can be specified for this non-existent key attribute but it
must either be completely wildcarded or completely exact match.
CC: Andy Zhou <azhou@nicira.com>
Signed-off-by: Jesse Gross <jesse@nicira.com>
Diffstat (limited to 'datapath')
-rw-r--r-- | datapath/flow.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/datapath/flow.c b/datapath/flow.c index e97bf494..7d6392ea 100644 --- a/datapath/flow.c +++ b/datapath/flow.c @@ -133,6 +133,10 @@ static bool ovs_match_validate(const struct sw_flow_match *match, | (1ULL << OVS_KEY_ATTR_ARP) | (1ULL << OVS_KEY_ATTR_ND)); + if (match->key->phy.in_port == DP_MAX_PORTS && + match->mask && (match->mask->key.phy.in_port == 0xffff)) + mask_allowed |= (1ULL << OVS_KEY_ATTR_IN_PORT); + if (match->key->eth.type == htons(ETH_P_802_2) && match->mask && (match->mask->key.eth.type == htons(0xffff))) mask_allowed |= (1ULL << OVS_KEY_ATTR_ETHERTYPE); @@ -1312,6 +1316,8 @@ static int metadata_from_nlattrs(struct sw_flow_match *match, u64 *attrs, return -EINVAL; SW_FLOW_KEY_PUT(match, phy.in_port, in_port, is_mask); *attrs &= ~(1ULL << OVS_KEY_ATTR_IN_PORT); + } else if (!is_mask) { + SW_FLOW_KEY_PUT(match, phy.in_port, DP_MAX_PORTS, is_mask); } if (*attrs & (1ULL << OVS_KEY_ATTR_SKB_MARK)) { |