Re: [ovs-discuss] [ovs-dev] a problem the userspace datapath failed to create a new datapath flow for the QinQ packet

2019-01-15 Thread Ben Pfaff
On Mon, Jan 14, 2019 at 12:12:34PM +, wangyunjian wrote:
> Hi,
> 
> I found a problem the userspace datapath failed to create a 
> new datapath flow for the QinQ packet. When getting mask from
> nlattrs with the mask include ip,udp,etc, it parsed 802.1Q
> header wrongly.
> 
> This patch fixes it.  Any ideas?  

Thanks for working to fix the problem.  This introduces a lot of
redundant code.  Would you mind refactoring so that there is less code
duplication?
___
discuss mailing list
disc...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-discuss


Re: [ovs-discuss] ovs-ofctl transactions syntax

2019-01-15 Thread Ben Pfaff
Please don't drop the list.

ovs-ofctl --bundle add-flows SWITCH - < Is there a way to do that without a file, i. e. specifying the flows
> inline?
> 
> Il giorno mar 15 gen 2019, 01:10 Ben Pfaff  ha scritto:
> 
> > On Mon, Jan 14, 2019 at 09:29:56PM -0500, Matteo Olivi wrote:
> > > What's the syntax for adding multiple flows to an OvS bridge within a
> > > single transaction? I read the man page of ovs-ofctl but could not
> > > understand how to do that. I also tried to use the same syntax as
> > ovs-vsctl
> > > (separate commands with "--", I separated flows instead of commands), but
> > > it did not work.
> >
> > --bundle add-flows SWITCH FILE
> >
___
discuss mailing list
disc...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-discuss