Hi all,

In my test, vlan-limit is set to 0 that means unlimited the count of vlan
headers.
ovs-vsctl set Open_vSwitch . other_config:vlan-limit=0

But in fact the macro FLOW_MAX_VLAN_HEADERS is defined as 2, so we can
only support max two vlan headers. It doesn't work as the config vlan-limit's
description.

So, when VM sends a packet already with two vlan headers, the vport configured 
with
qinq mode can't add another s-vlan headers. FLOW_MAX_VLAN_HEADERS need to
be larger to support more vlan headers.

Thanks

Change-Id: I8449e308d406ce3757b43a2636ff0f326ca12a9d
Signed-off-by: Lilijun <[email protected]>
---
include/openvswitch/flow.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/openvswitch/flow.h b/include/openvswitch/flow.h
index 5d2cf09..b42710c 100644
--- a/include/openvswitch/flow.h
+++ b/include/openvswitch/flow.h
@@ -72,7 +72,7 @@ const char *flow_tun_flag_to_string(uint32_t flags);
  *
  * We require this to be a multiple of 2 so that vlans[] in struct flow is a
  * multiple of 64 bits. */
-#define FLOW_MAX_VLAN_HEADERS 2
+#define FLOW_MAX_VLAN_HEADERS 4
BUILD_ASSERT_DECL(FLOW_MAX_VLAN_HEADERS % 2 == 0);

 /* Legacy maximum VLAN headers */


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to