[ovs-dev] [PATCH 2/6] OXM: Allow VLAN VID to include CFI bit

Simon Horman horms at verge.net.au
Thu Jul 5 08:27:28 UTC 2012


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 <horms at 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




More information about the dev mailing list