From: Vladimir Kropylev <vladimir.kropylev@enea.com>
Date: Mon, 17 Jun 2019 18:38:00 +0000 (+0300)
Subject: session: call session_dequeue_notify after svm_fifo_dequeue_drop
X-Git-Tag: v20.01-rc0~379
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F92%2F20192%2F4;p=vpp.git

session: call session_dequeue_notify after svm_fifo_dequeue_drop

Type: fix

dequeue_notify callback at app-layer needs to know available space in fifo,
so, session_dequeue_notify should be called after svm_fifo_dequeue_drop

Change-Id: I136675d29ec32bea9b33a05deb6710f72ce8d5b1
Signed-off-by: Vladimir Kropylev <vladimir.kropylev@enea.com>
---

diff --git a/src/vnet/session/session.c b/src/vnet/session/session.c
index 4b3ce016be5..8aa4af935e9 100644
--- a/src/vnet/session/session.c
+++ b/src/vnet/session/session.c
@@ -453,11 +453,14 @@ u32
 session_tx_fifo_dequeue_drop (transport_connection_t * tc, u32 max_bytes)
 {
   session_t *s = session_get (tc->s_index, tc->thread_index);
+  u32 rv;
+
+  rv = svm_fifo_dequeue_drop (s->tx_fifo, max_bytes);
 
   if (svm_fifo_needs_tx_ntf (s->tx_fifo, max_bytes))
     session_dequeue_notify (s);
 
-  return svm_fifo_dequeue_drop (s->tx_fifo, max_bytes);
+  return rv;
 }
 
 static inline int