tcp: fix fin_wait_1 condition to send fin
[vpp.git] / src / vnet / tcp / tcp_input.c
index ac0e996..32de369 100644 (file)
@@ -121,10 +121,11 @@ tcp_segment_in_rcv_wnd (tcp_connection_t * tc, u32 seq, u32 end_seq)
  *
  * @param th TCP header
  * @param to TCP options data structure to be populated
+ * @param is_syn set if packet is syn
  * @return -1 if parsing failed
  */
-static int
-tcp_options_parse (tcp_header_t * th, tcp_options_t * to)
+static inline int
+tcp_options_parse (tcp_header_t * th, tcp_options_t * to, u8 is_syn)
 {
   const u8 *data;
   u8 opt_len, opts_len, kind;
@@ -136,7 +137,7 @@ tcp_options_parse (tcp_header_t * th, tcp_options_t * to)
 
   /* Zero out all flags but those set in SYN */
   to->flags &= (TCP_OPTS_FLAG_SACK_PERMITTED | TCP_OPTS_FLAG_WSCALE
-               | TCP_OPTS_FLAG_SACK);
+               | TCP_OPTS_FLAG_TSTAMP | TCP_OPTION_MSS);
 
   for (; opts_len > 0; opts_len -= opt_len, data += opt_len)
     {
@@ -166,6 +167,8 @@ tcp_options_parse (tcp_header_t * th, tcp_options_t * to)
       switch (kind)
        {
        case TCP_OPTION_MSS:
+         if (!is_syn)
+           break;
          if ((opt_len == TCP_OPTION_LEN_MSS) && tcp_syn (th))
            {
              to->flags |= TCP_OPTS_FLAG_MSS;
@@ -173,27 +176,29 @@ tcp_options_parse (tcp_header_t * th, tcp_options_t * to)
            }
          break;
        case TCP_OPTION_WINDOW_SCALE:
+         if (!is_syn)
+           break;
          if ((opt_len == TCP_OPTION_LEN_WINDOW_SCALE) && tcp_syn (th))
            {
              to->flags |= TCP_OPTS_FLAG_WSCALE;
              to->wscale = data[2];
              if (to->wscale > TCP_MAX_WND_SCALE)
-               {
-                 clib_warning ("Illegal window scaling value: %d",
-                               to->wscale);
-                 to->wscale = TCP_MAX_WND_SCALE;
-               }
+               to->wscale = TCP_MAX_WND_SCALE;
            }
          break;
        case TCP_OPTION_TIMESTAMP:
-         if (opt_len == TCP_OPTION_LEN_TIMESTAMP)
+         if (is_syn)
+           to->flags |= TCP_OPTS_FLAG_TSTAMP;
+         if ((to->flags & TCP_OPTS_FLAG_TSTAMP)
+             && opt_len == TCP_OPTION_LEN_TIMESTAMP)
            {
-             to->flags |= TCP_OPTS_FLAG_TSTAMP;
              to->tsval = clib_net_to_host_u32 (*(u32 *) (data + 2));
              to->tsecr = clib_net_to_host_u32 (*(u32 *) (data + 6));
            }
          break;
        case TCP_OPTION_SACK_PERMITTED:
+         if (!is_syn)
+           break;
          if (opt_len == TCP_OPTION_LEN_SACK_PERMITTED && tcp_syn (th))
            to->flags |= TCP_OPTS_FLAG_SACK_PERMITTED;
          break;
@@ -236,7 +241,7 @@ tcp_options_parse (tcp_header_t * th, tcp_options_t * to)
 always_inline int
 tcp_segment_check_paws (tcp_connection_t * tc)
 {
-  return tcp_opts_tstamp (&tc->rcv_opts) && tc->tsval_recent
+  return tcp_opts_tstamp (&tc->rcv_opts)
     && timestamp_lt (tc->rcv_opts.tsval, tc->tsval_recent);
 }
 
@@ -258,7 +263,7 @@ tcp_update_timestamp (tcp_connection_t * tc, u32 seq, u32 seq_end)
     {
       ASSERT (timestamp_leq (tc->tsval_recent, tc->rcv_opts.tsval));
       tc->tsval_recent = tc->rcv_opts.tsval;
-      tc->tsval_recent_age = tcp_time_now ();
+      tc->tsval_recent_age = tcp_time_now_w_thread (tc->c_thread_index);
     }
 }
 
@@ -272,59 +277,53 @@ tcp_update_timestamp (tcp_connection_t * tc, u32 seq, u32 seq_end)
  * @return 0 if segments passes validation.
  */
 static int
-tcp_segment_validate (vlib_main_t * vm, tcp_connection_t * tc0,
-                     vlib_buffer_t * b0, tcp_header_t * th0,
-                     u32 * next0, u32 * error0)
+tcp_segment_validate (tcp_worker_ctx_t * wrk, tcp_connection_t * tc0,
+                     vlib_buffer_t * b0, tcp_header_t * th0, u32 * error0)
 {
   /* We could get a burst of RSTs interleaved with acks */
   if (PREDICT_FALSE (tc0->state == TCP_STATE_CLOSED))
     {
       tcp_send_reset (tc0);
       *error0 = TCP_ERROR_CONNECTION_CLOSED;
-      goto drop;
+      goto error;
     }
 
   if (PREDICT_FALSE (!tcp_ack (th0) && !tcp_rst (th0) && !tcp_syn (th0)))
     {
       *error0 = TCP_ERROR_SEGMENT_INVALID;
-      goto drop;
+      goto error;
     }
 
-  if (PREDICT_FALSE (tcp_options_parse (th0, &tc0->rcv_opts)))
+  if (PREDICT_FALSE (tcp_options_parse (th0, &tc0->rcv_opts, 0)))
     {
-      clib_warning ("options parse error");
       *error0 = TCP_ERROR_OPTIONS;
-      goto drop;
+      goto error;
     }
 
   if (PREDICT_FALSE (tcp_segment_check_paws (tc0)))
     {
       *error0 = TCP_ERROR_PAWS;
-      if (CLIB_DEBUG > 2)
-       clib_warning ("paws failed\n%U", format_tcp_connection, tc0, 2);
       TCP_EVT_DBG (TCP_EVT_PAWS_FAIL, tc0, vnet_buffer (b0)->tcp.seq_number,
                   vnet_buffer (b0)->tcp.seq_end);
 
       /* If it just so happens that a segment updates tsval_recent for a
        * segment over 24 days old, invalidate tsval_recent. */
       if (timestamp_lt (tc0->tsval_recent_age + TCP_PAWS_IDLE,
-                       tcp_time_now ()))
+                       tcp_time_now_w_thread (tc0->c_thread_index)))
        {
-         /* Age isn't reset until we get a valid tsval (bsd inspired) */
-         tc0->tsval_recent = 0;
-         clib_warning ("paws failed - really old segment. REALLY?");
+         tc0->tsval_recent = tc0->rcv_opts.tsval;
+         clib_warning ("paws failed: 24-day old segment");
        }
-      else
+      /* Drop after ack if not rst. Resets can fail paws check as per
+       * RFC 7323 sec. 5.2: When an <RST> segment is received, it MUST NOT
+       * be subjected to the PAWS check by verifying an acceptable value in
+       * SEG.TSval */
+      else if (!tcp_rst (th0))
        {
-         /* Drop after ack if not rst */
-         if (!tcp_rst (th0))
-           {
-             tcp_make_ack (tc0, b0);
-             TCP_EVT_DBG (TCP_EVT_DUPACK_SENT, tc0, vnet_buffer (b0)->tcp);
-             goto error;
-           }
+         tcp_program_ack (wrk, tc0);
+         TCP_EVT_DBG (TCP_EVT_DUPACK_SENT, tc0, vnet_buffer (b0)->tcp);
+         goto error;
        }
-      goto drop;
     }
 
   /* 1st: check sequence number */
@@ -340,11 +339,10 @@ tcp_segment_validate (vlib_main_t * vm, tcp_connection_t * tc0,
          /* If not RST, send dup ack */
          if (!tcp_rst (th0))
            {
-             tcp_make_ack (tc0, b0);
+             tcp_program_dupack (wrk, tc0);
              TCP_EVT_DBG (TCP_EVT_DUPACK_SENT, tc0, vnet_buffer (b0)->tcp);
-             goto error;
            }
-         goto drop;
+         goto error;
        }
     }
 
@@ -353,7 +351,7 @@ tcp_segment_validate (vlib_main_t * vm, tcp_connection_t * tc0,
     {
       tcp_connection_reset (tc0);
       *error0 = TCP_ERROR_RST_RCVD;
-      goto drop;
+      goto error;
     }
 
   /* 3rd: check security and precedence (skip) */
@@ -364,12 +362,13 @@ tcp_segment_validate (vlib_main_t * vm, tcp_connection_t * tc0,
       /* TODO implement RFC 5961 */
       if (tc0->state == TCP_STATE_SYN_RCVD)
        {
-         tcp_make_synack (tc0, b0);
+         tcp_options_parse (th0, &tc0->rcv_opts, 1);
+         tcp_send_synack (tc0);
          TCP_EVT_DBG (TCP_EVT_SYN_RCVD, tc0, 0);
        }
       else
        {
-         tcp_make_ack (tc0, b0);
+         tcp_program_ack (wrk, tc0);
          TCP_EVT_DBG (TCP_EVT_SYNACK_RCVD, tc0);
        }
       goto error;
@@ -380,11 +379,7 @@ tcp_segment_validate (vlib_main_t * vm, tcp_connection_t * tc0,
                        vnet_buffer (b0)->tcp.seq_end);
   return 0;
 
-drop:
-  *next0 = tcp_next_drop (tc0->c_is_ip4);
-  return -1;
 error:
-  *next0 = tcp_next_output (tc0->c_is_ip4);
   return -1;
 }
 
@@ -462,15 +457,19 @@ tcp_update_rtt (tcp_connection_t * tc, u32 ack)
 
   if (tc->rtt_ts && seq_geq (ack, tc->rtt_seq))
     {
-      tc->mrtt_us = tcp_time_now_us (tc->c_thread_index) - tc->rtt_ts;
-      mrtt = clib_max ((u32) (tc->mrtt_us * THZ), 1);
+      f64 sample = tcp_time_now_us (tc->c_thread_index) - tc->rtt_ts;
+      tc->mrtt_us = tc->mrtt_us + (sample - tc->mrtt_us) * 0.125;
+      mrtt = clib_max ((u32) (sample * THZ), 1);
+      /* Allow measuring of a new RTT */
+      tc->rtt_ts = 0;
     }
   /* As per RFC7323 TSecr can be used for RTTM only if the segment advances
    * snd_una, i.e., the left side of the send window:
    * seq_lt (tc->snd_una, ack). This is a condition for calling update_rtt */
   else if (tcp_opts_tstamp (&tc->rcv_opts) && tc->rcv_opts.tsecr)
     {
-      mrtt = clib_max (tcp_time_now () - tc->rcv_opts.tsecr, 1);
+      u32 now = tcp_time_now_w_thread (tc->c_thread_index);
+      mrtt = clib_max (now - tc->rcv_opts.tsecr, 1);
     }
 
   /* Ignore dubious measurements */
@@ -481,9 +480,6 @@ tcp_update_rtt (tcp_connection_t * tc, u32 ack)
 
 done:
 
-  /* Allow measuring of a new RTT */
-  tc->rtt_ts = 0;
-
   /* If we got here something must've been ACKed so make sure boff is 0,
    * even if mrtt is not valid since we update the rto lower */
   tc->rto_boff = 0;
@@ -492,24 +488,84 @@ done:
   return 0;
 }
 
+static void
+tcp_estimate_initial_rtt (tcp_connection_t * tc)
+{
+  u8 thread_index = vlib_num_workers ()? 1 : 0;
+  int mrtt;
+
+  if (tc->rtt_ts)
+    {
+      tc->mrtt_us = tcp_time_now_us (thread_index) - tc->rtt_ts;
+      mrtt = clib_max ((u32) (tc->mrtt_us * THZ), 1);
+      tc->rtt_ts = 0;
+    }
+  else
+    {
+      mrtt = tcp_time_now_w_thread (thread_index) - tc->rcv_opts.tsecr;
+      mrtt = clib_max (mrtt, 1);
+      tc->mrtt_us = (f64) mrtt *TCP_TICK;
+    }
+
+  if (mrtt > 0 && mrtt < TCP_RTT_MAX)
+    tcp_estimate_rtt (tc, mrtt);
+}
+
 /**
- * Dequeue bytes that have been acked and while at it update RTT estimates.
+ * Dequeue bytes for connections that have received acks in last burst
  */
 static void
-tcp_dequeue_acked (tcp_connection_t * tc, u32 ack)
+tcp_handle_postponed_dequeues (tcp_worker_ctx_t * wrk)
 {
-  /* Dequeue the newly ACKed add SACKed bytes */
-  stream_session_dequeue_drop (&tc->connection,
-                              tc->bytes_acked + tc->sack_sb.snd_una_adv);
+  u32 thread_index = wrk->vm->thread_index;
+  u32 *pending_deq_acked;
+  tcp_connection_t *tc;
+  int i;
 
-  tcp_validate_txf_size (tc, tc->snd_una_max - tc->snd_una);
+  if (!vec_len (wrk->pending_deq_acked))
+    return;
 
-  /* Update rtt and rto */
-  tcp_update_rtt (tc, ack);
+  pending_deq_acked = wrk->pending_deq_acked;
+  for (i = 0; i < vec_len (pending_deq_acked); i++)
+    {
+      tc = tcp_connection_get (pending_deq_acked[i], thread_index);
+      tc->flags &= ~TCP_CONN_DEQ_PENDING;
 
-  /* If everything has been acked, stop retransmit timer
-   * otherwise update. */
-  tcp_retransmit_timer_update (tc);
+      if (PREDICT_FALSE (!tc->burst_acked))
+       continue;
+
+      /* Dequeue the newly ACKed bytes */
+      stream_session_dequeue_drop (&tc->connection, tc->burst_acked);
+      tc->burst_acked = 0;
+      tcp_validate_txf_size (tc, tc->snd_una_max - tc->snd_una);
+
+      if (PREDICT_FALSE (tc->flags & TCP_CONN_PSH_PENDING))
+       {
+         if (seq_leq (tc->psh_seq, tc->snd_una))
+           tc->flags &= ~TCP_CONN_PSH_PENDING;
+       }
+
+      /* If everything has been acked, stop retransmit timer
+       * otherwise update. */
+      tcp_retransmit_timer_update (tc);
+
+      /* If not congested, update pacer based on our new
+       * cwnd estimate */
+      if (!tcp_in_fastrecovery (tc))
+       tcp_connection_tx_pacer_update (tc);
+    }
+  _vec_len (wrk->pending_deq_acked) = 0;
+}
+
+static void
+tcp_program_dequeue (tcp_worker_ctx_t * wrk, tcp_connection_t * tc)
+{
+  if (!(tc->flags & TCP_CONN_DEQ_PENDING))
+    {
+      vec_add1 (wrk->pending_deq_acked, tc->c_c_index);
+      tc->flags |= TCP_CONN_DEQ_PENDING;
+    }
+  tc->burst_acked += tc->bytes_acked + tc->sack_sb.snd_una_adv;
 }
 
 /**
@@ -718,8 +774,7 @@ scoreboard_update_bytes (tcp_connection_t * tc, sack_scoreboard_t * sb)
 sack_scoreboard_hole_t *
 scoreboard_next_rxt_hole (sack_scoreboard_t * sb,
                          sack_scoreboard_hole_t * start,
-                         u8 have_sent_1_smss,
-                         u8 * can_rescue, u8 * snd_limited)
+                         u8 have_unsent, u8 * can_rescue, u8 * snd_limited)
 {
   sack_scoreboard_hole_t *hole = 0;
 
@@ -741,11 +796,11 @@ scoreboard_next_rxt_hole (sack_scoreboard_t * sb,
     }
   else
     {
-      /* Rule (2): output takes care of transmitting new data */
-      if (!have_sent_1_smss)
+      /* Rule (2): available unsent data */
+      if (have_unsent)
        {
-         hole = 0;
          sb->cur_rxt_hole = TCP_INVALID_SACK_HOLE_INDEX;
+         return 0;
        }
       /* Rule (3): if hole not lost */
       else if (seq_lt (hole->start, sb->high_sacked))
