ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

session: push tx notifications only if needed

Browse Source 
VPP-1340

Change-Id: Ia3f2fc054dbfb2ba912a768c70466e7042024e32
Signed-off-by: Florin Coras <fcoras@cisco.com>
This commit is contained in:
Florin Coras
2018-07-11 05:25:06 -07:00
committed by Dave Barach
parent bf2f3261b2
commit 46f001d2ac
2 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/svm/svm_fifo.h
View File

@ -113,6 +113,12 @@ svm_fifo_max_enqueue (svm_fifo_t * f)
return f->nitems - svm_fifo_max_dequeue (f);
}
static inline int
svm_fifo_has_event (svm_fifo_t * f)
{
return f->has_event;
}
static inline u8
svm_fifo_has_ooo_data (svm_fifo_t * f)
{

7
src/vnet/session/session_node.c
View File

@ -653,6 +653,7 @@ skip_dequeue:
{
stream_session_t *s; /* $$$ prefetch 1 ahead maybe */
session_fifo_event_t *e;
u32 to_dequeue;
e = &fifo_events[i];
switch (e->event_type)
@ -671,6 +672,7 @@ skip_dequeue:
clib_warning ("It's dead, Jim!");
continue;
}
to_dequeue = svm_fifo_max_dequeue (s->server_tx_fifo);
/* Spray packets in per session type frames, since they go to
* different nodes */
@ -678,7 +680,10 @@ skip_dequeue:
&n_tx_packets);
if (PREDICT_TRUE (rv == SESSION_TX_OK))
{
session_dequeue_notify (s);
/* Notify app there's tx space if not polling */
if (PREDICT_FALSE (to_dequeue == s->server_tx_fifo->nitems
&& !svm_fifo_has_event (s->server_tx_fifo)))
session_dequeue_notify (s);
}
else if (PREDICT_FALSE (rv == SESSION_TX_NO_BUFFERS))
{