From 3348a4cf070b90a9c23bbc0b3752fa2801f832a9 Mon Sep 17 00:00:00 2001 From: Florin Coras Date: Fri, 7 Sep 2018 17:09:35 -0700 Subject: [PATCH] vcl: set worker pthread stop key Otherwise the key destructor is not called on pthread_exit. Change-Id: I11e6b9683a926eecd3f40a44aab41924ff9c3101 Signed-off-by: Florin Coras --- src/vcl/vcl_bapi.c | 3 +++ src/vcl/vcl_private.c | 6 +++++- src/vcl/vcl_private.h | 3 +++ src/vnet/session/session_api.c | 2 +- 4 files changed, 12 insertions(+), 2 deletions(-) diff --git a/src/vcl/vcl_bapi.c b/src/vcl/vcl_bapi.c index d702d7c0736..7d02c4fc793 100644 --- a/src/vcl/vcl_bapi.c +++ b/src/vcl/vcl_bapi.c @@ -155,6 +155,9 @@ vl_api_app_worker_add_del_reply_t_handler (vl_api_app_worker_add_del_reply_t * getpid (), mp->context, wrk_index); goto failed; } + if (!mp->is_add) + return; + wrk = vcl_worker_get (wrk_index); wrk->app_event_queue = uword_to_pointer (mp->app_event_queue_address, svm_msg_q_t *); diff --git a/src/vcl/vcl_private.c b/src/vcl/vcl_private.c index ae4498ef4b5..0b8c2da45db 100644 --- a/src/vcl/vcl_private.c +++ b/src/vcl/vcl_private.c @@ -15,7 +15,7 @@ #include -pthread_key_t vcl_worker_stop_key; +static pthread_key_t vcl_worker_stop_key; static const char * vppcom_app_state_str (app_state_t state) @@ -224,6 +224,7 @@ static void vcl_worker_cleanup (void *arg) { vcl_worker_t *wrk = vcl_worker_get_current (); + VDBG (0, "cleaning up worker %u", wrk->wrk_index); vcl_send_app_worker_add_del (0 /* is_add */ ); close (wrk->mqs_epfd); @@ -290,6 +291,9 @@ vcl_worker_alloc_and_init () if (pthread_key_create (&vcl_worker_stop_key, vcl_worker_cleanup)) clib_warning ("failed to add pthread cleanup function"); + if (pthread_setspecific (vcl_worker_stop_key, &wrk->thread_id)) + clib_warning ("failed to setup key value"); + wrk->thread_id = pthread_self (); clib_spinlock_unlock (&vcm->workers_lock); diff --git a/src/vcl/vcl_private.h b/src/vcl/vcl_private.h index 75f16ca74c2..11d957a1317 100644 --- a/src/vcl/vcl_private.h +++ b/src/vcl/vcl_private.h @@ -257,6 +257,9 @@ typedef struct vcl_worker_ /** Vector acting as buffer for mq messages */ svm_msg_q_msg_t *mq_msg_vector; + + /** Used also as a thread stop key buffer */ + pthread_t thread_id; } vcl_worker_t; typedef struct vppcom_main_t_ diff --git a/src/vnet/session/session_api.c b/src/vnet/session/session_api.c index 6b72e7bc482..05b3bb89f0c 100755 --- a/src/vnet/session/session_api.c +++ b/src/vnet/session/session_api.c @@ -1375,9 +1375,9 @@ vl_api_app_worker_add_del_t_handler (vl_api_app_worker_add_del_t * mp) done: REPLY_MACRO2 (VL_API_APP_WORKER_ADD_DEL_REPLY, ({ rmp->is_add = mp->is_add; + rmp->wrk_index = clib_host_to_net_u32 (args.wrk_index); if (!rv && mp->is_add) { - rmp->wrk_index = clib_host_to_net_u32 (args.wrk_index); if (vec_len (args.segment->name)) { memcpy (rmp->segment_name, args.segment->name, -- 2.16.6