@@ -771,16 +826,17 @@ scoreboard_next_rxt_hole (sack_scoreboard_t * sb,
 }
 
 static void
-scoreboard_init_high_rxt (sack_scoreboard_t * sb, u32 seq)
+scoreboard_init_high_rxt (sack_scoreboard_t * sb, u32 snd_una)
 {
   sack_scoreboard_hole_t *hole;
   hole = scoreboard_first_hole (sb);
   if (hole)
     {
-      seq = seq_gt (seq, hole->start) ? seq : hole->start;
+      snd_una = seq_gt (snd_una, hole->start) ? snd_una : hole->start;
       sb->cur_rxt_hole = sb->head;
     }
-  sb->high_rxt = seq;
+  sb->high_rxt = snd_una;
+  sb->rescue_rxt = snd_una - 1;
 }
 
 void
@@ -851,7 +907,9 @@ tcp_rcv_sacks (tcp_connection_t * tc, u32 ack)
     {
       if (seq_lt (blk->start, blk->end)
          && seq_gt (blk->start, tc->snd_una)
-         && seq_gt (blk->start, ack) && seq_leq (blk->end, tc->snd_una_max))
+         && seq_gt (blk->start, ack)
+         && seq_lt (blk->start, tc->snd_una_max)
+         && seq_leq (blk->end, tc->snd_una_max))
        {
          blk++;
          continue;
@@ -1022,7 +1080,7 @@ tcp_update_snd_wnd (tcp_connection_t * tc, u32 seq, u32 ack, u32 snd_wnd)
       tc->snd_wl2 = ack;
       TCP_EVT_DBG (TCP_EVT_SND_WND, tc);
 
-      if (tc->snd_wnd < tc->snd_mss)
+      if (PREDICT_FALSE (tc->snd_wnd < tc->snd_mss))
        {
          /* Set persist timer if not set and we just got 0 wnd */
          if (!tcp_timer_is_active (tc, TCP_TIMER_PERSIST)
@@ -1032,7 +1090,7 @@ tcp_update_snd_wnd (tcp_connection_t * tc, u32 seq, u32 ack, u32 snd_wnd)
       else
        {
          tcp_persist_timer_reset (tc);
-         if (!tcp_in_recovery (tc) && tc->rto_boff > 0)
+         if (PREDICT_FALSE (!tcp_in_recovery (tc) && tc->rto_boff > 0))
            {
              tc->rto_boff = 0;
              tcp_update_rto (tc);
@@ -1067,6 +1125,7 @@ tcp_cc_recovery_exit (tcp_connection_t * tc)
   tcp_update_rto (tc);
   tc->snd_rxt_ts = 0;
   tc->snd_nxt = tc->snd_una_max;
+  tc->rtt_ts = 0;
   tcp_recovery_off (tc);
   TCP_EVT_DBG (TCP_EVT_CC_EVT, tc, 3);
 }
@@ -1079,15 +1138,11 @@ tcp_cc_fastrecovery_exit (tcp_connection_t * tc)
   tc->rcv_dupacks = 0;
   tc->snd_nxt = tc->snd_una_max;
   tc->snd_rxt_bytes = 0;
+  tc->rtt_ts = 0;
 
   tcp_fastrecovery_off (tc);
-  tcp_fastrecovery_1_smss_off (tc);
   tcp_fastrecovery_first_off (tc);
 
-  /* Update pacer because our cwnd changed. Also makes sure
-   * that we recompute the max burst size */
-  tcp_update_pacer (tc);
-
   TCP_EVT_DBG (TCP_EVT_CC_EVT, tc, 3);
 }
 
@@ -1185,9 +1240,8 @@ tcp_should_fastrecover (tcp_connection_t * tc)
 }
 
 void
-tcp_program_fastretransmit (tcp_connection_t * tc)
+tcp_program_fastretransmit (tcp_worker_ctx_t * wrk, tcp_connection_t * tc)
 {
-  tcp_worker_ctx_t *wrk = &tcp_main.wrk_ctx[tc->c_thread_index];
   if (!(tc->flags & TCP_CONN_FRXT_PENDING))
     {
       vec_add1 (wrk->pending_fast_rxt, tc->c_c_index);
@@ -1196,70 +1250,62 @@ tcp_program_fastretransmit (tcp_connection_t * tc)
 }
 
 void
-tcp_do_fastretransmits (u32 thread_index)
+tcp_do_fastretransmits (tcp_worker_ctx_t * wrk)
 {
-  tcp_worker_ctx_t *wrk = &tcp_main.wrk_ctx[thread_index];
-  u32 max_burst_size, burst_size, n_segs = 0;
+  u32 *ongoing_fast_rxt, burst_bytes, sent_bytes, thread_index;
+  u32 max_burst_size, burst_size, n_segs = 0, n_segs_now;
   tcp_connection_t *tc;
+  u64 last_cpu_time;
   int i;
 
-  if (vec_len (wrk->pending_fast_rxt) == 0)
+  if (vec_len (wrk->pending_fast_rxt) == 0
+      && vec_len (wrk->postponed_fast_rxt) == 0)
     return;
 
-  vec_append (wrk->ongoing_fast_rxt, wrk->pending_fast_rxt);
-  vec_reset_length (wrk->pending_fast_rxt);
+  thread_index = wrk->vm->thread_index;
+  last_cpu_time = wrk->vm->clib_time.last_cpu_time;
+  ongoing_fast_rxt = wrk->ongoing_fast_rxt;
+  vec_append (ongoing_fast_rxt, wrk->postponed_fast_rxt);
+  vec_append (ongoing_fast_rxt, wrk->pending_fast_rxt);
 
-  max_burst_size = VLIB_FRAME_SIZE / vec_len (wrk->ongoing_fast_rxt);
+  _vec_len (wrk->postponed_fast_rxt) = 0;
+  _vec_len (wrk->pending_fast_rxt) = 0;
+
+  max_burst_size = VLIB_FRAME_SIZE / vec_len (ongoing_fast_rxt);
   max_burst_size = clib_max (max_burst_size, 1);
 
-  for (i = 0; i < vec_len (wrk->ongoing_fast_rxt); i++)
+  for (i = 0; i < vec_len (ongoing_fast_rxt); i++)
     {
-      tc = tcp_connection_get (wrk->ongoing_fast_rxt[i], thread_index);
+      if (n_segs >= VLIB_FRAME_SIZE)
+       {
+         vec_add1 (wrk->postponed_fast_rxt, ongoing_fast_rxt[i]);
+         continue;
+       }
+
+      tc = tcp_connection_get (ongoing_fast_rxt[i], thread_index);
       tc->flags &= ~TCP_CONN_FRXT_PENDING;
 
       if (!tcp_in_fastrecovery (tc))
        continue;
 
-      /* TODO tx pacer instead of this */
-      if (n_segs >= VLIB_FRAME_SIZE)
+      burst_size = clib_min (max_burst_size, VLIB_FRAME_SIZE - n_segs);
+      burst_bytes = transport_connection_tx_pacer_burst (&tc->connection,
+                                                        last_cpu_time);
+      burst_size = clib_min (burst_size, burst_bytes / tc->snd_mss);
+      if (!burst_size)
        {
-         tcp_program_fastretransmit (tc);
+         tcp_program_fastretransmit (wrk, tc);
          continue;
        }
 
-      burst_size = clib_min (max_burst_size, VLIB_FRAME_SIZE - n_segs);
-
-      if (tc->cwnd > tc->ssthresh + 3 * tc->snd_mss)
-       {
-         /* The first segment MUST be retransmitted */
-         if (tcp_retransmit_first_unacked (tc))
-           {
-             tcp_program_fastretransmit (tc);
-             continue;
-           }
-
-         /* Post retransmit update cwnd to ssthresh and account for the
-          * three segments that have left the network and should've been
-          * buffered at the receiver XXX */
-         tc->cwnd = tc->ssthresh + 3 * tc->snd_mss;
-
-         /* If cwnd allows, send more data */
-         if (tcp_opts_sack_permitted (&tc->rcv_opts))
-           {
-             scoreboard_init_high_rxt (&tc->sack_sb,
-                                       tc->snd_una + tc->snd_mss);
-             tc->sack_sb.rescue_rxt = tc->snd_una - 1;
-             n_segs += tcp_fast_retransmit_sack (tc, burst_size);
-           }
-         else
-           {
-             n_segs += tcp_fast_retransmit_no_sack (tc, burst_size);
-           }
-       }
-      else
-       n_segs += tcp_fast_retransmit (tc, burst_size);
+      n_segs_now = tcp_fast_retransmit (wrk, tc, burst_size);
+      sent_bytes = clib_min (n_segs_now * tc->snd_mss, burst_bytes);
+      transport_connection_tx_pacer_update_bytes (&tc->connection,
+                                                 sent_bytes);
+      n_segs += n_segs_now;
     }
-  vec_reset_length (wrk->ongoing_fast_rxt);
+  _vec_len (ongoing_fast_rxt) = 0;
+  wrk->ongoing_fast_rxt = ongoing_fast_rxt;
 }
 
 /**
@@ -1274,7 +1320,7 @@ tcp_cc_handle_event (tcp_connection_t * tc, u32 is_dack)
     {
       if (tc->bytes_acked)
        goto partial_ack;
-      tcp_program_fastretransmit (tc);
+      tcp_program_fastretransmit (tcp_get_worker (tc->c_thread_index), tc);
       return;
     }
   /*
@@ -1298,6 +1344,8 @@ tcp_cc_handle_event (tcp_connection_t * tc, u32 is_dack)
        }
       else if (tcp_should_fastrecover (tc))
        {
+         u32 pacer_wnd;
+
          ASSERT (!tcp_in_fastrecovery (tc));
 
          /* Heuristic to catch potential late dupacks
@@ -1313,9 +1361,24 @@ tcp_cc_handle_event (tcp_connection_t * tc, u32 is_dack)
          tc->cc_algo->rcv_cong_ack (tc, TCP_CC_DUPACK);
 
          if (tcp_opts_sack_permitted (&tc->rcv_opts))
-           tc->sack_sb.high_rxt = tc->snd_una;
+           {
+             tc->cwnd = tc->ssthresh;
+             scoreboard_init_high_rxt (&tc->sack_sb, tc->snd_una);
+           }
+         else
+           {
+             /* Post retransmit update cwnd to ssthresh and account for the
+              * three segments that have left the network and should've been
+              * buffered at the receiver XXX */
+             tc->cwnd = tc->ssthresh + 3 * tc->snd_mss;
+           }
 
-         tcp_program_fastretransmit (tc);
+         /* Constrain rate until we get a partial ack */
+         pacer_wnd = clib_max (0.1 * tc->cwnd, 2 * tc->snd_mss);
+         tcp_connection_tx_pacer_reset (tc, pacer_wnd,
+                                        0 /* start bucket */ );
+         tcp_program_fastretransmit (tcp_get_worker (tc->c_thread_index),
+                                     tc);
          return;
        }
       else if (!tc->bytes_acked
@@ -1360,6 +1423,10 @@ partial_ack:
    * Legitimate ACK. 1) See if we can exit recovery
    */
 
+  /* Update the pacing rate. For the first partial ack we move from
+   * the artificially constrained rate to the one after congestion */
+  tcp_connection_tx_pacer_update (tc);
+
   if (seq_geq (tc->snd_una, tc->snd_congestion))
     {
       tcp_retransmit_timer_update (tc);
@@ -1383,7 +1450,7 @@ partial_ack:
    */
 
   /* XXX limit this only to first partial ack? */
-  tcp_retransmit_timer_force_update (tc);
+  tcp_retransmit_timer_update (tc);
 
   /* RFC6675: If the incoming ACK is a cumulative acknowledgment,
    * reset dupacks to 0. Also needed if in congestion recovery */
@@ -1417,10 +1484,14 @@ partial_ack:
        {
          /* Apparently all retransmitted holes have been acked */
          tc->snd_rxt_bytes = 0;
+         tc->sack_sb.high_rxt = tc->snd_una;
        }
     }
   else
     {
+      tcp_fastrecovery_first_on (tc);
+      /* Reuse last bytes delivered to track total bytes acked */
+      tc->sack_sb.last_bytes_delivered += tc->bytes_acked;
       if (tc->snd_rxt_bytes > tc->bytes_acked)
        tc->snd_rxt_bytes -= tc->bytes_acked;
       else
@@ -1432,15 +1503,15 @@ partial_ack:
   /*
    * Since this was a partial ack, try to retransmit some more data
    */
-  tcp_program_fastretransmit (tc);
+  tcp_program_fastretransmit (tcp_get_worker (tc->c_thread_index), tc);
 }
 
 /**
  * Process incoming ACK
  */
 static int
-tcp_rcv_ack (tcp_connection_t * tc, vlib_buffer_t * b,
-            tcp_header_t * th, u32 * next, u32 * error)
+tcp_rcv_ack (tcp_worker_ctx_t * wrk, tcp_connection_t * tc, vlib_buffer_t * b,
+            tcp_header_t * th, u32 * error)
 {
   u32 prev_snd_wnd, prev_snd_una;
   u8 is_dack;
@@ -1466,9 +1537,8 @@ tcp_rcv_ack (tcp_connection_t * tc, vlib_buffer_t * b,
        * drop it */
       if (seq_gt (vnet_buffer (b)->tcp.ack_number, tc->snd_una_max))
        {
-         tcp_make_ack (tc, b);
-         *next = tcp_next_output (tc->c_is_ip4);
-         *error = TCP_ERROR_ACK_INVALID;
+         tcp_program_ack (wrk, tc);
+         *error = TCP_ERROR_ACK_FUTURE;
          TCP_EVT_DBG (TCP_EVT_ACK_RCV_ERR, tc, 0,
                       vnet_buffer (b)->tcp.ack_number);
          return -1;
@@ -1478,7 +1548,6 @@ tcp_rcv_ack (tcp_connection_t * tc, vlib_buffer_t * b,
                   vnet_buffer (b)->tcp.ack_number);
 
       tc->snd_nxt = vnet_buffer (b)->tcp.ack_number;
-      *error = TCP_ERROR_ACK_FUTURE;
     }
 
   /* If old ACK, probably it's an old dupack */
@@ -1510,7 +1579,10 @@ process_ack:
   tcp_validate_txf_size (tc, tc->bytes_acked);
 
   if (tc->bytes_acked)
-    tcp_dequeue_acked (tc, vnet_buffer (b)->tcp.ack_number);
+    {
+      tcp_program_dequeue (wrk, tc);
+      tcp_update_rtt (tc, vnet_buffer (b)->tcp.ack_number);
+    }
 
   TCP_EVT_DBG (TCP_EVT_ACK_RCVD, tc);
 
@@ -1522,7 +1594,10 @@ process_ack:
     {
       tcp_cc_handle_event (tc, is_dack);
       if (!tcp_in_cong_recovery (tc))
-       return 0;
+       {
+         *error = TCP_ERROR_ACK_OK;
+         return 0;
+       }
       *error = TCP_ERROR_ACK_DUP;
       if (vnet_buffer (b)->tcp.data_len || tcp_is_fin (th))
        return 0;
@@ -1537,6 +1612,54 @@ process_ack:
   return 0;
 }
 
+static void
+tcp_program_disconnect (tcp_worker_ctx_t * wrk, tcp_connection_t * tc)
+{
+  if (!tcp_disconnect_pending (tc))
+    {
+      vec_add1 (wrk->pending_disconnects, tc->c_c_index);
+      tcp_disconnect_pending_on (tc);
+    }
+}
+
+static void
+tcp_handle_disconnects (tcp_worker_ctx_t * wrk)
+{
+  u32 thread_index, *pending_disconnects;
+  tcp_connection_t *tc;
+  int i;
+
+  if (!vec_len (wrk->pending_disconnects))
+    return;
+
+  thread_index = wrk->vm->thread_index;
+  pending_disconnects = wrk->pending_disconnects;
+  for (i = 0; i < vec_len (pending_disconnects); i++)
+    {
+      tc = tcp_connection_get (pending_disconnects[i], thread_index);
+      tcp_disconnect_pending_off (tc);
+      stream_session_disconnect_notify (&tc->connection);
+    }
+  _vec_len (wrk->pending_disconnects) = 0;
+}
+
+static void
+tcp_rcv_fin (tcp_worker_ctx_t * wrk, tcp_connection_t * tc, vlib_buffer_t * b,
+            u32 * error)
+{
+  /* Enter CLOSE-WAIT and notify session. To avoid lingering
+   * in CLOSE-WAIT, set timer (reuse WAITCLOSE). */
+  /* Account for the FIN if nothing else was received */
+  if (vnet_buffer (b)->tcp.data_len == 0)
+    tc->rcv_nxt += 1;
+  tcp_program_ack (wrk, tc);
+  tc->state = TCP_STATE_CLOSE_WAIT;
+  tcp_program_disconnect (wrk, tc);
+  tcp_timer_update (tc, TCP_TIMER_WAITCLOSE, TCP_CLOSEWAIT_TIME);
+  TCP_EVT_DBG (TCP_EVT_FIN_RCVD, tc);
+  *error = TCP_ERROR_FIN_RCVD;
+}
+
 static u8
 tcp_sack_vector_is_sane (sack_block_t * sacks)
 {
@@ -1646,25 +1769,16 @@ tcp_session_enqueue_data (tcp_connection_t * tc, vlib_buffer_t * b,
   else if (written > data_len)
     {
       tc->rcv_nxt += written;
-
-      /* Send ACK confirming the update */
-      tc->flags |= TCP_CONN_SNDACK;
       TCP_EVT_DBG (TCP_EVT_CC_INPUT, tc, data_len, written);
     }
   else if (written > 0)
     {
       /* We've written something but FIFO is probably full now */
       tc->rcv_nxt += written;
-
-      /* Depending on how fast the app is, all remaining buffers in burst will
-       * not be enqueued. Inform peer */
-      tc->flags |= TCP_CONN_SNDACK;
-
       error = TCP_ERROR_PARTIALLY_ENQUEUED;
     }
   else
     {
-      tc->flags |= TCP_CONN_SNDACK;
       return TCP_ERROR_FIFO_FULL;
     }
 
@@ -1783,7 +1897,8 @@ tcp_buffer_discard_bytes (vlib_buffer_t * b, u32 n_bytes_to_drop)
  * It handles both in order or out-of-order data.
  */
 static int
-tcp_segment_rcv (tcp_connection_t * tc, vlib_buffer_t * b, u32 * next0)
+tcp_segment_rcv (tcp_worker_ctx_t * wrk, tcp_connection_t * tc,
+                vlib_buffer_t * b)
 {
   u32 error, n_bytes_to_drop, n_data_bytes;
 
@@ -1802,9 +1917,8 @@ tcp_segment_rcv (tcp_connection_t * tc, vlib_buffer_t * b, u32 * next0)
           * retransmissions since we may not have any data to send */
          if (seq_leq (vnet_buffer (b)->tcp.seq_end, tc->rcv_nxt))
            {
-             tcp_make_ack (tc, b);
+             tcp_program_ack (wrk, tc);
              error = TCP_ERROR_SEGMENT_OLD;
-             *next0 = tcp_next_output (tc->c_is_ip4);
              goto done;
            }
 
@@ -1816,7 +1930,6 @@ tcp_segment_rcv (tcp_connection_t * tc, vlib_buffer_t * b, u32 * next0)
          if (tcp_buffer_discard_bytes (b, n_bytes_to_drop))
            {
              error = TCP_ERROR_SEGMENT_OLD;
-             *next0 = tcp_next_drop (tc->c_is_ip4);
              goto done;
            }
          goto in_order;
@@ -1824,9 +1937,7 @@ tcp_segment_rcv (tcp_connection_t * tc, vlib_buffer_t * b, u32 * next0)
 
       /* RFC2581: Enqueue and send DUPACK for fast retransmit */
       error = tcp_session_enqueue_ooo (tc, b, n_data_bytes);
-      *next0 = tcp_next_output (tc->c_is_ip4);
-      tcp_make_ack (tc, b);
-      vnet_buffer (b)->tcp.flags = TCP_BUF_FLAG_DUPACK;
+      tcp_program_dupack (wrk, tc);
       TCP_EVT_DBG (TCP_EVT_DUPACK_SENT, tc, vnet_buffer (b)->tcp);
       goto done;
     }
@@ -1838,14 +1949,12 @@ in_order:
   error = tcp_session_enqueue_data (tc, b, n_data_bytes);
   if (tcp_can_delack (tc))
     {
-      *next0 = tcp_next_drop (tc->c_is_ip4);
       if (!tcp_timer_is_active (tc, TCP_TIMER_DELACK))
        tcp_timer_set (tc, TCP_TIMER_DELACK, TCP_DELACK_TIME);
       goto done;
     }
 
-  *next0 = tcp_next_output (tc->c_is_ip4);
-  tcp_make_ack (tc, b);
+  tcp_program_ack (wrk, tc);
 
 done:
   return error;
@@ -1894,13 +2003,14 @@ tcp_set_rx_trace_data (tcp_rx_trace_t * t0, tcp_connection_t * tc0,
 {
   if (tc0)
     {
-      clib_memcpy (&t0->tcp_connection, tc0, sizeof (t0->tcp_connection));
+      clib_memcpy_fast (&t0->tcp_connection, tc0,
+                       sizeof (t0->tcp_connection));
     }
   else
     {
       th0 = tcp_buffer_hdr (b0);
     }
-  clib_memcpy (&t0->tcp_header, th0, sizeof (t0->tcp_header));
+  clib_memcpy_fast (&t0->tcp_header, th0, sizeof (t0->tcp_header));
 }
 
 static void
@@ -1980,122 +2090,89 @@ tcp46_established_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
                          vlib_frame_t * frame, int is_ip4)
 {
   u32 thread_index = vm->thread_index, errors = 0;
-  u32 n_left_from, next_index, *from, *to_next;
+  tcp_worker_ctx_t *wrk = tcp_get_worker (thread_index);
+  u32 n_left_from, *from, *first_buffer;
   u16 err_counters[TCP_N_ERROR] = { 0 };
   u8 is_fin = 0;
 
   if (node->flags & VLIB_NODE_FLAG_TRACE)
     tcp_established_trace_frame (vm, node, frame, is_ip4);
 
-  from = vlib_frame_vector_args (frame);
+  first_buffer = from = vlib_frame_vector_args (frame);
   n_left_from = frame->n_vectors;
-  next_index = node->cached_next_index;
 
   while (n_left_from > 0)
     {
-      u32 n_left_to_next;
+      u32 bi0, error0 = TCP_ERROR_ACK_OK;
+      vlib_buffer_t *b0;
+      tcp_header_t *th0 = 0;
+      tcp_connection_t *tc0;
 
-      vlib_get_next_frame (vm, node, next_index, to_next, n_left_to_next);
-      while (n_left_from > 0 && n_left_to_next > 0)
+      if (n_left_from > 1)
        {
-         u32 bi0;
-         vlib_buffer_t *b0;
-         tcp_header_t *th0 = 0;
-         tcp_connection_t *tc0;
-         u32 next0 = tcp_next_drop (is_ip4), error0 = TCP_ERROR_ACK_OK;
-
-         if (n_left_from > 1)
-           {
-             vlib_buffer_t *pb;
-             pb = vlib_get_buffer (vm, from[1]);
-             vlib_prefetch_buffer_header (pb, LOAD);
-             CLIB_PREFETCH (pb->data, 2 * CLIB_CACHE_LINE_BYTES, LOAD);
-           }
-
-         bi0 = from[0];
-         to_next[0] = bi0;
-         from += 1;
-         to_next += 1;
-         n_left_from -= 1;
-         n_left_to_next -= 1;
+         vlib_buffer_t *pb;
+         pb = vlib_get_buffer (vm, from[1]);
+         vlib_prefetch_buffer_header (pb, LOAD);
+         CLIB_PREFETCH (pb->data, 2 * CLIB_CACHE_LINE_BYTES, LOAD);
+       }
 
-         b0 = vlib_get_buffer (vm, bi0);
-         tc0 = tcp_connection_get (vnet_buffer (b0)->tcp.connection_index,
-                                   thread_index);
+      bi0 = from[0];
+      from += 1;
+      n_left_from -= 1;
 
-         if (PREDICT_FALSE (tc0 == 0))
-           {
-             error0 = TCP_ERROR_INVALID_CONNECTION;
-             goto done;
-           }
+      b0 = vlib_get_buffer (vm, bi0);
+      tc0 = tcp_connection_get (vnet_buffer (b0)->tcp.connection_index,
+                               thread_index);
 
-         th0 = tcp_buffer_hdr (b0);
-         /* N.B. buffer is rewritten if segment is ooo. Thus, th0 becomes a
-          * dangling reference. */
-         is_fin = tcp_is_fin (th0);
+      if (PREDICT_FALSE (tc0 == 0))
+       {
+         error0 = TCP_ERROR_INVALID_CONNECTION;
+         goto done;
+       }
 
-         /* SYNs, FINs and data consume sequence numbers */
-         vnet_buffer (b0)->tcp.seq_end = vnet_buffer (b0)->tcp.seq_number
-           + tcp_is_syn (th0) + is_fin + vnet_buffer (b0)->tcp.data_len;
+      th0 = tcp_buffer_hdr (b0);
+      /* N.B. buffer is rewritten if segment is ooo. Thus, th0 becomes a
+       * dangling reference. */
+      is_fin = tcp_is_fin (th0);
 
-         /* TODO header prediction fast path */
+      /* SYNs, FINs and data consume sequence numbers */
+      vnet_buffer (b0)->tcp.seq_end = vnet_buffer (b0)->tcp.seq_number
+       + tcp_is_syn (th0) + is_fin + vnet_buffer (b0)->tcp.data_len;
 
-         /* 1-4: check SEQ, RST, SYN */
-         if (PREDICT_FALSE (tcp_segment_validate (vm, tc0, b0, th0, &next0,
-                                                  &error0)))
-           {
-             tcp_maybe_inc_err_counter (err_counters, error0);
-             TCP_EVT_DBG (TCP_EVT_SEG_INVALID, tc0, vnet_buffer (b0)->tcp);
-             goto done;
-           }
+      /* TODO header prediction fast path */
 
-         /* 5: check the ACK field  */
-         if (PREDICT_FALSE (tcp_rcv_ack (tc0, b0, th0, &next0, &error0)))
-           {
-             tcp_maybe_inc_err_counter (err_counters, error0);
-             goto done;
-           }
+      /* 1-4: check SEQ, RST, SYN */
+      if (PREDICT_FALSE (tcp_segment_validate (wrk, tc0, b0, th0, &error0)))
+       {
+         TCP_EVT_DBG (TCP_EVT_SEG_INVALID, tc0, vnet_buffer (b0)->tcp);
+         goto done;
+       }
 
-         /* 6: check the URG bit TODO */
+      /* 5: check the ACK field  */
+      if (PREDICT_FALSE (tcp_rcv_ack (wrk, tc0, b0, th0, &error0)))
+       goto done;
 
-         /* 7: process the segment text */
-         if (vnet_buffer (b0)->tcp.data_len)
-           {
-             error0 = tcp_segment_rcv (tc0, b0, &next0);
-             tcp_maybe_inc_err_counter (err_counters, error0);
-           }
+      /* 6: check the URG bit TODO */
 
-         /* 8: check the FIN bit */
-         if (PREDICT_FALSE (is_fin))
-           {
-             /* Enter CLOSE-WAIT and notify session. To avoid lingering
-              * in CLOSE-WAIT, set timer (reuse WAITCLOSE). */
-             /* Account for the FIN if nothing else was received */
-             if (vnet_buffer (b0)->tcp.data_len == 0)
-               tc0->rcv_nxt += 1;
-             tcp_make_ack (tc0, b0);
-             next0 = tcp_next_output (tc0->c_is_ip4);
-             tc0->state = TCP_STATE_CLOSE_WAIT;
-             stream_session_disconnect_notify (&tc0->connection);
-             tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_CLOSEWAIT_TIME);
-             TCP_EVT_DBG (TCP_EVT_FIN_RCVD, tc0);
-             tcp_inc_err_counter (err_counters, TCP_ERROR_FIN_RCVD, 1);
-           }
+      /* 7: process the segment text */
+      if (vnet_buffer (b0)->tcp.data_len)
+       error0 = tcp_segment_rcv (wrk, tc0, b0);
 
-       done:
-         b0->error = node->errors[error0];
-         vlib_validate_buffer_enqueue_x1 (vm, node, next_index, to_next,
-                                          n_left_to_next, bi0, next0);
-       }
+      /* 8: check the FIN bit */
+      if (PREDICT_FALSE (is_fin))
+       tcp_rcv_fin (wrk, tc0, b0, &error0);
 
-      vlib_put_next_frame (vm, node, next_index, n_left_to_next);
+    done:
+      tcp_inc_err_counter (err_counters, error0, 1);
     }
 
   errors = session_manager_flush_enqueue_events (TRANSPORT_PROTO_TCP,
                                                 thread_index);
-  err_counters[TCP_ERROR_EVENT_FIFO_FULL] = errors;
+  err_counters[TCP_ERROR_MSG_QUEUE_FULL] = errors;
   tcp_store_err_counters (established, err_counters);
-  tcp_flush_frame_to_output (vm, thread_index, is_ip4);
+  tcp_handle_postponed_dequeues (wrk);
+  tcp_handle_disconnects (wrk);
+  vlib_buffer_free (vm, first_buffer, frame->n_vectors);
 
   return frame->n_vectors;
 }
@@ -2246,262 +2323,254 @@ tcp46_syn_sent_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
                       vlib_frame_t * from_frame, int is_ip4)
 {
   tcp_main_t *tm = vnet_get_tcp_main ();
-  u32 n_left_from, next_index, *from, *to_next;
-  u32 my_thread_index = vm->thread_index, errors = 0;
+  u32 n_left_from, *from, *first_buffer, errors = 0;
+  u32 my_thread_index = vm->thread_index;
+  tcp_worker_ctx_t *wrk = tcp_get_worker (my_thread_index);
 
-  from = vlib_frame_vector_args (from_frame);
+  from = first_buffer = vlib_frame_vector_args (from_frame);
   n_left_from = from_frame->n_vectors;
 
-  next_index = node->cached_next_index;
-
   while (n_left_from > 0)
     {
-      u32 n_left_to_next;
+      u32 bi0, ack0, seq0, error0 = TCP_ERROR_NONE;
+      tcp_connection_t *tc0, *new_tc0;
+      tcp_header_t *tcp0 = 0;
+      tcp_rx_trace_t *t0;
+      vlib_buffer_t *b0;
 
-      vlib_get_next_frame (vm, node, next_index, to_next, n_left_to_next);
+      bi0 = from[0];
+      from += 1;
+      n_left_from -= 1;
 
-      while (n_left_from > 0 && n_left_to_next > 0)
+      b0 = vlib_get_buffer (vm, bi0);
+      tc0 =
+       tcp_half_open_connection_get (vnet_buffer (b0)->tcp.connection_index);
+      if (PREDICT_FALSE (tc0 == 0))
        {
-         u32 bi0, ack0, seq0;
-         vlib_buffer_t *b0;
-         tcp_rx_trace_t *t0;
-         tcp_header_t *tcp0 = 0;
-         tcp_connection_t *tc0;
-         tcp_connection_t *new_tc0;
-         u32 next0 = tcp_next_drop (is_ip4), error0 = TCP_ERROR_ENQUEUED;
-
-         bi0 = from[0];
-         to_next[0] = bi0;
-         from += 1;
-         to_next += 1;
-         n_left_from -= 1;
-         n_left_to_next -= 1;
-
-         b0 = vlib_get_buffer (vm, bi0);
-         tc0 =
-           tcp_half_open_connection_get (vnet_buffer (b0)->
-                                         tcp.connection_index);
-         if (PREDICT_FALSE (tc0 == 0))
-           {
-             error0 = TCP_ERROR_INVALID_CONNECTION;
-             goto drop;
-           }
-
-         /* Half-open completed recently but the connection was't removed
-          * yet by the owning thread */
-         if (PREDICT_FALSE (tc0->flags & TCP_CONN_HALF_OPEN_DONE))
-           {
-             /* Make sure the connection actually exists */
-             ASSERT (tcp_lookup_connection (tc0->c_fib_index, b0,
-                                            my_thread_index, is_ip4));
-             goto drop;
-           }
-
-         ack0 = vnet_buffer (b0)->tcp.ack_number;
-         seq0 = vnet_buffer (b0)->tcp.seq_number;
-         tcp0 = tcp_buffer_hdr (b0);
-
-         /* Crude check to see if the connection handle does not match
-          * the packet. Probably connection just switched to established */
-         if (PREDICT_FALSE (tcp0->dst_port != tc0->c_lcl_port
-                            || tcp0->src_port != tc0->c_rmt_port))
-           goto drop;
+         error0 = TCP_ERROR_INVALID_CONNECTION;
+         goto drop;
+       }
 
-         if (PREDICT_FALSE
-             (!tcp_ack (tcp0) && !tcp_rst (tcp0) && !tcp_syn (tcp0)))
-           goto drop;
+      /* Half-open completed recently but the connection was't removed
+       * yet by the owning thread */
+      if (PREDICT_FALSE (tc0->flags & TCP_CONN_HALF_OPEN_DONE))
+       {
+         /* Make sure the connection actually exists */
+         ASSERT (tcp_lookup_connection (tc0->c_fib_index, b0,
+                                        my_thread_index, is_ip4));
+         goto drop;
+       }
 
-         /* SYNs, FINs and data consume sequence numbers */
-         vnet_buffer (b0)->tcp.seq_end = seq0 + tcp_is_syn (tcp0)
-           + tcp_is_fin (tcp0) + vnet_buffer (b0)->tcp.data_len;
+      ack0 = vnet_buffer (b0)->tcp.ack_number;
+      seq0 = vnet_buffer (b0)->tcp.seq_number;
+      tcp0 = tcp_buffer_hdr (b0);
 
-         /*
-          *  1. check the ACK bit
-          */
+      /* Crude check to see if the connection handle does not match
+       * the packet. Probably connection just switched to established */
+      if (PREDICT_FALSE (tcp0->dst_port != tc0->c_lcl_port
+                        || tcp0->src_port != tc0->c_rmt_port))
+       {
+         error0 = TCP_ERROR_INVALID_CONNECTION;
+         goto drop;
+       }
 
-         /*
-          *   If the ACK bit is set
-          *     If SEG.ACK =< ISS, or SEG.ACK > SND.NXT, send a reset (unless
-          *     the RST bit is set, if so drop the segment and return)
-          *       <SEQ=SEG.ACK><CTL=RST>
-          *     and discard the segment.  Return.
-          *     If SND.UNA =< SEG.ACK =< SND.NXT then the ACK is acceptable.
-          */
-         if (tcp_ack (tcp0))
-           {
-             if (seq_leq (ack0, tc0->iss) || seq_gt (ack0, tc0->snd_nxt))
-               {
-                 clib_warning ("ack not in rcv wnd");
-                 if (!tcp_rst (tcp0))
-                   tcp_send_reset_w_pkt (tc0, b0, is_ip4);
-                 goto drop;
-               }
+      if (PREDICT_FALSE (!tcp_ack (tcp0) && !tcp_rst (tcp0)
+                        && !tcp_syn (tcp0)))
+       {
+         error0 = TCP_ERROR_SEGMENT_INVALID;
+         goto drop;
+       }
 
-             /* Make sure ACK is valid */
-             if (seq_gt (tc0->snd_una, ack0))
-               {
-                 clib_warning ("ack invalid");
-                 goto drop;
-               }
-           }
+      /* SYNs, FINs and data consume sequence numbers */
+      vnet_buffer (b0)->tcp.seq_end =
+       seq0 + tcp_is_syn (tcp0) + tcp_is_fin (tcp0) +
+       vnet_buffer (b0)->tcp.data_len;
 
-         /*
-          * 2. check the RST bit
-          */
+      /*
+       *  1. check the ACK bit
+       */
 
-         if (tcp_rst (tcp0))
+      /*
+       *   If the ACK bit is set
+       *     If SEG.ACK =< ISS, or SEG.ACK > SND.NXT, send a reset (unless
+       *     the RST bit is set, if so drop the segment and return)
+       *       <SEQ=SEG.ACK><CTL=RST>
+       *     and discard the segment.  Return.
+       *     If SND.UNA =< SEG.ACK =< SND.NXT then the ACK is acceptable.
+       */
+      if (tcp_ack (tcp0))
+       {
+         if (seq_leq (ack0, tc0->iss) || seq_gt (ack0, tc0->snd_nxt))
            {
-             /* If ACK is acceptable, signal client that peer is not
-              * willing to accept connection and drop connection*/
-             if (tcp_ack (tcp0))
-               tcp_connection_reset (tc0);
+             if (!tcp_rst (tcp0))
+               tcp_send_reset_w_pkt (tc0, b0, is_ip4);
+             error0 = TCP_ERROR_RCV_WND;
              goto drop;
            }
 
-         /*
-          * 3. check the security and precedence (skipped)
-          */
-
-         /*
-          * 4. check the SYN bit
-          */
-
-         /* No SYN flag. Drop. */
-         if (!tcp_syn (tcp0))
+         /* Make sure ACK is valid */
+         if (seq_gt (tc0->snd_una, ack0))
            {
-             clib_warning ("not synack");
+             error0 = TCP_ERROR_ACK_INVALID;
              goto drop;
            }
+       }
 
-         /* Parse options */
-         if (tcp_options_parse (tcp0, &tc0->rcv_opts))
-           {
-             clib_warning ("options parse fail");
-             goto drop;
-           }
+      /*
+       * 2. check the RST bit
+       */
 
-         /* Valid SYN or SYN-ACK. Move connection from half-open pool to
-          * current thread pool. */
-         pool_get (tm->connections[my_thread_index], new_tc0);
-         clib_memcpy (new_tc0, tc0, sizeof (*new_tc0));
-         new_tc0->c_c_index = new_tc0 - tm->connections[my_thread_index];
-         new_tc0->c_thread_index = my_thread_index;
-         new_tc0->rcv_nxt = vnet_buffer (b0)->tcp.seq_end;
-         new_tc0->irs = seq0;
-         new_tc0->timers[TCP_TIMER_ESTABLISH] = TCP_TIMER_HANDLE_INVALID;
-         new_tc0->timers[TCP_TIMER_RETRANSMIT_SYN] =
-           TCP_TIMER_HANDLE_INVALID;
-         new_tc0->sw_if_index = vnet_buffer (b0)->sw_if_index[VLIB_RX];
-
-         /* If this is not the owning thread, wait for syn retransmit to
-          * expire and cleanup then */
-         if (tcp_half_open_connection_cleanup (tc0))
-           tc0->flags |= TCP_CONN_HALF_OPEN_DONE;
-
-         if (tcp_opts_tstamp (&new_tc0->rcv_opts))
-           {
-             new_tc0->tsval_recent = new_tc0->rcv_opts.tsval;
-             new_tc0->tsval_recent_age = tcp_time_now ();
-           }
+      if (tcp_rst (tcp0))
+       {
+         /* If ACK is acceptable, signal client that peer is not
+          * willing to accept connection and drop connection*/
+         if (tcp_ack (tcp0))
+           tcp_connection_reset (tc0);
+         error0 = TCP_ERROR_RST_RCVD;
+         goto drop;
+       }
 
-         if (tcp_opts_wscale (&new_tc0->rcv_opts))
-           new_tc0->snd_wscale = new_tc0->rcv_opts.wscale;
+      /*
+       * 3. check the security and precedence (skipped)
+       */
 
-         new_tc0->snd_wnd = clib_net_to_host_u16 (tcp0->window)
-           << new_tc0->snd_wscale;
-         new_tc0->snd_wl1 = seq0;
-         new_tc0->snd_wl2 = ack0;
+      /*
+       * 4. check the SYN bit
+       */
 
-         tcp_connection_init_vars (new_tc0);
+      /* No SYN flag. Drop. */
+      if (!tcp_syn (tcp0))
+       {
+         clib_warning ("not synack");
+         error0 = TCP_ERROR_SEGMENT_INVALID;
+         goto drop;
+       }
 
-         /* SYN-ACK: See if we can switch to ESTABLISHED state */
-         if (PREDICT_TRUE (tcp_ack (tcp0)))
-           {
-             /* Our SYN is ACKed: we have iss < ack = snd_una */
+      /* Parse options */
+      if (tcp_options_parse (tcp0, &tc0->rcv_opts, 1))
+       {
+         clib_warning ("options parse fail");
+         error0 = TCP_ERROR_OPTIONS;
+         goto drop;
+       }
 
-             /* TODO Dequeue acknowledged segments if we support Fast Open */
-             new_tc0->snd_una = ack0;
-             new_tc0->state = TCP_STATE_ESTABLISHED;
+      /* Valid SYN or SYN-ACK. Move connection from half-open pool to
+       * current thread pool. */
+      pool_get (tm->connections[my_thread_index], new_tc0);
+      clib_memcpy_fast (new_tc0, tc0, sizeof (*new_tc0));
+      new_tc0->c_c_index = new_tc0 - tm->connections[my_thread_index];
+      new_tc0->c_thread_index = my_thread_index;
+      new_tc0->rcv_nxt = vnet_buffer (b0)->tcp.seq_end;
+      new_tc0->irs = seq0;
+      new_tc0->timers[TCP_TIMER_ESTABLISH] = TCP_TIMER_HANDLE_INVALID;
+      new_tc0->timers[TCP_TIMER_RETRANSMIT_SYN] = TCP_TIMER_HANDLE_INVALID;
+      new_tc0->sw_if_index = vnet_buffer (b0)->sw_if_index[VLIB_RX];
+
+      /* If this is not the owning thread, wait for syn retransmit to
+       * expire and cleanup then */
+      if (tcp_half_open_connection_cleanup (tc0))
+       tc0->flags |= TCP_CONN_HALF_OPEN_DONE;
+
+      if (tcp_opts_tstamp (&new_tc0->rcv_opts))
+       {
+         new_tc0->tsval_recent = new_tc0->rcv_opts.tsval;
+         new_tc0->tsval_recent_age = tcp_time_now ();
+       }
 
-             /* Make sure las is initialized for the wnd computation */
-             new_tc0->rcv_las = new_tc0->rcv_nxt;
+      if (tcp_opts_wscale (&new_tc0->rcv_opts))
+       new_tc0->snd_wscale = new_tc0->rcv_opts.wscale;
+      else
+       new_tc0->rcv_wscale = 0;
 
-             /* Notify app that we have connection. If session layer can't
-              * allocate session send reset */
-             if (session_stream_connect_notify (&new_tc0->connection, 0))
-               {
-                 clib_warning ("connect notify fail");
-                 tcp_send_reset_w_pkt (new_tc0, b0, is_ip4);
-                 tcp_connection_cleanup (new_tc0);
-                 goto drop;
-               }
+      new_tc0->snd_wnd = clib_net_to_host_u16 (tcp0->window)
+       << new_tc0->snd_wscale;
+      new_tc0->snd_wl1 = seq0;
+      new_tc0->snd_wl2 = ack0;
 
-             /* Make sure after data segment processing ACK is sent */
-             new_tc0->flags |= TCP_CONN_SNDACK;
+      tcp_connection_init_vars (new_tc0);
 
-             /* Update rtt with the syn-ack sample */
-             tcp_update_rtt (new_tc0, vnet_buffer (b0)->tcp.ack_number);
-             TCP_EVT_DBG (TCP_EVT_SYNACK_RCVD, new_tc0);
-           }
-         /* SYN: Simultaneous open. Change state to SYN-RCVD and send SYN-ACK */
-         else
-           {
-             new_tc0->state = TCP_STATE_SYN_RCVD;
+      /* SYN-ACK: See if we can switch to ESTABLISHED state */
+      if (PREDICT_TRUE (tcp_ack (tcp0)))
+       {
+         /* Our SYN is ACKed: we have iss < ack = snd_una */
 
-             /* Notify app that we have connection */
-             if (session_stream_connect_notify (&new_tc0->connection, 0))
-               {
-                 tcp_connection_cleanup (new_tc0);
-                 tcp_send_reset_w_pkt (tc0, b0, is_ip4);
-                 TCP_EVT_DBG (TCP_EVT_RST_SENT, tc0);
-                 goto drop;
-               }
+         /* TODO Dequeue acknowledged segments if we support Fast Open */
+         new_tc0->snd_una = ack0;
+         new_tc0->state = TCP_STATE_ESTABLISHED;
 
-             tc0->rtt_ts = 0;
-             tcp_init_snd_vars (tc0);
-             tcp_make_synack (new_tc0, b0);
-             next0 = tcp_next_output (is_ip4);
+         /* Make sure las is initialized for the wnd computation */
+         new_tc0->rcv_las = new_tc0->rcv_nxt;
 
-             goto drop;
-           }
-
-         /* Read data, if any */
-         if (PREDICT_FALSE (vnet_buffer (b0)->tcp.data_len))
-           {
-             clib_warning ("rcvd data in syn-sent");
-             error0 = tcp_segment_rcv (new_tc0, b0, &next0);
-             if (error0 == TCP_ERROR_ACK_OK)
-               error0 = TCP_ERROR_SYN_ACKS_RCVD;
-             tcp_maybe_inc_counter (syn_sent, error0, 1);
-           }
-         else
+         /* Notify app that we have connection. If session layer can't
+          * allocate session send reset */
+         if (session_stream_connect_notify (&new_tc0->connection, 0))
            {
-             tcp_make_ack (new_tc0, b0);
-             next0 = tcp_next_output (new_tc0->c_is_ip4);
+             clib_warning ("connect notify fail");
+             tcp_send_reset_w_pkt (new_tc0, b0, is_ip4);
+             tcp_connection_cleanup (new_tc0);
+             goto drop;
            }
 
-       drop:
+         new_tc0->tx_fifo_size =
+           transport_tx_fifo_size (&new_tc0->connection);
+         /* Update rtt with the syn-ack sample */
+         tcp_estimate_initial_rtt (new_tc0);
+         TCP_EVT_DBG (TCP_EVT_SYNACK_RCVD, new_tc0);
+         error0 = TCP_ERROR_SYN_ACKS_RCVD;
+       }
+      /* SYN: Simultaneous open. Change state to SYN-RCVD and send SYN-ACK */
+      else
+       {
+         new_tc0->state = TCP_STATE_SYN_RCVD;
 
-         b0->error = error0 ? node->errors[error0] : 0;
-         if (PREDICT_FALSE
-             ((b0->flags & VLIB_BUFFER_IS_TRACED) && tcp0 != 0))
+         /* Notify app that we have connection */
+         if (session_stream_connect_notify (&new_tc0->connection, 0))
            {
-             t0 = vlib_add_trace (vm, node, b0, sizeof (*t0));
-             clib_memcpy (&t0->tcp_header, tcp0, sizeof (t0->tcp_header));
-             clib_memcpy (&t0->tcp_connection, tc0,
-                          sizeof (t0->tcp_connection));
+             tcp_connection_cleanup (new_tc0);
+             tcp_send_reset_w_pkt (tc0, b0, is_ip4);
+             TCP_EVT_DBG (TCP_EVT_RST_SENT, tc0);
+             goto drop;
            }
 
-         vlib_validate_buffer_enqueue_x1 (vm, node, next_index, to_next,
-                                          n_left_to_next, bi0, next0);
+         new_tc0->tx_fifo_size =
+           transport_tx_fifo_size (&new_tc0->connection);
+         new_tc0->rtt_ts = 0;
+         tcp_init_snd_vars (new_tc0);
+         tcp_send_synack (new_tc0);
+         error0 = TCP_ERROR_SYNS_RCVD;
+         goto drop;
+       }
+
+      /* Read data, if any */
+      if (PREDICT_FALSE (vnet_buffer (b0)->tcp.data_len))
+       {
+         clib_warning ("rcvd data in syn-sent");
+         error0 = tcp_segment_rcv (wrk, new_tc0, b0);
+         if (error0 == TCP_ERROR_ACK_OK)
+           error0 = TCP_ERROR_SYN_ACKS_RCVD;
+       }
+      else
+       {
+         tcp_program_ack (wrk, new_tc0);
        }
 
-      vlib_put_next_frame (vm, node, next_index, n_left_to_next);
+    drop:
+
+      tcp_inc_counter (syn_sent, error0, 1);
+      if (PREDICT_FALSE ((b0->flags & VLIB_BUFFER_IS_TRACED) && tcp0 != 0))
+       {
+         t0 = vlib_add_trace (vm, node, b0, sizeof (*t0));
+         clib_memcpy_fast (&t0->tcp_header, tcp0, sizeof (t0->tcp_header));
+         clib_memcpy_fast (&t0->tcp_connection, tc0,
+                           sizeof (t0->tcp_connection));
+       }
     }
 
   errors = session_manager_flush_enqueue_events (TRANSPORT_PROTO_TCP,
                                                 my_thread_index);
-  tcp_inc_counter (syn_sent, TCP_ERROR_EVENT_FIFO_FULL, errors);
+  tcp_inc_counter (syn_sent, TCP_ERROR_MSG_QUEUE_FULL, errors);
+  vlib_buffer_free (vm, first_buffer, from_frame->n_vectors);
+
   return from_frame->n_vectors;
 }
 
@@ -2574,357 +2643,337 @@ always_inline uword
 tcp46_rcv_process_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
                          vlib_frame_t * from_frame, int is_ip4)
 {
-  u32 n_left_from, next_index, *from, *to_next, n_fins = 0;
-  u32 my_thread_index = vm->thread_index, errors = 0;
+  u32 thread_index = vm->thread_index, errors = 0, *first_buffer;
+  tcp_worker_ctx_t *wrk = tcp_get_worker (thread_index);
+  u32 n_left_from, *from, max_dequeue;
 
-  from = vlib_frame_vector_args (from_frame);
+  from = first_buffer = vlib_frame_vector_args (from_frame);
   n_left_from = from_frame->n_vectors;
-  next_index = node->cached_next_index;
 
   while (n_left_from > 0)
     {
-      u32 n_left_to_next;
+      u32 bi0, error0 = TCP_ERROR_NONE;
+      tcp_header_t *tcp0 = 0;
+      tcp_connection_t *tc0;
+      vlib_buffer_t *b0;
+      u8 is_fin0;
 
-      vlib_get_next_frame (vm, node, next_index, to_next, n_left_to_next);
+      bi0 = from[0];
+      from += 1;
+      n_left_from -= 1;
 
-      while (n_left_from > 0 && n_left_to_next > 0)
+      b0 = vlib_get_buffer (vm, bi0);
+      tc0 = tcp_connection_get (vnet_buffer (b0)->tcp.connection_index,
+                               thread_index);
+      if (PREDICT_FALSE (tc0 == 0))
        {
-         u32 bi0;
-         vlib_buffer_t *b0;
-         tcp_header_t *tcp0 = 0;
-         tcp_connection_t *tc0;
-         u32 next0 = tcp_next_drop (is_ip4), error0 = TCP_ERROR_NONE;
-         u8 is_fin0;
+         error0 = TCP_ERROR_INVALID_CONNECTION;
+         goto drop;
+       }
 
-         bi0 = from[0];
-         to_next[0] = bi0;
-         from += 1;
-         to_next += 1;
-         n_left_from -= 1;
-         n_left_to_next -= 1;
+      tcp0 = tcp_buffer_hdr (b0);
+      is_fin0 = tcp_is_fin (tcp0);
 
-         b0 = vlib_get_buffer (vm, bi0);
-         tc0 = tcp_connection_get (vnet_buffer (b0)->tcp.connection_index,
-                                   my_thread_index);
-         if (PREDICT_FALSE (tc0 == 0))
+      /* SYNs, FINs and data consume sequence numbers */
+      vnet_buffer (b0)->tcp.seq_end = vnet_buffer (b0)->tcp.seq_number
+       + tcp_is_syn (tcp0) + is_fin0 + vnet_buffer (b0)->tcp.data_len;
+
+      if (CLIB_DEBUG)
+       {
+         tcp_connection_t *tmp;
+         tmp = tcp_lookup_connection (tc0->c_fib_index, b0, thread_index,
+                                      is_ip4);
+         if (tmp->state != tc0->state)
            {
-             error0 = TCP_ERROR_INVALID_CONNECTION;
+             clib_warning ("state changed");
              goto drop;
            }
+       }
 
-         tcp0 = tcp_buffer_hdr (b0);
-         is_fin0 = tcp_is_fin (tcp0);
+      /*
+       * Special treatment for CLOSED
+       */
+      if (PREDICT_FALSE (tc0->state == TCP_STATE_CLOSED))
+       {
+         error0 = TCP_ERROR_CONNECTION_CLOSED;
+         goto drop;
+       }
 
-         /* SYNs, FINs and data consume sequence numbers */
-         vnet_buffer (b0)->tcp.seq_end = vnet_buffer (b0)->tcp.seq_number
-           + tcp_is_syn (tcp0) + is_fin0 + vnet_buffer (b0)->tcp.data_len;
+      /*
+       * For all other states (except LISTEN)
+       */
 
-         if (CLIB_DEBUG)
-           {
-             tcp_connection_t *tmp;
-             tmp = tcp_lookup_connection (tc0->c_fib_index, b0,
-                                          my_thread_index, is_ip4);
-             if (tmp->state != tc0->state)
-               {
-                 clib_warning ("state changed");
-                 goto drop;
-               }
-           }
+      /* 1-4: check SEQ, RST, SYN */
+      if (PREDICT_FALSE (tcp_segment_validate (wrk, tc0, b0, tcp0, &error0)))
+       goto drop;
 
+      /* 5: check the ACK field  */
+      switch (tc0->state)
+       {
+       case TCP_STATE_SYN_RCVD:
          /*
-          * Special treatment for CLOSED
+          * If the segment acknowledgment is not acceptable, form a
+          * reset segment,
+          *  <SEQ=SEG.ACK><CTL=RST>
+          * and send it.
           */
-         if (PREDICT_FALSE (tc0->state == TCP_STATE_CLOSED))
+         if (!tcp_rcv_ack_is_acceptable (tc0, b0))
            {
-             error0 = TCP_ERROR_CONNECTION_CLOSED;
+             tcp_connection_reset (tc0);
+             error0 = TCP_ERROR_ACK_INVALID;
              goto drop;
            }
 
-         /*
-          * For all other states (except LISTEN)
-          */
-
-         /* 1-4: check SEQ, RST, SYN */
-         if (PREDICT_FALSE (tcp_segment_validate (vm, tc0, b0, tcp0,
-                                                  &next0, &error0)))
+         /* Make sure the ack is exactly right */
+         if (tc0->rcv_nxt != vnet_buffer (b0)->tcp.seq_number || is_fin0
+             || vnet_buffer (b0)->tcp.data_len)
            {
-             tcp_maybe_inc_counter (rcv_process, error0, 1);
+             tcp_connection_reset (tc0);
+             error0 = TCP_ERROR_SEGMENT_INVALID;
              goto drop;
            }
 
-         /* 5: check the ACK field  */
-         switch (tc0->state)
-           {
-           case TCP_STATE_SYN_RCVD:
-             /*
-              * If the segment acknowledgment is not acceptable, form a
-              * reset segment,
-              *  <SEQ=SEG.ACK><CTL=RST>
-              * and send it.
-              */
-             if (!tcp_rcv_ack_is_acceptable (tc0, b0))
-               {
-                 TCP_DBG ("connection not accepted");
-                 tcp_send_reset_w_pkt (tc0, b0, is_ip4);
-                 error0 = TCP_ERROR_ACK_INVALID;
-                 goto drop;
-               }
+         /* Update rtt and rto */
+         tcp_estimate_initial_rtt (tc0);
 
-             /* Update rtt and rto */
-             tcp_update_rtt (tc0, vnet_buffer (b0)->tcp.ack_number);
+         /* Switch state to ESTABLISHED */
+         tc0->state = TCP_STATE_ESTABLISHED;
+         TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
 
-             /* Switch state to ESTABLISHED */
-             tc0->state = TCP_STATE_ESTABLISHED;
-             TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
+         /* Initialize session variables */
+         tc0->snd_una = vnet_buffer (b0)->tcp.ack_number;
+         tc0->snd_wnd = clib_net_to_host_u16 (tcp0->window)
+           << tc0->rcv_opts.wscale;
+         tc0->snd_wl1 = vnet_buffer (b0)->tcp.seq_number;
+         tc0->snd_wl2 = vnet_buffer (b0)->tcp.ack_number;
 
-             /* Initialize session variables */
-             tc0->snd_una = vnet_buffer (b0)->tcp.ack_number;
-             tc0->snd_wnd = clib_net_to_host_u16 (tcp0->window)
-               << tc0->rcv_opts.wscale;
-             tc0->snd_wl1 = vnet_buffer (b0)->tcp.seq_number;
-             tc0->snd_wl2 = vnet_buffer (b0)->tcp.ack_number;
-
-             /* Reset SYN-ACK retransmit and SYN_RCV establish timers */
-             tcp_retransmit_timer_reset (tc0);
-             tcp_timer_reset (tc0, TCP_TIMER_ESTABLISH);
-             stream_session_accept_notify (&tc0->connection);
-             error0 = TCP_ERROR_ACK_OK;
-             break;
-           case TCP_STATE_ESTABLISHED:
-             /* We can get packets in established state here because they
-              * were enqueued before state change */
-             if (tcp_rcv_ack (tc0, b0, tcp0, &next0, &error0))
-               {
-                 tcp_maybe_inc_counter (rcv_process, error0, 1);
-                 goto drop;
-               }
-
-             break;
-           case TCP_STATE_FIN_WAIT_1:
-             /* In addition to the processing for the ESTABLISHED state, if
-              * our FIN is now acknowledged then enter FIN-WAIT-2 and
-              * continue processing in that state. */
-             if (tcp_rcv_ack (tc0, b0, tcp0, &next0, &error0))
-               {
-                 tcp_maybe_inc_counter (rcv_process, error0, 1);
-                 goto drop;
-               }
-
-             /* Still have to send the FIN */
-             if (tc0->flags & TCP_CONN_FINPNDG)
-               {
-                 /* TX fifo finally drained */
-                 if (!session_tx_fifo_max_dequeue (&tc0->connection))
-                   tcp_send_fin (tc0);
-               }
-             /* If FIN is ACKed */
-             else if (tc0->snd_una == tc0->snd_una_max)
-               {
-                 tc0->state = TCP_STATE_FIN_WAIT_2;
-                 TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
+         /* Reset SYN-ACK retransmit and SYN_RCV establish timers */
+         tcp_retransmit_timer_reset (tc0);
+         tcp_timer_reset (tc0, TCP_TIMER_ESTABLISH);
+         if (stream_session_accept_notify (&tc0->connection))
+           {
+             error0 = TCP_ERROR_MSG_QUEUE_FULL;
+             tcp_connection_reset (tc0);
+             goto drop;
+           }
+         error0 = TCP_ERROR_ACK_OK;
+         break;
+       case TCP_STATE_ESTABLISHED:
+         /* We can get packets in established state here because they
+          * were enqueued before state change */
+         if (tcp_rcv_ack (wrk, tc0, b0, tcp0, &error0))
+           goto drop;
 
-                 /* Stop all retransmit timers because we have nothing more
-                  * to send. Enable waitclose though because we're willing to
-                  * wait for peer's FIN but not indefinitely. */
-                 tcp_connection_timers_reset (tc0);
-                 tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_2MSL_TIME);
-               }
-             break;
-           case TCP_STATE_FIN_WAIT_2:
-             /* In addition to the processing for the ESTABLISHED state, if
-              * the retransmission queue is empty, the user's CLOSE can be
-              * acknowledged ("ok") but do not delete the TCB. */
-             if (tcp_rcv_ack (tc0, b0, tcp0, &next0, &error0))
-               {
-                 tcp_maybe_inc_counter (rcv_process, error0, 1);
-                 goto drop;
-               }
-             break;
-           case TCP_STATE_CLOSE_WAIT:
-             /* Do the same processing as for the ESTABLISHED state. */
-             if (tcp_rcv_ack (tc0, b0, tcp0, &next0, &error0))
-               {
-                 tcp_maybe_inc_counter (rcv_process, error0, 1);
-                 goto drop;
-               }
-             if (tc0->flags & TCP_CONN_FINPNDG)
-               {
-                 /* TX fifo finally drained */
-                 if (!session_tx_fifo_max_dequeue (&tc0->connection))
-                   {
-                     tcp_send_fin (tc0);
-                     tcp_connection_timers_reset (tc0);
-                     tc0->state = TCP_STATE_LAST_ACK;
-                     tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE,
-                                       TCP_2MSL_TIME);
-                   }
-               }
-             break;
-           case TCP_STATE_CLOSING:
-             /* In addition to the processing for the ESTABLISHED state, if
-              * the ACK acknowledges our FIN then enter the TIME-WAIT state,
-              * otherwise ignore the segment. */
-             if (tcp_rcv_ack (tc0, b0, tcp0, &next0, &error0))
-               {
-                 tcp_maybe_inc_counter (rcv_process, error0, 1);
-                 goto drop;
-               }
+         break;
+       case TCP_STATE_FIN_WAIT_1:
+         /* In addition to the processing for the ESTABLISHED state, if
+          * our FIN is now acknowledged then enter FIN-WAIT-2 and
+          * continue processing in that state. */
+         if (tcp_rcv_ack (wrk, tc0, b0, tcp0, &error0))
+           goto drop;
 
-             tc0->state = TCP_STATE_TIME_WAIT;
+         /* Still have to send the FIN */
+         if (tc0->flags & TCP_CONN_FINPNDG)
+           {
+             /* TX fifo finally drained */
+             max_dequeue = session_tx_fifo_max_dequeue (&tc0->connection);
+             if (max_dequeue <= tc0->burst_acked)
+               tcp_send_fin (tc0);
+           }
+         /* If FIN is ACKed */
+         else if (tc0->snd_una == tc0->snd_una_max)
+           {
+             tc0->state = TCP_STATE_FIN_WAIT_2;
              TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
-             tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_TIMEWAIT_TIME);
-             goto drop;
 
-             break;
-           case TCP_STATE_LAST_ACK:
-             /* The only thing that [should] arrive in this state is an
-              * acknowledgment of our FIN. If our FIN is now acknowledged,
-              * delete the TCB, enter the CLOSED state, and return. */
+             /* Stop all retransmit timers because we have nothing more
+              * to send. Enable waitclose though because we're willing to
+              * wait for peer's FIN but not indefinitely. */
+             tcp_connection_timers_reset (tc0);
+             tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_2MSL_TIME);
 
-             if (!tcp_rcv_ack_is_acceptable (tc0, b0))
-               {
-                 error0 = TCP_ERROR_ACK_INVALID;
-                 goto drop;
-               }
-             error0 = TCP_ERROR_ACK_OK;
-             tc0->snd_una = vnet_buffer (b0)->tcp.ack_number;
-             /* Apparently our ACK for the peer's FIN was lost */
-             if (is_fin0 && tc0->snd_una != tc0->snd_una_max)
+             /* Don't try to deq the FIN acked */
+             if (tc0->burst_acked > 1)
+               stream_session_dequeue_drop (&tc0->connection,
+                                            tc0->burst_acked - 1);
+             tc0->burst_acked = 0;
+           }
+         break;
+       case TCP_STATE_FIN_WAIT_2:
+         /* In addition to the processing for the ESTABLISHED state, if
+          * the retransmission queue is empty, the user's CLOSE can be
+          * acknowledged ("ok") but do not delete the TCB. */
+         if (tcp_rcv_ack (wrk, tc0, b0, tcp0, &error0))
+           goto drop;
+         tc0->burst_acked = 0;
+         break;
+       case TCP_STATE_CLOSE_WAIT:
+         /* Do the same processing as for the ESTABLISHED state. */
+         if (tcp_rcv_ack (wrk, tc0, b0, tcp0, &error0))
+           goto drop;
+
+         if (tc0->flags & TCP_CONN_FINPNDG)
+           {
+             /* TX fifo finally drained */
+             if (!session_tx_fifo_max_dequeue (&tc0->connection))
                {
                  tcp_send_fin (tc0);
-                 goto drop;
+                 tcp_connection_timers_reset (tc0);
+                 tc0->state = TCP_STATE_LAST_ACK;
+                 TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
+                 tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_2MSL_TIME);
                }
+           }
+         break;
+       case TCP_STATE_CLOSING:
+         /* In addition to the processing for the ESTABLISHED state, if
+          * the ACK acknowledges our FIN then enter the TIME-WAIT state,
+          * otherwise ignore the segment. */
+         if (tcp_rcv_ack (wrk, tc0, b0, tcp0, &error0))
+           goto drop;
 
