On Fri, Jan 18, 2019 at 12:57:53PM +0300, Ilya Maximets wrote:
> On 18.01.2019 8:38, wangyunjian wrote:
> > From: Yunjian Wang <[email protected]>
> > 
> > A problem the userspace datapath failed to create a new datapath flow
> > when dealing with QinQ packets(the flow includeing ip,udp,etc). L2-L5
> > header should be considered before parsing the second 802.1Q header.
> > 
> > Fixes: f0fb825a3785 ("Add support for 802.1ad (QinQ tunneling)")
> > Signed-off-by: Yunjian Wang <[email protected]>
> > ---
> >  lib/odp-util.c | 58 
> > +++++++++++++++++++++++++++++++---------------------------
> >  1 file changed, 31 insertions(+), 27 deletions(-)
> > 
> 
> Hi. Thanks for the fix.
> 
> It'll be good to have a unit test for this issue to be sure that everything 
> works
> and will not be broken in the future.

I agree.

For now, I applied this to master.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to