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

session: move svm_fifo_clear_deq_ntf to before calling the app callback

Browse Source 
Type: fix

in case the app requests another dequeue notification, it should not be cleared

Change-Id: I7e94e8e30ef098f87f5e7f4880af276970ee5b37
Signed-off-by: Vladimir Kropylev <vladimir.kropylev@enea.com>
(cherry picked from commit 5c89fbf286)
This commit is contained in:
Vladimir Kropylev
2019-08-30 13:04:06 +03:00
committed by Andrew Yourtchenko
parent 61b1d46db0
commit b462d94038
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/vnet/session/session.c
View File

@ -604,6 +604,8 @@ session_dequeue_notify (session_t * s)
{ {
app_worker_t *app_wrk; app_worker_t *app_wrk;
svm_fifo_clear_deq_ntf (s->tx_fifo);
app_wrk = app_worker_get_if_valid (s->app_wrk_index); app_wrk = app_worker_get_if_valid (s->app_wrk_index);
if (PREDICT_FALSE (!app_wrk)) if (PREDICT_FALSE (!app_wrk))
return -1; return -1;
@ -616,8 +618,6 @@ session_dequeue_notify (session_t * s)
return session_notify_subscribers (app_wrk->app_index, s, return session_notify_subscribers (app_wrk->app_index, s,
s->tx_fifo, SESSION_IO_EVT_TX); s->tx_fifo, SESSION_IO_EVT_TX);
svm_fifo_clear_deq_ntf (s->tx_fifo);
return 0; return 0;
} }