hsa: do not drop the barrier when creating echo server 14/33814/2
authorFilip Tehlar <ftehlar@cisco.com>
Fri, 24 Sep 2021 06:21:25 +0000 (06:21 +0000)
committerFlorin Coras <florin.coras@gmail.com>
Mon, 4 Oct 2021 18:09:47 +0000 (18:09 +0000)
Type: fix

Signed-off-by: Filip Tehlar <ftehlar@cisco.com>
Change-Id: I25d3ac72360bea130f567095b486d8e295d2f2f7

src/plugins/hs_apps/echo_server.c

index b75a366..35641c3 100644 (file)
@@ -468,19 +468,9 @@ echo_server_create_command_fn (vlib_main_t * vm, unformat_input_t * input,
   u8 server_uri_set = 0, *appns_id = 0;
   u64 tmp, appns_flags = 0, appns_secret = 0;
   char *default_uri = "tcp://0.0.0.0/1234";
-  int rv, is_stop = 0, barrier_acq_needed = 0;
+  int rv, is_stop = 0;
   clib_error_t *error = 0;
 
-  /* The request came over the binary api and the inband cli handler
-   * is not mp_safe. Drop the barrier to make sure the workers are not
-   * blocked.
-   */
-  if (vlib_num_workers () && vlib_thread_is_main_w_barrier ())
-    {
-      barrier_acq_needed = 1;
-      vlib_worker_thread_barrier_release (vm);
-    }
-
   esm->no_echo = 0;
   esm->fifo_size = 64 << 10;
   esm->rcv_buffer_size = 128 << 10;
@@ -584,9 +574,6 @@ echo_server_create_command_fn (vlib_main_t * vm, unformat_input_t * input,
 cleanup:
   vec_free (appns_id);
 
-  if (barrier_acq_needed)
-    vlib_worker_thread_barrier_sync (vm);
-
   return error;
 }