-             tc0->state = TCP_STATE_CLOSED;
-             TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
+         tc0->state = TCP_STATE_TIME_WAIT;
+         TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
+         tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_TIMEWAIT_TIME);
+         goto drop;
 
-             /* Don't free the connection from the data path since
-              * we can't ensure that we have no packets already enqueued
-              * to output. Rely instead on the waitclose timer */
-             tcp_connection_timers_reset (tc0);
-             tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, 1);
+         break;
+       case TCP_STATE_LAST_ACK:
+         /* The only thing that [should] arrive in this state is an
+          * acknowledgment of our FIN. If our FIN is now acknowledged,
+          * delete the TCB, enter the CLOSED state, and return. */
 
+         if (!tcp_rcv_ack_is_acceptable (tc0, b0))
+           {
+             error0 = TCP_ERROR_ACK_INVALID;
              goto drop;
+           }
+         error0 = TCP_ERROR_ACK_OK;
+         tc0->snd_una = vnet_buffer (b0)->tcp.ack_number;
+         /* Apparently our ACK for the peer's FIN was lost */
+         if (is_fin0 && tc0->snd_una != tc0->snd_una_max)
+           {
+             tcp_send_fin (tc0);
+             goto drop;
+           }
 
-             break;
-           case TCP_STATE_TIME_WAIT:
-             /* The only thing that can arrive in this state is a
-              * retransmission of the remote FIN. Acknowledge it, and restart
-              * the 2 MSL timeout. */
+         tc0->state = TCP_STATE_CLOSED;
+         TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
 
