[ovs-dev] [PATCH] mpls: Fix MPLS restoration after patch port and group bucket.

2016-12-02 Thread Jarno Rajahalme
Morin <thomas.mo...@orange.com> Suggested-by: Takashi YAMAMOTO <yamam...@ovn.org> Fixes: 8bfd0fdac ("Enhance userspace support for MPLS, for up to 3 labels.") Fixes: 1b035ef20 ("mpls: Allow l3 and l4 actions to prior to a push_mpls action") Signed-off-by: Jarno Ra

Re: [ovs-dev] assertion failing in commit_set_ipv4_action(), flow->nw_proto != base_flow->nw_proto

2016-12-01 Thread Jarno Rajahalme
> On Nov 30, 2016, at 8:50 PM, Ben Pfaff <b...@ovn.org> wrote: > > On Wed, Nov 30, 2016 at 06:58:57PM -0800, Jarno Rajahalme wrote: >> >>> On Nov 30, 2016, at 8:41 AM, Ben Pfaff <b...@ovn.org> wrote: >>> >>> On Wed, Nov 30, 201

Re: [ovs-dev] [PATCH] ofproto: Support reset_counts on flow mod message.

2016-12-01 Thread Jarno Rajahalme
> On Nov 29, 2016, at 3:46 PM, Tony van der Peet > wrote: > > If the reset_counts flag is set on a flow modification message, the > flow counters must be cleared, even if the flow does not already have > the reset_counts flag set. And the flow modification

Re: [ovs-dev] [PATCH net-next v2] ofproto: Honor OFPFF_RESET_COUNTS flag in flow modify message.

2016-12-01 Thread Jarno Rajahalme
> On Nov 30, 2016, at 9:06 PM, Ben Pfaff <b...@ovn.org> wrote: > > On Wed, Nov 30, 2016 at 05:51:12PM -0800, Jarno Rajahalme wrote: >> While a flow modify must keep the original flow's flags, it must reset >> counts if (and only if) the reset_counts flag is present

Re: [ovs-dev] assertion failing in commit_set_ipv4_action(), flow->nw_proto != base_flow->nw_proto

2016-11-30 Thread Jarno Rajahalme
> On Nov 30, 2016, at 8:41 AM, Ben Pfaff wrote: > > On Wed, Nov 30, 2016 at 12:05:46PM +0100, Thomas Morin wrote: >> Hi Ben, >> >> 2016-11-30, Ben Pfaff: >>> Do you have any idea what in your OpenFlow pipeline might do that, >>> i.e. is there anything especially tricky in the

Re: [ovs-dev] relationship between dpcls_rule and dp_netdev_flow

2016-11-15 Thread Jarno Rajahalme
> On Nov 15, 2016, at 3:35 PM, Joo Kim wrote: > > Hello, > > In this OVS (2.6) code below, > It seems that a dp_netdev_flow obj contains a dpcls_rule struct, and given > a dpcls_rule, it gets the corresponding dp_netdev_flow via > dp_netdev_flow_cast(). > Does it mean

<    1   2   3   4