summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFilip Tehlar <ftehlar@cisco.com>2023-10-25 14:59:09 +0200
committerFlorin Coras <florin.coras@gmail.com>2023-10-30 15:56:38 +0000
commitc78eeaba49cba55a5e4c18aad1bd41e4c9b3aa33 (patch)
treeda92187684182e0b134ddb855187237686928721
parentf4e3ee1dd1b6edaca193a5a51057cf19c653aeb2 (diff)
hsa: preallocate sessions on server side
Type: test Change-Id: Ib809f5a6c9e2f08d87f0070231df04d5f0040ca0 Signed-off-by: Filip Tehlar <ftehlar@cisco.com>
-rw-r--r--src/plugins/hs_apps/echo_server.c217
1 files changed, 110 insertions, 107 deletions
diff --git a/src/plugins/hs_apps/echo_server.c b/src/plugins/hs_apps/echo_server.c
index eeaf2d70088..f4f9acf1d0b 100644
--- a/src/plugins/hs_apps/echo_server.c
+++ b/src/plugins/hs_apps/echo_server.c
@@ -24,15 +24,26 @@ static void es_set_echo_rx_callbacks (u8 no_echo);
typedef struct
{
- /*
- * Server app parameters
- */
- svm_msg_q_t **vpp_queue;
- svm_queue_t *vl_input_queue; /**< Sever's event queue */
+ CLIB_CACHE_LINE_ALIGN_MARK (cacheline0);
+ u32 session_index;
+ u64 vpp_session_handle;
+ u32 vpp_session_index;
+ u32 rx_retries;
+ u8 byte_index;
+} es_session_t;
+
+typedef struct
+{
+ CLIB_CACHE_LINE_ALIGN_MARK (cacheline0);
+ es_session_t *sessions;
+ u8 *rx_buf; /**< Per-thread RX buffer */
+ svm_msg_q_t *vpp_event_queue;
+ u32 thread_index;
+} es_worker_t;
+typedef struct
+{
u32 app_index; /**< Server app index */
- u32 my_client_index; /**< API client handle */
- u32 node_index; /**< process node index for event scheduling */
/*
* Config params
@@ -51,11 +62,8 @@ typedef struct
/*
* Test state
*/
+ es_worker_t *wrk;
int (*rx_callback) (session_t *session);
- u64 **session_handles;
- u8 **rx_buf; /**< Per-thread RX buffer */
- u32 **rx_retries;
- u8 byte_index;
u8 transport_proto;
u64 listener_handle; /**< Session handle of the root listener */
u64 ctrl_listener_handle;
@@ -77,6 +85,28 @@ echo_server_main_t echo_server_main;
#define es_cli(_fmt, _args...) vlib_cli_output (vm, _fmt, ##_args)
+static inline es_worker_t *
+es_worker_get (u32 thread_index)
+{
+ return vec_elt_at_index (echo_server_main.wrk, thread_index);
+}
+
+static inline es_session_t *
+es_session_alloc (es_worker_t *wrk)
+{
+ es_session_t *es;
+
+ pool_get_zero (wrk->sessions, es);
+ es->session_index = es - wrk->sessions;
+ return es;
+}
+
+static inline es_session_t *
+es_session_get (es_worker_t *wrk, u32 es_index)
+{
+ return pool_elt_at_index (wrk->sessions, es_index);
+}
+
int
quic_echo_server_qsession_accept_callback (session_t * s)
{
@@ -91,6 +121,18 @@ echo_server_ctrl_session_accept_callback (session_t *s)
return 0;
}
+static void
+es_session_alloc_and_init (session_t *s)
+{
+ es_session_t *es;
+ es_worker_t *wrk = es_worker_get (s->thread_index);
+
+ es = es_session_alloc (wrk);
+ es->vpp_session_index = s->session_index;
+ es->vpp_session_handle = session_handle (s);
+ s->opaque = es->session_index;
+}
+
int
quic_echo_server_session_accept_callback (session_t * s)
{
@@ -104,12 +146,8 @@ quic_echo_server_session_accept_callback (session_t * s)
es_dbg ("SSESSION %u accept w/opaque %d", s->session_index, s->opaque);
- esm->vpp_queue[s->thread_index] =
- session_main_get_vpp_event_queue (s->thread_index);
s->session_state = SESSION_STATE_READY;
- ASSERT (vec_len (esm->rx_retries) > s->thread_index);
- vec_validate (esm->rx_retries[s->thread_index], s->session_index);
- esm->rx_retries[s->thread_index][s->session_index] = 0;
+ es_session_alloc_and_init (s);
return 0;
}
@@ -121,16 +159,8 @@ echo_server_session_accept_callback (session_t * s)
if (PREDICT_FALSE (esm->ctrl_listener_handle == s->listener_handle))
return echo_server_ctrl_session_accept_callback (s);
- esm->vpp_queue[s->thread_index] =
- session_main_get_vpp_event_queue (s->thread_index);
s->session_state = SESSION_STATE_READY;
- ASSERT (vec_len (esm->rx_retries) > s->thread_index);
- vec_validate (esm->rx_retries[s->thread_index], s->session_index);
- esm->rx_retries[s->thread_index][s->session_index] = 0;
- if (session_get_transport_proto (s) == TRANSPORT_PROTO_UDP)
- {
- vec_add1 (esm->session_handles[s->thread_index], session_handle (s));
- }
+ es_session_alloc_and_init (s);
return 0;
}
@@ -183,8 +213,7 @@ es_foreach_thread (void *fp)
{
echo_server_main_t *esm = &echo_server_main;
uword thread_index;
- for (thread_index = 0; thread_index < vec_len (esm->session_handles);
- thread_index++)
+ for (thread_index = 0; thread_index < vec_len (esm->wrk); thread_index++)
{
session_send_rpc_evt_to_thread (thread_index, fp,
uword_to_pointer (thread_index, void *));
@@ -195,32 +224,25 @@ static int
es_wrk_prealloc_sessions (void *args)
{
echo_server_main_t *esm = &echo_server_main;
- u32 thread_index = pointer_to_uword (args);
-
- vec_validate (esm->rx_retries[thread_index], esm->cfg.num_test_sessions);
+ u32 sessions_per_wrk, n_wrks, thread_index;
+ thread_index = pointer_to_uword (args);
+ es_worker_t *wrk = es_worker_get (thread_index);
+ n_wrks = vlib_num_workers () ? vlib_num_workers () : 1;
+ sessions_per_wrk = esm->cfg.num_test_sessions / n_wrks;
+ pool_alloc (wrk->sessions, 1.1 * sessions_per_wrk);
return 0;
}
static int
echo_server_setup_test (hs_test_cfg_t *c)
{
- echo_server_main_t *esm = &echo_server_main;
-
if (c->test == HS_TEST_TYPE_UNI)
es_set_echo_rx_callbacks (1 /* no echo */);
else
es_set_echo_rx_callbacks (0 /* no echo */);
es_foreach_thread (es_wrk_prealloc_sessions);
-
- if (c->test_bytes && c->num_test_sessions > 1)
- {
- es_err ("test bytes not supported for more sessions; turning it off");
- c->test_bytes = 0;
- }
- esm->byte_index = 0;
-
return 0;
}
@@ -250,23 +272,24 @@ es_test_cmd_sync (echo_server_main_t *esm, session_t *s)
}
static int
-es_wrk_cleanup_session (void *args)
+es_wrk_cleanup_sessions (void *args)
{
echo_server_main_t *esm = &echo_server_main;
- u32 thread_index = pointer_to_uword (args);
- session_handle_t *session_handles, *sh;
vnet_disconnect_args_t _a = {}, *a = &_a;
+ u32 thread_index = pointer_to_uword (args);
+ es_session_t *es;
+ es_worker_t *wrk;
+ wrk = es_worker_get (thread_index);
a->app_index = esm->app_index;
- session_handles = esm->session_handles[thread_index];
-
- vec_foreach (sh, session_handles)
+ pool_foreach (es, wrk->sessions)
{
- a->handle = sh[0];
+ a->handle = es->vpp_session_handle;
vnet_disconnect_session (a);
}
- vec_reset_length (session_handles);
+ pool_free (wrk->sessions);
+
return 0;
}
@@ -290,7 +313,7 @@ echo_server_rx_ctrl_callback (session_t *s)
{
case HS_TEST_TYPE_ECHO:
case HS_TEST_TYPE_NONE:
- es_foreach_thread (es_wrk_cleanup_session);
+ es_foreach_thread (es_wrk_cleanup_sessions);
echo_server_ctrl_reply (s);
break;
case HS_TEST_TYPE_UNI:
@@ -328,19 +351,18 @@ echo_server_builtin_server_rx_callback_no_echo (session_t * s)
}
static void
-es_test_bytes (echo_server_main_t *esm, int actual_transfer)
+es_test_bytes (es_worker_t *wrk, es_session_t *es, int actual_transfer)
{
int i;
- u32 my_thread_id = vlib_get_thread_index ();
for (i = 0; i < actual_transfer; i++)
{
- if (esm->rx_buf[my_thread_id][i] != ((esm->byte_index + i) & 0xff))
+ if (wrk->rx_buf[i] != ((es->byte_index + i) & 0xff))
{
- es_err ("at %lld expected %d got %d", esm->byte_index + i,
- (esm->byte_index + i) & 0xff, esm->rx_buf[my_thread_id][i]);
+ es_err ("at %lld expected %d got %d", es->byte_index + i,
+ (es->byte_index + i) & 0xff, wrk->rx_buf[i]);
}
}
- esm->byte_index += actual_transfer;
+ es->byte_index += actual_transfer;
}
int
@@ -352,6 +374,8 @@ echo_server_rx_callback (session_t * s)
echo_server_main_t *esm = &echo_server_main;
u32 thread_index = vlib_get_thread_index ();
app_session_transport_t at;
+ es_worker_t *wrk;
+ es_session_t *es;
ASSERT (s->thread_index == thread_index);
@@ -364,7 +388,9 @@ echo_server_rx_callback (session_t * s)
if (PREDICT_FALSE (esm->ctrl_listener_handle == s->listener_handle))
return echo_server_rx_ctrl_callback (s);
+ wrk = es_worker_get (thread_index);
max_enqueue = svm_fifo_max_enqueue_prod (tx_fifo);
+
if (!esm->is_dgram)
{
max_dequeue = svm_fifo_max_dequeue_cons (rx_fifo);
@@ -374,18 +400,15 @@ echo_server_rx_callback (session_t * s)
session_dgram_pre_hdr_t ph;
svm_fifo_peek (rx_fifo, 0, sizeof (ph), (u8 *) & ph);
max_dequeue = ph.data_length - ph.data_offset;
- if (!esm->vpp_queue[s->thread_index])
- {
- svm_msg_q_t *mq;
- mq = session_main_get_vpp_event_queue (s->thread_index);
- esm->vpp_queue[s->thread_index] = mq;
- }
+ ASSERT (wrk->vpp_event_queue);
max_enqueue -= sizeof (session_dgram_hdr_t);
}
if (PREDICT_FALSE (max_dequeue == 0))
return 0;
+ es = es_session_get (wrk, s->opaque);
+
/* Number of bytes we're going to copy */
max_transfer = clib_min (max_dequeue, max_enqueue);
@@ -404,40 +427,35 @@ echo_server_rx_callback (session_t * s)
SESSION_IO_EVT_BUILTIN_RX))
es_err ("failed to enqueue self-tap");
- vec_validate (esm->rx_retries[s->thread_index], s->session_index);
- if (esm->rx_retries[thread_index][s->session_index] == 500000)
+ if (es->rx_retries == 500000)
{
es_err ("session stuck: %U", format_session, s, 2);
}
- if (esm->rx_retries[thread_index][s->session_index] < 500001)
- esm->rx_retries[thread_index][s->session_index]++;
+ if (es->rx_retries < 500001)
+ es->rx_retries++;
}
return 0;
}
- vec_validate (esm->rx_buf[thread_index], max_transfer);
+ vec_validate (wrk->rx_buf, max_transfer);
if (!esm->is_dgram)
{
- actual_transfer = app_recv_stream_raw (rx_fifo,
- esm->rx_buf[thread_index],
- max_transfer,
- 0 /* don't clear event */ ,
- 0 /* peek */ );
+ actual_transfer =
+ app_recv_stream_raw (rx_fifo, wrk->rx_buf, max_transfer,
+ 0 /* don't clear event */, 0 /* peek */);
}
else
{
- actual_transfer = app_recv_dgram_raw (rx_fifo,
- esm->rx_buf[thread_index],
- max_transfer, &at,
- 0 /* don't clear event */ ,
- 0 /* peek */ );
+ actual_transfer =
+ app_recv_dgram_raw (rx_fifo, wrk->rx_buf, max_transfer, &at,
+ 0 /* don't clear event */, 0 /* peek */);
}
ASSERT (actual_transfer == max_transfer);
if (esm->cfg.test_bytes)
{
- es_test_bytes (esm, actual_transfer);
+ es_test_bytes (wrk, es, actual_transfer);
}
/*
@@ -446,19 +464,15 @@ echo_server_rx_callback (session_t * s)
if (!esm->is_dgram)
{
- n_written = app_send_stream_raw (tx_fifo,
- esm->vpp_queue[thread_index],
- esm->rx_buf[thread_index],
- actual_transfer, SESSION_IO_EVT_TX,
- 1 /* do_evt */ , 0);
+ n_written = app_send_stream_raw (tx_fifo, wrk->vpp_event_queue,
+ wrk->rx_buf, actual_transfer,
+ SESSION_IO_EVT_TX, 1 /* do_evt */, 0);
}
else
{
- n_written = app_send_dgram_raw (tx_fifo, &at,
- esm->vpp_queue[s->thread_index],
- esm->rx_buf[thread_index],
- actual_transfer, SESSION_IO_EVT_TX,
- 1 /* do_evt */ , 0);
+ n_written = app_send_dgram_raw (tx_fifo, &at, wrk->vpp_event_queue,
+ wrk->rx_buf, actual_transfer,
+ SESSION_IO_EVT_TX, 1 /* do_evt */, 0);
}
if (n_written != max_transfer)
@@ -631,27 +645,18 @@ echo_server_create (vlib_main_t * vm, u8 * appns_id, u64 appns_flags,
{
echo_server_main_t *esm = &echo_server_main;
vlib_thread_main_t *vtm = vlib_get_thread_main ();
- u32 num_threads;
- int i;
+ es_worker_t *wrk;
+
+ esm->rcv_buffer_size = clib_max (esm->rcv_buffer_size, esm->fifo_size);
+ vec_validate (esm->wrk, vtm->n_threads);
- num_threads = 1 /* main thread */ + vtm->n_threads;
- vec_validate (echo_server_main.vpp_queue, num_threads - 1);
- vec_validate (esm->rx_buf, num_threads - 1);
- vec_validate (esm->rx_retries, num_threads - 1);
- vec_validate (esm->session_handles, num_threads - 1);
- for (i = 0; i < vec_len (esm->rx_retries); i++)
+ vec_foreach (wrk, esm->wrk)
{
- vec_validate (esm->rx_retries[i],
- pool_elts (session_main.wrk[i].sessions));
- vec_validate (esm->session_handles[i],
- pool_elts (session_main.wrk[i].sessions));
- clib_memset (esm->session_handles[i], ~0,
- sizeof (u64) * vec_len (esm->session_handles[i]));
- vec_reset_length (esm->session_handles[i]);
+ wrk->thread_index = wrk - esm->wrk;
+ vec_validate (wrk->rx_buf, esm->rcv_buffer_size);
+ wrk->vpp_event_queue =
+ session_main_get_vpp_event_queue (wrk->thread_index);
}
- esm->rcv_buffer_size = clib_max (esm->rcv_buffer_size, esm->fifo_size);
- for (i = 0; i < num_threads; i++)
- vec_validate (esm->rx_buf[i], esm->rcv_buffer_size);
if (echo_server_attach (appns_id, appns_flags, appns_secret))
{
@@ -798,8 +803,6 @@ VLIB_CLI_COMMAND (echo_server_create_command, static) = {
clib_error_t *
echo_server_main_init (vlib_main_t * vm)
{
- echo_server_main_t *esm = &echo_server_main;
- esm->my_client_index = ~0;
return 0;
}