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

Reply via email to