-             if (tcp_rcv_ack (tc0, b0, tcp0, &next0, &error0))
-               {
-                 tcp_maybe_inc_counter (rcv_process, error0, 1);
-                 goto drop;
-               }
+         /* Don't free the connection from the data path since
+          * we can't ensure that we have no packets already enqueued
+          * to output. Rely instead on the waitclose timer */
+         tcp_connection_timers_reset (tc0);
+         tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_CLEANUP_TIME);
 
-             tcp_make_ack (tc0, b0);
-             next0 = tcp_next_output (is_ip4);
-             tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_TIMEWAIT_TIME);
-             goto drop;
+         goto drop;
 
-             break;
-           default:
-             ASSERT (0);
-           }
+         break;
+       case TCP_STATE_TIME_WAIT:
+         /* The only thing that can arrive in this state is a
+          * retransmission of the remote FIN. Acknowledge it, and restart
+          * the 2 MSL timeout. */
 
-         /* 6: check the URG bit TODO */
+         if (tcp_rcv_ack (wrk, tc0, b0, tcp0, &error0))
+           goto drop;
 
-         /* 7: process the segment text */
-         switch (tc0->state)
-           {
-           case TCP_STATE_ESTABLISHED:
-           case TCP_STATE_FIN_WAIT_1:
-           case TCP_STATE_FIN_WAIT_2:
-             if (vnet_buffer (b0)->tcp.data_len)
-               {
-                 error0 = tcp_segment_rcv (tc0, b0, &next0);
-                 tcp_maybe_inc_counter (rcv_process, error0, 1);
-               }
-             else if (is_fin0)
-               tc0->rcv_nxt += 1;
-             break;
-           case TCP_STATE_CLOSE_WAIT:
-           case TCP_STATE_CLOSING:
-           case TCP_STATE_LAST_ACK:
-           case TCP_STATE_TIME_WAIT:
-             /* This should not occur, since a FIN has been received from the
-              * remote side.  Ignore the segment text. */
-             break;
-           }
+         tcp_program_ack (wrk, tc0);
+         tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_TIMEWAIT_TIME);
+         goto drop;
 
