aboutsummaryrefslogtreecommitdiffstats
path: root/src/vcl/vcl_test_server.c
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2018-09-04 22:07:27 -0700
committerMarco Varlese <marco.varlese@suse.de>2018-09-07 09:36:00 +0000
commit6d4bb42540768a94abd70d37da6716341a515c40 (patch)
tree0e12d555d8121a33acffa106ae2f0f61f74411ee /src/vcl/vcl_test_server.c
parent49ca2601114a8a29520cba5f0ab3ea1dc1968f7f (diff)
vcl: refactor test client to support workers
Change-Id: I7fcfddc2bc7d9a64f8aa0d57ba5d11d325a15ce1 Signed-off-by: Florin Coras <fcoras@cisco.com>
Diffstat (limited to 'src/vcl/vcl_test_server.c')
-rw-r--r--src/vcl/vcl_test_server.c158
1 files changed, 62 insertions, 96 deletions
diff --git a/src/vcl/vcl_test_server.c b/src/vcl/vcl_test_server.c
index d1f2db72872..8d97d0cfaa6 100644
--- a/src/vcl/vcl_test_server.c
+++ b/src/vcl/vcl_test_server.c
@@ -78,26 +78,6 @@ static __thread int __wrk_index = 0;
static vcl_test_server_main_t sock_server_main;
-#define vtfail(_fn, _rv) \
-{ \
- errno = -_rv; \
- perror ("ERROR when calling " _fn); \
- fprintf (stderr, "\nSERVER ERROR: " _fn " failed (errno = %d)!\n", -_rv);\
- exit (1); \
-}
-
-#define vterr(_fn, _rv) \
-{ \
- errno = -_rv; \
- fprintf (stderr, "\nSERVER ERROR: " _fn " failed (errno = %d)!\n", -_rv);\
-}
-
-#define vtwrn(_fmt, _args...) \
- fprintf (stderr, "\nSERVER ERROR: " _fmt "\n", ##_args) \
-
-#define vtinf(_fmt, _args...) \
- fprintf (stdout, "\nvts<w%u>: " _fmt "\n", __wrk_index, ##_args) \
-
static inline void
conn_pool_expand (vcl_test_server_worker_t * wrk, size_t expand_size)
{
@@ -171,67 +151,60 @@ sync_config_and_reply (vcl_test_server_conn_t * conn,
}
static void
-stream_test_server_start_stop (vcl_test_server_worker_t * wrk,
- vcl_test_server_conn_t * conn,
- sock_test_cfg_t * rx_cfg)
+vts_server_start_stop (vcl_test_server_worker_t * wrk,
+ vcl_test_server_conn_t * conn,
+ sock_test_cfg_t * rx_cfg)
{
- int client_fd = conn->fd;
- sock_test_t test = rx_cfg->test;
+ u8 is_bi = rx_cfg->test == SOCK_TEST_TYPE_BI;
+ int client_fd = conn->fd, i;
+ vcl_test_server_conn_t *tc;
+ char buf[64];
if (rx_cfg->ctrl_handle == conn->fd)
{
- int i;
clock_gettime (CLOCK_REALTIME, &conn->stats.stop);
for (i = 0; i < wrk->conn_pool_size; i++)
{
- vcl_test_server_conn_t *tc = &wrk->conn_pool[i];
-
- if (tc->cfg.ctrl_handle == conn->fd)
- {
- sock_test_stats_accumulate (&conn->stats, &tc->stats);
+ tc = &wrk->conn_pool[i];
+ if (tc->cfg.ctrl_handle != conn->fd)
+ continue;
- if (conn->cfg.verbose)
- {
- static char buf[64];
+ sock_test_stats_accumulate (&conn->stats, &tc->stats);
- sprintf (buf, "SERVER (fd %d) RESULTS", tc->fd);
- sock_test_stats_dump (buf, &tc->stats, 1 /* show_rx */ ,
- test == SOCK_TEST_TYPE_BI
- /* show tx */ ,
- conn->cfg.verbose);
- }
+ if (conn->cfg.verbose)
+ {
+ sprintf (buf, "SERVER (fd %d) RESULTS", tc->fd);
+ sock_test_stats_dump (buf, &tc->stats, 1 /* show_rx */ ,
+ is_bi /* show tx */ , conn->cfg.verbose);
}
}
sock_test_stats_dump ("SERVER RESULTS", &conn->stats, 1 /* show_rx */ ,
- (test == SOCK_TEST_TYPE_BI) /* show_tx */ ,
- conn->cfg.verbose);
+ is_bi /* show_tx */ , conn->cfg.verbose);
sock_test_cfg_dump (&conn->cfg, 0 /* is_client */ );
if (conn->cfg.verbose)
{
- printf (" sock server main\n"
- SOCK_TEST_SEPARATOR_STRING
- " buf: %p\n"
- " buf size: %u (0x%08x)\n"
- SOCK_TEST_SEPARATOR_STRING,
- conn->buf, conn->buf_size, conn->buf_size);
+ vtinf (" sock server main\n"
+ SOCK_TEST_SEPARATOR_STRING
+ " buf: %p\n"
+ " buf size: %u (0x%08x)\n"
+ SOCK_TEST_SEPARATOR_STRING,
+ conn->buf, conn->buf_size, conn->buf_size);
}
sync_config_and_reply (conn, rx_cfg);
- printf ("\nSERVER (fd %d): %s-directional Stream Test Complete!\n"
- SOCK_TEST_BANNER_STRING "\n", conn->fd,
- test == SOCK_TEST_TYPE_BI ? "Bi" : "Uni");
+ vtinf ("(fd %d): %s-directional Stream Test Complete!\n"
+ SOCK_TEST_BANNER_STRING "\n", conn->fd, is_bi ? "Bi" : "Uni");
}
else
{
- printf ("\n" SOCK_TEST_BANNER_STRING
- "SERVER (fd %d): %s-directional Stream Test!\n"
- " Sending client the test cfg to start streaming data...\n",
- client_fd, test == SOCK_TEST_TYPE_BI ? "Bi" : "Uni");
+ vtinf (SOCK_TEST_BANNER_STRING "(fd %d): %s-directional Stream Test!\n"
+ " Sending client the test cfg to start streaming data...\n",
+ client_fd, is_bi ? "Bi" : "Uni");
- rx_cfg->ctrl_handle = (rx_cfg->ctrl_handle == ~0) ? conn->fd :
- rx_cfg->ctrl_handle;
+ if (rx_cfg->ctrl_handle == ~0)
+ rx_cfg->ctrl_handle = conn->fd;
sync_config_and_reply (conn, rx_cfg);
@@ -241,9 +214,8 @@ stream_test_server_start_stop (vcl_test_server_worker_t * wrk,
}
}
-
static inline void
-stream_test_server (vcl_test_server_conn_t * conn, int rx_bytes)
+vts_server_rx (vcl_test_server_conn_t * conn, int rx_bytes)
{
int client_fd = conn->fd;
sock_test_t test = conn->cfg.test;
@@ -253,13 +225,11 @@ stream_test_server (vcl_test_server_conn_t * conn, int rx_bytes)
conn->cfg.verbose);
if (conn->stats.rx_bytes >= conn->cfg.total_bytes)
- {
- clock_gettime (CLOCK_REALTIME, &conn->stats.stop);
- }
+ clock_gettime (CLOCK_REALTIME, &conn->stats.stop);
}
static void
-vcl_test_server_echo (vcl_test_server_conn_t * conn, int rx_bytes)
+vts_server_echo (vcl_test_server_conn_t * conn, int rx_bytes)
{
int tx_bytes, nbytes, pos;
@@ -281,7 +251,7 @@ vcl_test_server_echo (vcl_test_server_conn_t * conn, int rx_bytes)
}
static inline void
-vcl_test_server_new_client (vcl_test_server_worker_t * wrk)
+vts_new_client (vcl_test_server_worker_t * wrk)
{
int client_fd;
vcl_test_server_conn_t *conn;
@@ -370,7 +340,7 @@ vcl_test_init_endpoint_addr (vcl_test_server_main_t * ssm)
}
}
-void
+static void
vcl_test_server_process_opts (vcl_test_server_main_t * ssm, int argc,
char **argv)
{
@@ -403,11 +373,9 @@ vcl_test_server_process_opts (vcl_test_server_main_t * ssm, int argc,
{
default:
if (isprint (optopt))
- fprintf (stderr, "SERVER: ERROR: Unknown "
- "option `-%c'.\n", optopt);
+ vtwrn ("Unknown option `-%c'.", optopt);
else
- fprintf (stderr, "SERVER: ERROR: Unknown "
- "option character `\\x%x'.\n", optopt);
+ vtwrn ("Unknown option character `\\x%x'.", optopt);
}
/* fall thru */
case 'h':
@@ -433,9 +401,8 @@ vcl_test_server_process_opts (vcl_test_server_main_t * ssm, int argc,
}
int
-vcl_test_server_handle_cfg (vcl_test_server_worker_t * wrk,
- sock_test_cfg_t * rx_cfg,
- vcl_test_server_conn_t * conn, int rx_bytes)
+vts_handle_cfg (vcl_test_server_worker_t * wrk, sock_test_cfg_t * rx_cfg,
+ vcl_test_server_conn_t * conn, int rx_bytes)
{
if (rx_cfg->verbose)
{
@@ -468,7 +435,7 @@ vcl_test_server_handle_cfg (vcl_test_server_worker_t * wrk,
case SOCK_TEST_TYPE_BI:
case SOCK_TEST_TYPE_UNI:
- stream_test_server_start_stop (wrk, conn, rx_cfg);
+ vts_server_start_stop (wrk, conn, rx_cfg);
break;
case SOCK_TEST_TYPE_EXIT:
@@ -488,7 +455,7 @@ vcl_test_server_handle_cfg (vcl_test_server_worker_t * wrk,
}
static void
-vcl_test_server_worker_init (vcl_test_server_worker_t * wrk)
+vts_worker_init (vcl_test_server_worker_t * wrk)
{
vcl_test_server_main_t *ssm = &sock_server_main;
struct epoll_event listen_ev;
@@ -533,7 +500,7 @@ vcl_test_server_worker_init (vcl_test_server_worker_t * wrk)
}
static void *
-vcl_test_server_worker_loop (void *arg)
+vts_worker_loop (void *arg)
{
vcl_test_server_main_t *ssm = &sock_server_main;
vcl_test_server_worker_t *wrk = arg;
@@ -542,7 +509,7 @@ vcl_test_server_worker_loop (void *arg)
sock_test_cfg_t *rx_cfg;
if (wrk->wrk_index)
- vcl_test_server_worker_init (wrk);
+ vts_worker_init (wrk);
while (1)
{
@@ -568,7 +535,7 @@ vcl_test_server_worker_loop (void *arg)
}
if (wrk->wait_events[i].data.u32 == ~0)
{
- vcl_test_server_new_client (wrk);
+ vts_new_client (wrk);
continue;
}
@@ -591,7 +558,7 @@ vcl_test_server_worker_loop (void *arg)
rx_cfg = (sock_test_cfg_t *) conn->buf;
if (rx_cfg->magic == SOCK_TEST_CFG_CTRL_MAGIC)
{
- vcl_test_server_handle_cfg (wrk, rx_cfg, conn, rx_bytes);
+ vts_handle_cfg (wrk, rx_cfg, conn, rx_bytes);
if (!wrk->nfds)
{
vtinf ("All client connections closed\n");
@@ -603,12 +570,12 @@ vcl_test_server_worker_loop (void *arg)
else if ((conn->cfg.test == SOCK_TEST_TYPE_UNI)
|| (conn->cfg.test == SOCK_TEST_TYPE_BI))
{
- stream_test_server (conn, rx_bytes);
+ vts_server_rx (conn, rx_bytes);
continue;
}
else if (isascii (conn->buf[0]))
{
- vcl_test_server_echo (conn, rx_bytes);
+ vts_server_echo (conn, rx_bytes);
}
else
{
@@ -637,37 +604,36 @@ done:
int
main (int argc, char **argv)
{
- vcl_test_server_main_t *ssm = &sock_server_main;
+ vcl_test_server_main_t *vsm = &sock_server_main;
int rv, i;
clib_mem_init_thread_safe (0, 64 << 20);
- ssm->cfg.port = SOCK_TEST_SERVER_PORT;
- ssm->cfg.workers = 1;
- ssm->active_workers = 1;
- vcl_test_server_process_opts (ssm, argc, argv);
+ vsm->cfg.port = SOCK_TEST_SERVER_PORT;
+ vsm->cfg.workers = 1;
+ vsm->active_workers = 1;
+ vcl_test_server_process_opts (vsm, argc, argv);
rv = vppcom_app_create ("vcl_test_server");
if (rv)
vtfail ("vppcom_app_create()", rv);
- ssm->workers = calloc (ssm->cfg.workers, sizeof (*ssm->workers));
- vcl_test_server_worker_init (&ssm->workers[0]);
- for (i = 1; i < ssm->cfg.workers; i++)
+ vsm->workers = calloc (vsm->cfg.workers, sizeof (*vsm->workers));
+ vts_worker_init (&vsm->workers[0]);
+ for (i = 1; i < vsm->cfg.workers; i++)
{
- ssm->workers[i].wrk_index = i;
- rv = pthread_create (&ssm->workers[i].thread_handle, NULL,
- vcl_test_server_worker_loop,
- (void *) &ssm->workers[i]);
+ vsm->workers[i].wrk_index = i;
+ rv = pthread_create (&vsm->workers[i].thread_handle, NULL,
+ vts_worker_loop, (void *) &vsm->workers[i]);
}
- vcl_test_server_worker_loop (&ssm->workers[0]);
+ vts_worker_loop (&vsm->workers[0]);
- while (ssm->active_workers > 0)
+ while (vsm->active_workers > 0)
;
vppcom_app_destroy ();
- free (ssm->workers);
+ free (vsm->workers);
- return ssm->worker_fails;
+ return vsm->worker_fails;
}
/*