OXM VLAN VID matches may include the CFI bit. It seems harmless enough to allow them for the other use of this code. The other use is ovs-ofctl add-flow ... dl_vlan....
Signed-off-by: Simon Horman <ho...@verge.net.au> --- lib/meta-flow.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/meta-flow.c b/lib/meta-flow.c index 9b2797c..b7f6fcc 100644 --- a/lib/meta-flow.c +++ b/lib/meta-flow.c @@ -875,7 +875,7 @@ mf_is_value_valid(const struct mf_field *mf, const union mf_value *value) return !(value->be16 & htons(0xff00)); case MFF_VLAN_VID: - return !(value->be16 & htons(VLAN_CFI | VLAN_PCP_MASK)); + return !(value->be16 & htons(VLAN_PCP_MASK)); case MFF_VLAN_PCP: return !(value->u8 & ~(VLAN_PCP_MASK >> VLAN_PCP_SHIFT)); -- 1.7.10.2.484.gcd07cc5 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev