Re: [ovs-dev] [ovs-discuss] [PATCH v2] [PATCH] QinQ: support more vlan headers.

2018-09-17 Thread Eric Garver
On Tue, Sep 11, 2018 at 01:10:09AM +, Lilijun (Jerry, Cloud Networking) wrote: > Thanks for your reply. > > In my use case, it's OVS userspace datapath with dpdk. > > My detail case was a bit complicated as follows: > 1. Start the OVS userspace datapath with dpdk in my host server. > 2. A

Re: [ovs-dev] [ovs-discuss] [PATCH v2] [PATCH] QinQ: support more vlan headers.

2018-09-10 Thread Lilijun (Jerry, Cloud Networking)
Thanks for your reply. In my use case, it's OVS userspace datapath with dpdk. My detail case was a bit complicated as follows: 1. Start the OVS userspace datapath with dpdk in my host server. 2. A VM was running and the VNIC's vhostuser port on the userspace datapath is configured as QinQ

Re: [ovs-dev] [ovs-discuss] [PATCH v2] [PATCH] QinQ: support more vlan headers.

2018-09-10 Thread Ben Pfaff
On Mon, Sep 10, 2018 at 03:15:21PM -0400, Eric Garver wrote: > On Mon, Sep 10, 2018 at 03:03:19AM +, Lilijun (Jerry, Cloud Networking) > wrote: > > Hi Eric, > > > > Yes, I agree with that effect. > > But how about this issue of QinQ that we can only support at most 2 VLANs ? > > Do you

Re: [ovs-dev] [ovs-discuss] [PATCH v2] [PATCH] QinQ: support more vlan headers.

2018-09-10 Thread Eric Garver
On Mon, Sep 10, 2018 at 03:03:19AM +, Lilijun (Jerry, Cloud Networking) wrote: > Hi Eric, > > Yes, I agree with that effect. > But how about this issue of QinQ that we can only support at most 2 VLANs ? > Do you have any ideas? I was not NACKing the idea. Just wanted everyone to

Re: [ovs-dev] [ovs-discuss] [PATCH v2] [PATCH] QinQ: support more vlan headers.

2018-09-09 Thread Lilijun (Jerry, Cloud Networking)
Hi Eric, Yes, I agree with that effect. But how about this issue of QinQ that we can only support at most 2 VLANs ? Do you have any ideas? Thanks. -Original Message- From: Eric Garver [mailto:e...@garver.life] Sent: Friday, September 07, 2018 10:14 PM To: Lilijun (Jerry, Cloud

Re: [ovs-dev] [ovs-discuss] [PATCH v2] [PATCH] QinQ: support more vlan headers.

2018-09-07 Thread Eric Garver
On Fri, Sep 07, 2018 at 09:53:44AM +, Lilijun (Jerry, Cloud Networking) wrote: > 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