-         /* 8: check the FIN bit */
-         if (!is_fin0)
-           goto drop;
+         break;
+       default:
+         ASSERT (0);
+       }
 
-         switch (tc0->state)
-           {
-           case TCP_STATE_ESTABLISHED:
-           case TCP_STATE_SYN_RCVD:
-             /* Send FIN-ACK notify app and enter CLOSE-WAIT */
-             tcp_connection_timers_reset (tc0);
-             tcp_make_fin (tc0, b0);
-             tc0->snd_nxt += 1;
-             tc0->snd_una_max = tc0->snd_nxt;
-             tcp_retransmit_timer_set (tc0);
-             next0 = tcp_next_output (tc0->c_is_ip4);
-             stream_session_disconnect_notify (&tc0->connection);
-             tc0->state = TCP_STATE_CLOSE_WAIT;
-             TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
-             break;
-           case TCP_STATE_CLOSE_WAIT:
-           case TCP_STATE_CLOSING:
-           case TCP_STATE_LAST_ACK:
-             /* move along .. */
-             break;
-           case TCP_STATE_FIN_WAIT_1:
-             tc0->state = TCP_STATE_CLOSING;
-             tcp_make_ack (tc0, b0);
-             next0 = tcp_next_output (is_ip4);
-             TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
-             /* Wait for ACK but not forever */
-             tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_2MSL_TIME);
-             break;
-           case TCP_STATE_FIN_WAIT_2:
-             /* Got FIN, send ACK! Be more aggressive with resource cleanup */
-             tc0->state = TCP_STATE_TIME_WAIT;
-             tcp_connection_timers_reset (tc0);
-             tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_TIMEWAIT_TIME);
-             tcp_make_ack (tc0, b0);
-             next0 = tcp_next_output (is_ip4);
-             TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
-             break;
-           case TCP_STATE_TIME_WAIT:
-             /* Remain in the TIME-WAIT state. Restart the time-wait
-              * timeout.
-              */
-             tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_TIMEWAIT_TIME);
-             break;
-           }
-         TCP_EVT_DBG (TCP_EVT_FIN_RCVD, tc0);
-         n_fins += 1;
+      /* 6: check the URG bit TODO */
 
