Acked-by: Ethan Jackson <[email protected]> I made a slight tweak to the patch. Specifically I passed the tcp_flags into action_xlate_ctx_init() instead of mucking with the context directly. I'll merge this and backport it to the relevant branches.
On Sun, May 12, 2013 at 2:53 PM, Ben Pfaff <[email protected]> wrote: > The xlate_actions() call in handle_flow_miss_without_facet() didn't > implement fin_timeout properly because tcp_flags wasn't getting set. > > I have not tested that this fixes the problem, but it seems "obviously > correct". > > Bug #16506. > Reported-by: Ying Chen <[email protected]> > Signed-off-by: Ben Pfaff <[email protected]> > --- > AUTHORS | 1 + > ofproto/ofproto-dpif.c | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/AUTHORS b/AUTHORS > index c113f16..498e9f0 100644 > --- a/AUTHORS > +++ b/AUTHORS > @@ -206,6 +206,7 @@ Vjekoslav Brajkovic [email protected] > Voravit T. [email protected] > YAMAMOTO Takashi [email protected] > Yeming Zhao [email protected] > +Ying Chen [email protected] > Yongqiang Liu [email protected] > kk yap [email protected] > likunyun [email protected] > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c > index 1da8ffb..ecb50d5 100644 > --- a/ofproto/ofproto-dpif.c > +++ b/ofproto/ofproto-dpif.c > @@ -3689,6 +3689,7 @@ handle_flow_miss_without_facet(struct flow_miss *miss, > action_xlate_ctx_init(&ctx, ofproto, &miss->flow, > &miss->initial_vals, rule, 0, packet); > ctx.resubmit_stats = &stats; > + ctx.tcp_flags = stats.tcp_flags; > xlate_actions(&ctx, rule->up.ofpacts, rule->up.ofpacts_len, > &odp_actions); > > -- > 1.7.10.4 > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
