Thanks a lot. On May 13, 2013 11:28 AM, "Ethan Jackson" <[email protected]> wrote:
> 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