-       drop:
-         b0->error = error0 ? node->errors[error0] : 0;
+      /* 7: process the segment text */
+      switch (tc0->state)
+       {
+       case TCP_STATE_ESTABLISHED:
+       case TCP_STATE_FIN_WAIT_1:
+       case TCP_STATE_FIN_WAIT_2:
+         if (vnet_buffer (b0)->tcp.data_len)
+           error0 = tcp_segment_rcv (wrk, tc0, b0);
+         else if (is_fin0)
+           tc0->rcv_nxt += 1;
+         break;
+       case TCP_STATE_CLOSE_WAIT:
+       case TCP_STATE_CLOSING:
+       case TCP_STATE_LAST_ACK:
+       case TCP_STATE_TIME_WAIT:
+         /* This should not occur, since a FIN has been received from the
+          * remote side.  Ignore the segment text. */
+         break;
+       }
 
-         if (PREDICT_FALSE (b0->flags & VLIB_BUFFER_IS_TRACED))
-           {
-             tcp_rx_trace_t *t0 =
-               vlib_add_trace (vm, node, b0, sizeof (*t0));
-             tcp_set_rx_trace_data (t0, tc0, tcp0, b0, is_ip4);
-           }
+      /* 8: check the FIN bit */
+      if (!is_fin0)
+       goto drop;
 
-         vlib_validate_buffer_enqueue_x1 (vm, node, next_index, to_next,
-                                          n_left_to_next, bi0, next0);
+      switch (tc0->state)
+       {
+       case TCP_STATE_ESTABLISHED:
+         tcp_rcv_fin (wrk, tc0, b0, &error0);
+         break;
+       case TCP_STATE_SYN_RCVD:
+         /* Send FIN-ACK, enter LAST-ACK and because the app was not
+          * notified yet, set a cleanup timer instead of relying on
+          * disconnect notify and the implicit close call. */
+         tcp_connection_timers_reset (tc0);
+         tc0->rcv_nxt += 1;
+         tcp_send_fin (tc0);
+         tc0->state = TCP_STATE_LAST_ACK;
+         TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
+         tcp_timer_set (tc0, TCP_TIMER_WAITCLOSE, TCP_2MSL_TIME);
+         break;
+       case TCP_STATE_CLOSE_WAIT:
+       case TCP_STATE_CLOSING:
+       case TCP_STATE_LAST_ACK:
+         /* move along .. */
+         break;
+       case TCP_STATE_FIN_WAIT_1:
+         tc0->state = TCP_STATE_CLOSING;
+         tcp_program_ack (wrk, tc0);
+         TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
+         /* Wait for ACK but not forever */
+         tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_2MSL_TIME);
+         break;
+       case TCP_STATE_FIN_WAIT_2:
+         /* Got FIN, send ACK! Be more aggressive with resource cleanup */
+         tc0->state = TCP_STATE_TIME_WAIT;
+         tcp_connection_timers_reset (tc0);
+         tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_TIMEWAIT_TIME);
+         tcp_program_ack (wrk, tc0);
+         TCP_EVT_DBG (TCP_EVT_STATE_CHANGE, tc0);
+         break;
+       case TCP_STATE_TIME_WAIT:
+         /* Remain in the TIME-WAIT state. Restart the time-wait
+          * timeout.
+          */
+         tcp_timer_update (tc0, TCP_TIMER_WAITCLOSE, TCP_TIMEWAIT_TIME);
+         break;
        }
+      error0 = TCP_ERROR_FIN_RCVD;
+      TCP_EVT_DBG (TCP_EVT_FIN_RCVD, tc0);
+
+    drop:
 
-      vlib_put_next_frame (vm, node, next_index, n_left_to_next);
+      tcp_inc_counter (rcv_process, error0, 1);
+      if (PREDICT_FALSE (b0->flags & VLIB_BUFFER_IS_TRACED))
+       {
+         tcp_rx_trace_t *t0 = vlib_add_trace (vm, node, b0, sizeof (*t0));
+         tcp_set_rx_trace_data (t0, tc0, tcp0, b0, is_ip4);
+       }
     }
 
   errors = session_manager_flush_enqueue_events (TRANSPORT_PROTO_TCP,
-                                                my_thread_index);
-  tcp_inc_counter (rcv_process, TCP_ERROR_EVENT_FIFO_FULL, errors);
-  tcp_inc_counter (rcv_process, TCP_ERROR_FIN_RCVD, n_fins);
+                                                thread_index);
+  tcp_inc_counter (rcv_process, TCP_ERROR_MSG_QUEUE_FULL, errors);
+  tcp_handle_postponed_dequeues (wrk);
+  vlib_buffer_free (vm, first_buffer, from_frame->n_vectors);
+
   return from_frame->n_vectors;
 }
 
@@ -2996,162 +3045,148 @@ always_inline uword
 tcp46_listen_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
                     vlib_frame_t * from_frame, int is_ip4)
 {
-  u32 n_left_from, next_index, *from, *to_next, n_syns = 0;
+  u32 n_left_from, *from, n_syns = 0, *first_buffer;
   u32 my_thread_index = vm->thread_index;
 
-  from = vlib_frame_vector_args (from_frame);
+  from = first_buffer = vlib_frame_vector_args (from_frame);
   n_left_from = from_frame->n_vectors;
 
-  next_index = node->cached_next_index;
-
   while (n_left_from > 0)
     {
-      u32 n_left_to_next;
-
-      vlib_get_next_frame (vm, node, next_index, to_next, n_left_to_next);
-
-      while (n_left_from > 0 && n_left_to_next > 0)
-       {
-         u32 bi0;
-         vlib_buffer_t *b0;
-         tcp_rx_trace_t *t0;
-         tcp_header_t *th0 = 0;
-         tcp_connection_t *lc0;
-         ip4_header_t *ip40;
-         ip6_header_t *ip60;
-         tcp_connection_t *child0;
-         u32 error0 = TCP_ERROR_NONE, next0 = tcp_next_drop (is_ip4);
-
-         bi0 = from[0];
-         to_next[0] = bi0;
-         from += 1;
-         to_next += 1;
-         n_left_from -= 1;
-         n_left_to_next -= 1;
+      u32 bi0;
+      vlib_buffer_t *b0;
+      tcp_rx_trace_t *t0;
+      tcp_header_t *th0 = 0;
+      tcp_connection_t *lc0;
+      ip4_header_t *ip40;
+      ip6_header_t *ip60;
+      tcp_connection_t *child0;
+      u32 error0 = TCP_ERROR_NONE;
 
-         b0 = vlib_get_buffer (vm, bi0);
-         lc0 = tcp_listener_get (vnet_buffer (b0)->tcp.connection_index);
+      bi0 = from[0];
+      from += 1;
+      n_left_from -= 1;
 
-         if (is_ip4)
-           {
-             ip40 = vlib_buffer_get_current (b0);
-             th0 = ip4_next_header (ip40);
-           }
-         else
-           {
-             ip60 = vlib_buffer_get_current (b0);
-             th0 = ip6_next_header (ip60);
-           }
+      b0 = vlib_get_buffer (vm, bi0);
+      lc0 = tcp_listener_get (vnet_buffer (b0)->tcp.connection_index);
 
-         /* Create child session. For syn-flood protection use filter */
+      if (is_ip4)
+       {
+         ip40 = vlib_buffer_get_current (b0);
+         th0 = ip4_next_header (ip40);
+       }
+      else
+       {
+         ip60 = vlib_buffer_get_current (b0);
+         th0 = ip6_next_header (ip60);
+       }
 
-         /* 1. first check for an RST: handled in dispatch */
-         /* if (tcp_rst (th0))
-            goto drop; */
+      /* Create child session. For syn-flood protection use filter */
 
-         /* 2. second check for an ACK: handled in dispatch */
-         /* if (tcp_ack (th0))
-            {
-            tcp_send_reset (b0, is_ip4);
-            goto drop;
-            } */
+      /* 1. first check for an RST: handled in dispatch */
+      /* if (tcp_rst (th0))
+         goto drop;
+       */
 
-         /* 3. check for a SYN (did that already) */
+      /* 2. second check for an ACK: handled in dispatch */
+      /* if (tcp_ack (th0))
+         {
+         tcp_send_reset (b0, is_ip4);
+         goto drop;
+         }
+       */
 
-         /* Make sure connection wasn't just created */
-         child0 = tcp_lookup_connection (lc0->c_fib_index, b0,
-                                         my_thread_index, is_ip4);
-         if (PREDICT_FALSE (child0->state != TCP_STATE_LISTEN))
-           {
-             error0 = TCP_ERROR_CREATE_EXISTS;
-             goto drop;
-           }
+      /* 3. check for a SYN (did that already) */
 
-         /* Create child session and send SYN-ACK */
-         child0 = tcp_connection_new (my_thread_index);
-         child0->c_lcl_port = th0->dst_port;
-         child0->c_rmt_port = th0->src_port;
-         child0->c_is_ip4 = is_ip4;
-         child0->state = TCP_STATE_SYN_RCVD;
-         child0->c_fib_index = lc0->c_fib_index;
+      /* Make sure connection wasn't just created */
+      child0 = tcp_lookup_connection (lc0->c_fib_index, b0, my_thread_index,
+                                     is_ip4);
+      if (PREDICT_FALSE (child0->state != TCP_STATE_LISTEN))
+       {
+         error0 = TCP_ERROR_CREATE_EXISTS;
+         goto drop;
+       }
 
-         if (is_ip4)
-           {
-             child0->c_lcl_ip4.as_u32 = ip40->dst_address.as_u32;
-             child0->c_rmt_ip4.as_u32 = ip40->src_address.as_u32;
-           }
-         else
-           {
-             clib_memcpy (&child0->c_lcl_ip6, &ip60->dst_address,
-                          sizeof (ip6_address_t));
-             clib_memcpy (&child0->c_rmt_ip6, &ip60->src_address,
-                          sizeof (ip6_address_t));
-           }
+      /* Create child session and send SYN-ACK */
+      child0 = tcp_connection_alloc (my_thread_index);
+      child0->c_lcl_port = th0->dst_port;
+      child0->c_rmt_port = th0->src_port;
+      child0->c_is_ip4 = is_ip4;
+      child0->state = TCP_STATE_SYN_RCVD;
+      child0->c_fib_index = lc0->c_fib_index;
 
-         if (tcp_options_parse (th0, &child0->rcv_opts))
-           {
-             clib_warning ("options parse fail");
-             goto drop;
-           }
+      if (is_ip4)
+       {
+         child0->c_lcl_ip4.as_u32 = ip40->dst_address.as_u32;
+         child0->c_rmt_ip4.as_u32 = ip40->src_address.as_u32;
+       }
+      else
+       {
+         clib_memcpy_fast (&child0->c_lcl_ip6, &ip60->dst_address,
+                           sizeof (ip6_address_t));
+         clib_memcpy_fast (&child0->c_rmt_ip6, &ip60->src_address,
+                           sizeof (ip6_address_t));
+       }
 
-         child0->irs = vnet_buffer (b0)->tcp.seq_number;
-         child0->rcv_nxt = vnet_buffer (b0)->tcp.seq_number + 1;
-         child0->rcv_las = child0->rcv_nxt;
-         child0->sw_if_index = vnet_buffer (b0)->sw_if_index[VLIB_RX];
+      if (tcp_options_parse (th0, &child0->rcv_opts, 1))
+       {
+         error0 = TCP_ERROR_OPTIONS;
+         tcp_connection_free (child0);
+         goto drop;
+       }
 
-         /* RFC1323: TSval timestamps sent on {SYN} and {SYN,ACK}
-          * segments are used to initialize PAWS. */
-         if (tcp_opts_tstamp (&child0->rcv_opts))
-           {
-             child0->tsval_recent = child0->rcv_opts.tsval;
-             child0->tsval_recent_age = tcp_time_now ();
-           }
+      child0->irs = vnet_buffer (b0)->tcp.seq_number;
+      child0->rcv_nxt = vnet_buffer (b0)->tcp.seq_number + 1;
+      child0->rcv_las = child0->rcv_nxt;
+      child0->sw_if_index = vnet_buffer (b0)->sw_if_index[VLIB_RX];
 
-         if (tcp_opts_wscale (&child0->rcv_opts))
-           child0->snd_wscale = child0->rcv_opts.wscale;
+      /* RFC1323: TSval timestamps sent on {SYN} and {SYN,ACK}
+       * segments are used to initialize PAWS. */
+      if (tcp_opts_tstamp (&child0->rcv_opts))
+       {
+         child0->tsval_recent = child0->rcv_opts.tsval;
+         child0->tsval_recent_age = tcp_time_now ();
+       }
 
-         child0->snd_wnd = clib_net_to_host_u16 (th0->window)
-           << child0->snd_wscale;
-         child0->snd_wl1 = vnet_buffer (b0)->tcp.seq_number;
-         child0->snd_wl2 = vnet_buffer (b0)->tcp.ack_number;
+      if (tcp_opts_wscale (&child0->rcv_opts))
+       child0->snd_wscale = child0->rcv_opts.wscale;
 
-         tcp_connection_init_vars (child0);
-         TCP_EVT_DBG (TCP_EVT_SYN_RCVD, child0, 1);
+      child0->snd_wnd = clib_net_to_host_u16 (th0->window)
+       << child0->snd_wscale;
+      child0->snd_wl1 = vnet_buffer (b0)->tcp.seq_number;
+      child0->snd_wl2 = vnet_buffer (b0)->tcp.ack_number;
 
-         if (stream_session_accept (&child0->connection, lc0->c_s_index,
-                                    0 /* notify */ ))
-           {
-             clib_warning ("session accept fail");
-             tcp_connection_cleanup (child0);
-             error0 = TCP_ERROR_CREATE_SESSION_FAIL;
-             goto drop;
-           }
+      tcp_connection_init_vars (child0);
+      TCP_EVT_DBG (TCP_EVT_SYN_RCVD, child0, 1);
 
-         /* Reuse buffer to make syn-ack and send */
-         tcp_make_synack (child0, b0);
-         next0 = tcp_next_output (is_ip4);
-         tcp_timer_set (child0, TCP_TIMER_ESTABLISH, TCP_SYN_RCVD_TIME);
+      if (stream_session_accept (&child0->connection, lc0->c_s_index,
+                                0 /* notify */ ))
+       {
+         tcp_connection_cleanup (child0);
+         error0 = TCP_ERROR_CREATE_SESSION_FAIL;
+         goto drop;
+       }
 
-       drop:
-         if (PREDICT_FALSE (b0->flags & VLIB_BUFFER_IS_TRACED))
-           {
-             t0 = vlib_add_trace (vm, node, b0, sizeof (*t0));
-             clib_memcpy (&t0->tcp_header, th0, sizeof (t0->tcp_header));
-             clib_memcpy (&t0->tcp_connection, lc0,
-                          sizeof (t0->tcp_connection));
-           }
+      child0->tx_fifo_size = transport_tx_fifo_size (&child0->connection);
+      tcp_send_synack (child0);
+      tcp_timer_set (child0, TCP_TIMER_ESTABLISH, TCP_SYN_RCVD_TIME);
 
-         n_syns += (error0 == TCP_ERROR_NONE);
-         b0->error = node->errors[error0];
+    drop:
 
-         vlib_validate_buffer_enqueue_x1 (vm, node, next_index, to_next,
-                                          n_left_to_next, bi0, next0);
+      if (PREDICT_FALSE (b0->flags & VLIB_BUFFER_IS_TRACED))
+       {
+         t0 = vlib_add_trace (vm, node, b0, sizeof (*t0));
+         clib_memcpy_fast (&t0->tcp_header, th0, sizeof (t0->tcp_header));
+         clib_memcpy_fast (&t0->tcp_connection, lc0,
+                           sizeof (t0->tcp_connection));
        }
 
-      vlib_put_next_frame (vm, node, next_index, n_left_to_next);
+      n_syns += (error0 == TCP_ERROR_NONE);
     }
 
   tcp_inc_counter (listen, TCP_ERROR_SYNS_RCVD, n_syns);
+  vlib_buffer_free (vm, first_buffer, from_frame->n_vectors);
+
   return from_frame->n_vectors;
 }
 
@@ -3302,13 +3337,19 @@ tcp_input_lookup_buffer (vlib_buffer_t * b, u8 thread_index, u32 * error,
   if (is_ip4)
     {
       ip4_header_t *ip4 = vlib_buffer_get_current (b);
+      int ip_hdr_bytes = ip4_header_bytes (ip4);
+      if (PREDICT_FALSE (b->current_length < ip_hdr_bytes + sizeof (*tcp)))
+       {
+         *error = TCP_ERROR_LENGTH;
+         return 0;
+       }
       tcp = ip4_next_header (ip4);
       vnet_buffer (b)->tcp.hdr_offset = (u8 *) tcp - (u8 *) ip4;
-      n_advance_bytes = (ip4_header_bytes (ip4) + tcp_header_bytes (tcp));
+      n_advance_bytes = (ip_hdr_bytes + tcp_header_bytes (tcp));
       n_data_bytes = clib_net_to_host_u16 (ip4->length) - n_advance_bytes;
 
       /* Length check. Checksum computed by ipx_local no need to compute again */
-      if (PREDICT_FALSE (n_advance_bytes < 0))
+      if (PREDICT_FALSE (n_data_bytes < 0))
        {
          *error = TCP_ERROR_LENGTH;
          return 0;
@@ -3322,6 +3363,11 @@ tcp_input_lookup_buffer (vlib_buffer_t * b, u8 thread_index, u32 * error,
   else
     {
       ip6_header_t *ip6 = vlib_buffer_get_current (b);
+      if (PREDICT_FALSE (b->current_length < sizeof (*ip6) + sizeof (*tcp)))
+       {
+         *error = TCP_ERROR_LENGTH;
+         return 0;
+       }
       tcp = ip6_next_header (ip6);
       vnet_buffer (b)->tcp.hdr_offset = (u8 *) tcp - (u8 *) ip6;
       n_advance_bytes = tcp_header_bytes (tcp);
@@ -3329,7 +3375,7 @@ tcp_input_lookup_buffer (vlib_buffer_t * b, u8 thread_index, u32 * error,
        - n_advance_bytes;
       n_advance_bytes += sizeof (ip6[0]);
 
-      if (PREDICT_FALSE (n_advance_bytes < 0))
+      if (PREDICT_FALSE (n_data_bytes < 0))
        {
          *error = TCP_ERROR_LENGTH;
          return 0;
@@ -3372,8 +3418,9 @@ tcp_input_dispatch_buffer (tcp_main_t * tm, tcp_connection_t * tc,
       vnet_buffer (b)->tcp.flags = tc->state;
 
       if (*error == TCP_ERROR_DISPATCH)
-       clib_warning ("disp error state %U flags %U", format_tcp_state,
-                     state, format_tcp_flags, (int) flags);
+       clib_warning ("tcp conn %u disp error state %U flags %U",
+                     tc->c_c_index, format_tcp_state, state,
+                     format_tcp_flags, (int) flags);
     }
 }
 
@@ -3386,7 +3433,7 @@ tcp46_input_inline (vlib_main_t * vm, vlib_node_runtime_t * node,
   vlib_buffer_t *bufs[VLIB_FRAME_SIZE], **b;
   u16 nexts[VLIB_FRAME_SIZE], *next;
 
-  tcp_set_time_now (thread_index);
+  tcp_set_time_now (tcp_get_worker (thread_index));
 
   from = vlib_frame_vector_args (frame);
   n_left_from = frame->n_vectors;
@@ -3560,20 +3607,62 @@ do {                                                            \
     tm->dispatch_table[TCP_STATE_##t][f].error = (e);          \
 } while (0)
 
-  /* SYNs for new connections -> tcp-listen. */
+  /* RFC 793: In LISTEN if RST drop and if ACK return RST */
+  _(LISTEN, 0, TCP_INPUT_NEXT_DROP, TCP_ERROR_SEGMENT_INVALID);
+  _(LISTEN, TCP_FLAG_ACK, TCP_INPUT_NEXT_RESET, TCP_ERROR_ACK_INVALID);
+  _(LISTEN, TCP_FLAG_RST, TCP_INPUT_NEXT_DROP, TCP_ERROR_INVALID_CONNECTION);
   _(LISTEN, TCP_FLAG_SYN, TCP_INPUT_NEXT_LISTEN, TCP_ERROR_NONE);
-  _(LISTEN, TCP_FLAG_ACK, TCP_INPUT_NEXT_RESET, TCP_ERROR_NONE);
-  _(LISTEN, TCP_FLAG_RST, TCP_INPUT_NEXT_DROP, TCP_ERROR_RST_RCVD);
+  _(LISTEN, TCP_FLAG_SYN | TCP_FLAG_ACK, TCP_INPUT_NEXT_RESET,
+    TCP_ERROR_ACK_INVALID);
+  _(LISTEN, TCP_FLAG_SYN | TCP_FLAG_RST, TCP_INPUT_NEXT_DROP,
+    TCP_ERROR_SEGMENT_INVALID);
+  _(LISTEN, TCP_FLAG_SYN | TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_DROP,
+    TCP_ERROR_SEGMENT_INVALID);
+  _(LISTEN, TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_DROP,
+    TCP_ERROR_INVALID_CONNECTION);
+  _(LISTEN, TCP_FLAG_FIN, TCP_INPUT_NEXT_RESET, TCP_ERROR_SEGMENT_INVALID);
   _(LISTEN, TCP_FLAG_FIN | TCP_FLAG_ACK, TCP_INPUT_NEXT_RESET,
+    TCP_ERROR_SEGMENT_INVALID);
+  _(LISTEN, TCP_FLAG_FIN | TCP_FLAG_RST, TCP_INPUT_NEXT_DROP,
+    TCP_ERROR_SEGMENT_INVALID);
+  _(LISTEN, TCP_FLAG_FIN | TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_DROP,
     TCP_ERROR_NONE);
+  _(LISTEN, TCP_FLAG_FIN | TCP_FLAG_SYN, TCP_INPUT_NEXT_DROP,
+    TCP_ERROR_SEGMENT_INVALID);
+  _(LISTEN, TCP_FLAG_FIN | TCP_FLAG_SYN | TCP_FLAG_ACK, TCP_INPUT_NEXT_DROP,
+    TCP_ERROR_SEGMENT_INVALID);
+  _(LISTEN, TCP_FLAG_FIN | TCP_FLAG_SYN | TCP_FLAG_RST, TCP_INPUT_NEXT_DROP,
+    TCP_ERROR_SEGMENT_INVALID);
+  _(LISTEN, TCP_FLAG_FIN | TCP_FLAG_SYN | TCP_FLAG_RST | TCP_FLAG_ACK,
+    TCP_INPUT_NEXT_DROP, TCP_ERROR_SEGMENT_INVALID);
   /* ACK for for a SYN-ACK -> tcp-rcv-process. */
   _(SYN_RCVD, TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(SYN_RCVD, TCP_FLAG_RST, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(SYN_RCVD, TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
     TCP_ERROR_NONE);
   _(SYN_RCVD, TCP_FLAG_SYN, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_SYN | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
+    TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_SYN | TCP_FLAG_RST, TCP_INPUT_NEXT_RCV_PROCESS,
+    TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_SYN | TCP_FLAG_RST | TCP_FLAG_ACK,
+    TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_FIN, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(SYN_RCVD, TCP_FLAG_FIN | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
     TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_FIN | TCP_FLAG_RST, TCP_INPUT_NEXT_RCV_PROCESS,
+    TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_FIN | TCP_FLAG_RST | TCP_FLAG_ACK,
+    TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_FIN | TCP_FLAG_SYN, TCP_INPUT_NEXT_RCV_PROCESS,
+    TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_FIN | TCP_FLAG_SYN | TCP_FLAG_RST,
+    TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_FIN | TCP_FLAG_SYN | TCP_FLAG_ACK,
+    TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(SYN_RCVD, TCP_FLAG_FIN | TCP_FLAG_SYN | TCP_FLAG_RST | TCP_FLAG_ACK,
+    TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(SYN_RCVD, 0, TCP_INPUT_NEXT_DROP, TCP_ERROR_SEGMENT_INVALID);
   /* SYN-ACK for a SYN */
   _(SYN_SENT, TCP_FLAG_SYN | TCP_FLAG_ACK, TCP_INPUT_NEXT_SYN_SENT,
     TCP_ERROR_NONE);
@@ -3587,43 +3676,79 @@ do {                                                            \
   _(ESTABLISHED, TCP_FLAG_FIN, TCP_INPUT_NEXT_ESTABLISHED, TCP_ERROR_NONE);
   _(ESTABLISHED, TCP_FLAG_FIN | TCP_FLAG_ACK, TCP_INPUT_NEXT_ESTABLISHED,
     TCP_ERROR_NONE);
+  _(ESTABLISHED, TCP_FLAG_FIN | TCP_FLAG_RST, TCP_INPUT_NEXT_ESTABLISHED,
+    TCP_ERROR_NONE);
+  _(ESTABLISHED, TCP_FLAG_FIN | TCP_FLAG_RST | TCP_FLAG_ACK,
+    TCP_INPUT_NEXT_ESTABLISHED, TCP_ERROR_NONE);
+  _(ESTABLISHED, TCP_FLAG_FIN | TCP_FLAG_SYN, TCP_INPUT_NEXT_ESTABLISHED,
+    TCP_ERROR_NONE);
+  _(ESTABLISHED, TCP_FLAG_FIN | TCP_FLAG_SYN | TCP_FLAG_ACK,
+    TCP_INPUT_NEXT_ESTABLISHED, TCP_ERROR_NONE);
+  _(ESTABLISHED, TCP_FLAG_FIN | TCP_FLAG_SYN | TCP_FLAG_RST,
+    TCP_INPUT_NEXT_ESTABLISHED, TCP_ERROR_NONE);
+  _(ESTABLISHED, TCP_FLAG_FIN | TCP_FLAG_SYN | TCP_FLAG_RST | TCP_FLAG_ACK,
+    TCP_INPUT_NEXT_ESTABLISHED, TCP_ERROR_NONE);
   _(ESTABLISHED, TCP_FLAG_RST, TCP_INPUT_NEXT_ESTABLISHED, TCP_ERROR_NONE);
   _(ESTABLISHED, TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_ESTABLISHED,
     TCP_ERROR_NONE);
   _(ESTABLISHED, TCP_FLAG_SYN, TCP_INPUT_NEXT_ESTABLISHED, TCP_ERROR_NONE);
   _(ESTABLISHED, TCP_FLAG_SYN | TCP_FLAG_ACK, TCP_INPUT_NEXT_ESTABLISHED,
     TCP_ERROR_NONE);
+  _(ESTABLISHED, TCP_FLAG_SYN | TCP_FLAG_RST, TCP_INPUT_NEXT_ESTABLISHED,
+    TCP_ERROR_NONE);
+  _(ESTABLISHED, TCP_FLAG_SYN | TCP_FLAG_RST | TCP_FLAG_ACK,
+    TCP_INPUT_NEXT_ESTABLISHED, TCP_ERROR_NONE);
+  _(ESTABLISHED, 0, TCP_INPUT_NEXT_DROP, TCP_ERROR_SEGMENT_INVALID);
   /* ACK or FIN-ACK to our FIN */
   _(FIN_WAIT_1, TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(FIN_WAIT_1, TCP_FLAG_ACK | TCP_FLAG_FIN, TCP_INPUT_NEXT_RCV_PROCESS,
     TCP_ERROR_NONE);
   /* FIN in reply to our FIN from the other side */
   _(FIN_WAIT_1, TCP_FLAG_FIN, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(FIN_WAIT_1, TCP_FLAG_SYN, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(FIN_WAIT_1, TCP_FLAG_RST, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(FIN_WAIT_1, TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
+    TCP_ERROR_NONE);
   _(CLOSING, TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   /* FIN confirming that the peer (app) has closed */
   _(FIN_WAIT_2, TCP_FLAG_FIN, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(FIN_WAIT_2, TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(FIN_WAIT_2, TCP_FLAG_FIN | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
     TCP_ERROR_NONE);
+  _(FIN_WAIT_2, TCP_FLAG_RST, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(FIN_WAIT_2, TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
+    TCP_ERROR_NONE);
   _(CLOSE_WAIT, TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(CLOSE_WAIT, TCP_FLAG_FIN | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
     TCP_ERROR_NONE);
+  _(CLOSE_WAIT, TCP_FLAG_RST, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(CLOSE_WAIT, TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
+    TCP_ERROR_NONE);
   _(LAST_ACK, TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(LAST_ACK, TCP_FLAG_FIN, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(LAST_ACK, TCP_FLAG_FIN | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
     TCP_ERROR_NONE);
   _(LAST_ACK, TCP_FLAG_RST, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(LAST_ACK, TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
+    TCP_ERROR_NONE);
   _(LAST_ACK, TCP_FLAG_SYN, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(TIME_WAIT, TCP_FLAG_FIN, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
   _(TIME_WAIT, TCP_FLAG_FIN | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
     TCP_ERROR_NONE);
   _(TIME_WAIT, TCP_FLAG_RST, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
+  _(TIME_WAIT, TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS,
+    TCP_ERROR_NONE);
   _(TIME_WAIT, TCP_FLAG_ACK, TCP_INPUT_NEXT_RCV_PROCESS, TCP_ERROR_NONE);
-  _(CLOSED, TCP_FLAG_ACK, TCP_INPUT_NEXT_DROP, TCP_ERROR_CONNECTION_CLOSED);
+  /* RFC793 CLOSED: An incoming segment containing a RST is discarded. An
+   * incoming segment not containing a RST causes a RST to be sent in
+   * response.*/
   _(CLOSED, TCP_FLAG_RST, TCP_INPUT_NEXT_DROP, TCP_ERROR_CONNECTION_CLOSED);
-  _(CLOSED, TCP_FLAG_FIN | TCP_FLAG_ACK, TCP_INPUT_NEXT_DROP,
+  _(CLOSED, TCP_FLAG_RST | TCP_FLAG_ACK, TCP_INPUT_NEXT_DROP,
     TCP_ERROR_CONNECTION_CLOSED);
+  _(CLOSED, TCP_FLAG_ACK, TCP_INPUT_NEXT_RESET, TCP_ERROR_NONE);
+  _(CLOSED, TCP_FLAG_SYN, TCP_INPUT_NEXT_RESET, TCP_ERROR_NONE);
+  _(CLOSED, TCP_FLAG_FIN | TCP_FLAG_ACK, TCP_INPUT_NEXT_RESET,
+    TCP_ERROR_NONE);
 #undef _
 }