vlib: refactor node function variants
[vpp.git] / src / vlib / main.c
index cb651e4..02fdc89 100644 (file)
@@ -122,6 +122,8 @@ vlib_frame_alloc_to_node (vlib_main_t * vm, u32 to_node_index,
   vlib_frame_t *f;
   u32 l, n, scalar_size, vector_size;
 
+  ASSERT (vm == vlib_get_main ());
+
   to_node = vlib_get_node (vm, to_node_index);
 
   scalar_size = to_node->scalar_size;
@@ -221,6 +223,8 @@ vlib_put_frame_to_node (vlib_main_t * vm, u32 to_node_index, vlib_frame_t * f)
   if (f->n_vectors == 0)
     return;
 
+  ASSERT (vm == vlib_get_main ());
+
   vlib_validate_frame_indices (f);
 
   to_node = vlib_get_node (vm, to_node_index);
@@ -241,6 +245,7 @@ vlib_frame_free (vlib_main_t * vm, vlib_node_runtime_t * r, vlib_frame_t * f)
   vlib_node_t *node;
   vlib_frame_size_t *fs;
 
+  ASSERT (vm == vlib_get_main ());
   ASSERT (f->frame_flags & VLIB_FRAME_IS_ALLOCATED);
 
   node = vlib_get_node (vm, r->node_index);
@@ -734,21 +739,12 @@ elog_save_buffer (vlib_main_t * vm,
 }
 
 void
-elog_post_mortem_dump (void)
+vlib_post_mortem_dump (void)
 {
   vlib_main_t *vm = &vlib_global_main;
-  elog_main_t *em = &vm->elog_main;
-  u8 *filename;
-  clib_error_t *error;
 
-  if (!vm->elog_post_mortem_dump)
-    return;
-
-  filename = format (0, "/tmp/elog_post_mortem.%d%c", getpid (), 0);
-  error = elog_write_file (em, (char *) filename, 1 /* flush ring */ );
-  if (error)
-    clib_error_report (error);
-  vec_free (filename);
+  for (int i = 0; i < vec_len (vm->post_mortem_callbacks); i++)
+    (vm->post_mortem_callbacks[i]) ();
 }
 
 /* *INDENT-OFF* */
@@ -800,8 +796,8 @@ VLIB_CLI_COMMAND (elog_restart_cli, static) = {
 /* *INDENT-ON* */
 
 static clib_error_t *
-elog_resize (vlib_main_t * vm,
-            unformat_input_t * input, vlib_cli_command_t * cmd)
+elog_resize_command_fn (vlib_main_t * vm,
+                       unformat_input_t * input, vlib_cli_command_t * cmd)
 {
   elog_main_t *em = &vm->elog_main;
   u32 tmp;
@@ -825,7 +821,7 @@ elog_resize (vlib_main_t * vm,
 VLIB_CLI_COMMAND (elog_resize_cli, static) = {
   .path = "event-logger resize",
   .short_help = "event-logger resize <nnn>",
-  .function = elog_resize,
+  .function = elog_resize_command_fn,
 };
 /* *INDENT-ON* */
 
@@ -958,163 +954,6 @@ add_trajectory_trace (vlib_buffer_t * b, u32 node_index)
 #endif
 }
 
-u8 *format_vnet_buffer_flags (u8 * s, va_list * args) __attribute__ ((weak));
-u8 *
-format_vnet_buffer_flags (u8 * s, va_list * args)
-{
-  s = format (s, "BUG STUB %s", __FUNCTION__);
-  return s;
-}
-
-u8 *format_vnet_buffer_opaque (u8 * s, va_list * args) __attribute__ ((weak));
-u8 *
-format_vnet_buffer_opaque (u8 * s, va_list * args)
-{
-  s = format (s, "BUG STUB %s", __FUNCTION__);
-  return s;
-}
-
-u8 *format_vnet_buffer_opaque2 (u8 * s, va_list * args)
-  __attribute__ ((weak));
-u8 *
-format_vnet_buffer_opaque2 (u8 * s, va_list * args)
-{
-  s = format (s, "BUG STUB %s", __FUNCTION__);
-  return s;
-}
-
-static u8 *
-format_buffer_metadata (u8 * s, va_list * args)
-{
-  vlib_buffer_t *b = va_arg (*args, vlib_buffer_t *);
-
-  s = format (s, "flags: %U\n", format_vnet_buffer_flags, b);
-  s = format (s, "current_data: %d, current_length: %d\n",
-             (i32) (b->current_data), (i32) (b->current_length));
-  s = format
-    (s,
-     "current_config_index/punt_reason: %d, flow_id: %x, next_buffer: %x\n",
-     b->current_config_index, b->flow_id, b->next_buffer);
-  s =
-    format (s, "error: %d, ref_count: %d, buffer_pool_index: %d\n",
-           (u32) (b->error), (u32) (b->ref_count),
-           (u32) (b->buffer_pool_index));
-  s =
-    format (s, "trace_handle: 0x%x, len_not_first_buf: %d\n", b->trace_handle,
-           b->total_length_not_including_first_buffer);
-  return s;
-}
-
-#define A(x) vec_add1(vm->pcap_buffer, (x))
-
-static void
-dispatch_pcap_trace (vlib_main_t * vm,
-                    vlib_node_runtime_t * node, vlib_frame_t * frame)
-{
-  int i;
-  vlib_buffer_t *bufs[VLIB_FRAME_SIZE], **bufp, *b;
-  pcap_main_t *pm = &vlib_global_main.dispatch_pcap_main;
-  vlib_trace_main_t *tm = &vm->trace_main;
-  u32 capture_size;
-  vlib_node_t *n;
-  i32 n_left;
-  f64 time_now = vlib_time_now (vm);
-  u32 *from;
-  u8 *d;
-  u8 string_count;
-
-  /* Input nodes don't have frames yet */
-  if (frame == 0 || frame->n_vectors == 0)
-    return;
-
-  from = vlib_frame_vector_args (frame);
-  vlib_get_buffers (vm, from, bufs, frame->n_vectors);
-  bufp = bufs;
-
-  n = vlib_get_node (vm, node->node_index);
-
-  for (i = 0; i < frame->n_vectors; i++)
-    {
-      if (PREDICT_TRUE (pm->n_packets_captured < pm->n_packets_to_capture))
-       {
-         b = bufp[i];
-
-         vec_reset_length (vm->pcap_buffer);
-         string_count = 0;
-
-         /* Version, flags */
-         A ((u8) VLIB_PCAP_MAJOR_VERSION);
-         A ((u8) VLIB_PCAP_MINOR_VERSION);
-         A (0 /* string_count */ );
-         A (n->protocol_hint);
-
-         /* Buffer index (big endian) */
-         A ((from[i] >> 24) & 0xff);
-         A ((from[i] >> 16) & 0xff);
-         A ((from[i] >> 8) & 0xff);
-         A ((from[i] >> 0) & 0xff);
-
-         /* Node name, NULL-terminated ASCII */
-         vm->pcap_buffer = format (vm->pcap_buffer, "%v%c", n->name, 0);
-         string_count++;
-
-         vm->pcap_buffer = format (vm->pcap_buffer, "%U%c",
-                                   format_buffer_metadata, b, 0);
-         string_count++;
-         vm->pcap_buffer = format (vm->pcap_buffer, "%U%c",
-                                   format_vnet_buffer_opaque, b, 0);
-         string_count++;
-         vm->pcap_buffer = format (vm->pcap_buffer, "%U%c",
-                                   format_vnet_buffer_opaque2, b, 0);
-         string_count++;
-
-         /* Is this packet traced? */
-         if (PREDICT_FALSE (b->flags & VLIB_BUFFER_IS_TRACED))
-           {
-             vlib_trace_header_t **h
-               = pool_elt_at_index (tm->trace_buffer_pool,
-                                    vlib_buffer_get_trace_index (b));
-
-             vm->pcap_buffer = format (vm->pcap_buffer, "%U%c",
-                                       format_vlib_trace, vm, h[0], 0);
-             string_count++;
-           }
-
-         /* Save the string count */
-         vm->pcap_buffer[2] = string_count;
-
-         /* Figure out how many bytes in the pcap trace */
-         capture_size = vec_len (vm->pcap_buffer) +
-           +vlib_buffer_length_in_chain (vm, b);
-
-         clib_spinlock_lock_if_init (&pm->lock);
-         n_left = clib_min (capture_size, 16384);
-         d = pcap_add_packet (pm, time_now, n_left, capture_size);
-
-         /* Copy the header */
-         clib_memcpy_fast (d, vm->pcap_buffer, vec_len (vm->pcap_buffer));
-         d += vec_len (vm->pcap_buffer);
-
-         n_left = clib_min
-           (vlib_buffer_length_in_chain (vm, b),
-            (16384 - vec_len (vm->pcap_buffer)));
-         /* Copy the packet data */
-         while (1)
-           {
-             u32 copy_length = clib_min ((u32) n_left, b->current_length);
-             clib_memcpy_fast (d, b->data + b->current_data, copy_length);
-             n_left -= b->current_length;
-             if (n_left <= 0)
-               break;
-             d += b->current_length;
-             ASSERT (b->flags & VLIB_BUFFER_NEXT_PRESENT);
-             b = vlib_get_buffer (vm, b->next_buffer);
-           }
-         clib_spinlock_unlock_if_init (&pm->lock);
-       }
-    }
-}
-
 static_always_inline u64
 dispatch_node (vlib_main_t * vm,
               vlib_node_runtime_t * node,
@@ -1183,15 +1022,17 @@ dispatch_node (vlib_main_t * vm,
          vlib_buffer_t *b = vlib_get_buffer (vm, from[i]);
          add_trajectory_trace (b, node->node_index);
        }
-      if (PREDICT_FALSE (vm->dispatch_pcap_enable))
-       dispatch_pcap_trace (vm, node, frame);
-      n = node->function (vm, node, frame);
+      if (PREDICT_TRUE (vm->dispatch_wrapper_fn == 0))
+       n = node->function (vm, node, frame);
+      else
+       n = vm->dispatch_wrapper_fn (vm, node, frame);
     }
   else
     {
-      if (PREDICT_FALSE (vm->dispatch_pcap_enable))
-       dispatch_pcap_trace (vm, node, frame);
-      n = node->function (vm, node, frame);
+      if (PREDICT_TRUE (vm->dispatch_wrapper_fn == 0))
+       n = node->function (vm, node, frame);
+      else
+       n = vm->dispatch_wrapper_fn (vm, node, frame);
     }
 
   t = clib_cpu_time_now ();
@@ -1304,7 +1145,7 @@ dispatch_pending_node (vlib_main_t * vm, uword pending_frame_index,
 {
   vlib_node_main_t *nm = &vm->node_main;
   vlib_frame_t *f;
-  vlib_next_frame_t *nf, nf_dummy;
+  vlib_next_frame_t *nf, nf_placeholder;
   vlib_node_runtime_t *n;
   vlib_frame_t *restore_frame;
   vlib_pending_frame_t *p;
@@ -1318,8 +1159,8 @@ dispatch_pending_node (vlib_main_t * vm, uword pending_frame_index,
   f = vlib_get_frame (vm, p->frame);
   if (p->next_frame_index == VLIB_PENDING_FRAME_NO_NEXT_FRAME)
     {
-      /* No next frame: so use dummy on stack. */
-      nf = &nf_dummy;
+      /* No next frame: so use placeholder on stack. */
+      nf = &nf_placeholder;
       nf->flags = f->frame_flags & VLIB_NODE_FLAG_TRACE;
       nf->frame = NULL;
     }
@@ -1677,27 +1518,6 @@ vl_api_send_pending_rpc_requests (vlib_main_t * vm)
 {
 }
 
-static_always_inline u64
-dispatch_pending_interrupts (vlib_main_t * vm, vlib_node_main_t * nm,
-                            u64 cpu_time_now)
-{
-  vlib_node_runtime_t *n;
-
-  for (int i = 0; i < _vec_len (nm->pending_local_interrupts); i++)
-    {
-      vlib_node_interrupt_t *in;
-      in = vec_elt_at_index (nm->pending_local_interrupts, i);
-      n = vec_elt_at_index (nm->nodes_by_type[VLIB_NODE_TYPE_INPUT],
-                           in->node_runtime_index);
-      n->interrupt_data = in->data;
-      cpu_time_now = dispatch_node (vm, n, VLIB_NODE_TYPE_INPUT,
-                                   VLIB_NODE_STATE_INTERRUPT, /* frame */ 0,
-                                   cpu_time_now);
-    }
-  vec_reset_length (nm->pending_local_interrupts);
-  return cpu_time_now;
-}
-
 static_always_inline void
 vlib_main_or_worker_loop (vlib_main_t * vm, int is_main)
 {
@@ -1726,11 +1546,7 @@ vlib_main_or_worker_loop (vlib_main_t * vm, int is_main)
     cpu_time_now = clib_cpu_time_now ();
 
   /* Pre-allocate interupt runtime indices and lock. */
-  vec_alloc (nm->pending_local_interrupts, 32);
-  vec_alloc (nm->pending_remote_interrupts, 32);
-  vec_alloc_aligned (nm->pending_remote_interrupts_notify, 1,
-                    CLIB_CACHE_LINE_BYTES);
-  clib_spinlock_init (&nm->pending_interrupt_lock);
+  vec_alloc_aligned (nm->pending_interrupts, 1, CLIB_CACHE_LINE_BYTES);
 
   /* Pre-allocate expired nodes. */
   if (!nm->polling_threshold_vector_length)
@@ -1771,28 +1587,26 @@ vlib_main_or_worker_loop (vlib_main_t * vm, int is_main)
        }
 
       if (!is_main)
+       vlib_worker_thread_barrier_check ();
+
+      if (PREDICT_FALSE (vm->check_frame_queues + frame_queue_check_counter))
        {
-         vlib_worker_thread_barrier_check ();
-         if (PREDICT_FALSE (vm->check_frame_queues +
-                            frame_queue_check_counter))
-           {
-             u32 processed = 0;
+         u32 processed = 0;
 
-             if (vm->check_frame_queues)
-               {
-                 frame_queue_check_counter = 100;
-                 vm->check_frame_queues = 0;
-               }
+         if (vm->check_frame_queues)
+           {
+             frame_queue_check_counter = 100;
+             vm->check_frame_queues = 0;
+           }
 
-             vec_foreach (fqm, tm->frame_queue_mains)
-               processed += vlib_frame_queue_dequeue (vm, fqm);
+         vec_foreach (fqm, tm->frame_queue_mains)
+           processed += vlib_frame_queue_dequeue (vm, fqm);
 
-             /* No handoff queue work found? */
-             if (processed)
-               frame_queue_check_counter = 100;
-             else
-               frame_queue_check_counter--;
-           }
+         /* No handoff queue work found? */
+         if (processed)
+           frame_queue_check_counter = 100;
+         else
+           frame_queue_check_counter--;
        }
 
       if (PREDICT_FALSE (vec_len (vm->worker_thread_main_loop_callbacks)))
@@ -1819,26 +1633,22 @@ vlib_main_or_worker_loop (vlib_main_t * vm, int is_main)
       if (PREDICT_TRUE (is_main && vm->queue_signal_pending == 0))
        vm->queue_signal_callback (vm);
 
-      /* handle local interruots */
-      if (_vec_len (nm->pending_local_interrupts))
-       cpu_time_now = dispatch_pending_interrupts (vm, nm, cpu_time_now);
-
-      /* handle remote interruots */
-      if (PREDICT_FALSE (_vec_len (nm->pending_remote_interrupts)))
+      if (__atomic_load_n (nm->pending_interrupts, __ATOMIC_ACQUIRE))
        {
-         vlib_node_interrupt_t *in;
-
-         /* at this point it is known that
-          * vec_len (nm->pending_local_interrupts) is zero so we quickly swap
-          * local and remote vector under the spinlock */
-         clib_spinlock_lock (&nm->pending_interrupt_lock);
-         in = nm->pending_local_interrupts;
-         nm->pending_local_interrupts = nm->pending_remote_interrupts;
-         nm->pending_remote_interrupts = in;
-         *nm->pending_remote_interrupts_notify = 0;
-         clib_spinlock_unlock (&nm->pending_interrupt_lock);
-
-         cpu_time_now = dispatch_pending_interrupts (vm, nm, cpu_time_now);
+         int int_num = -1;
+         *nm->pending_interrupts = 0;
+
+         while ((int_num =
+                   clib_interrupt_get_next (nm->interrupts, int_num)) != -1)
+           {
+             vlib_node_runtime_t *n;
+             clib_interrupt_clear (nm->interrupts, int_num);
+             n = vec_elt_at_index (nm->nodes_by_type[VLIB_NODE_TYPE_INPUT],
+                                   int_num);
+             cpu_time_now = dispatch_node (vm, n, VLIB_NODE_TYPE_INPUT,
+                                           VLIB_NODE_STATE_INTERRUPT,
+                                           /* frame */ 0, cpu_time_now);
+           }
        }
 
       /* Input nodes may have added work to the pending vector.
@@ -1984,6 +1794,44 @@ vlib_worker_loop (vlib_main_t * vm)
 
 vlib_main_t vlib_global_main;
 
+void
+vlib_add_del_post_mortem_callback (void *cb, int is_add)
+{
+  vlib_main_t *vm = &vlib_global_main;
+  int i;
+
+  if (is_add == 0)
+    {
+      for (i = vec_len (vm->post_mortem_callbacks) - 1; i >= 0; i--)
+       if (vm->post_mortem_callbacks[i] == cb)
+         vec_del1 (vm->post_mortem_callbacks, i);
+      return;
+    }
+
+  for (i = 0; i < vec_len (vm->post_mortem_callbacks); i++)
+    if (vm->post_mortem_callbacks[i] == cb)
+      return;
+  vec_add1 (vm->post_mortem_callbacks, cb);
+}
+
+static void
+elog_post_mortem_dump (void)
+{
+  vlib_main_t *vm = &vlib_global_main;
+  elog_main_t *em = &vm->elog_main;
+
+  u8 *filename;
+  clib_error_t *error;
+
+  filename = format (0, "/tmp/elog_post_mortem.%d%c", getpid (), 0);
+  error = elog_write_file (em, (char *) filename, 1 /* flush ring */);
+  if (error)
+    clib_error_report (error);
+  /*
+   * We're in the middle of crashing. Don't try to free the filename.
+   */
+}
+
 static clib_error_t *
 vlib_main_configure (vlib_main_t * vm, unformat_input_t * input)
 {
@@ -1995,10 +1843,12 @@ vlib_main_configure (vlib_main_t * vm, unformat_input_t * input)
        turn_on_mem_trace = 1;
 
       else if (unformat (input, "elog-events %d",
-                        &vm->elog_main.event_ring_size))
-       ;
+                        &vm->configured_elog_ring_size))
+       vm->configured_elog_ring_size =
+         1 << max_log2 (vm->configured_elog_ring_size);
       else if (unformat (input, "elog-post-mortem-dump"))
-       vm->elog_post_mortem_dump = 1;
+       vlib_add_del_post_mortem_callback (elog_post_mortem_dump,
+                                          /* is_add */ 1);
       else if (unformat (input, "buffer-alloc-success-rate %f",
                         &vm->buffer_alloc_success_rate))
        {
@@ -2029,7 +1879,7 @@ vlib_main_configure (vlib_main_t * vm, unformat_input_t * input)
 VLIB_EARLY_CONFIG_FUNCTION (vlib_main_configure, "vlib");
 
 static void
-dummy_queue_signal_callback (vlib_main_t * vm)
+placeholder_queue_signal_callback (vlib_main_t * vm)
 {
 }
 
@@ -2075,13 +1925,12 @@ vlib_main (vlib_main_t * volatile vm, unformat_input_t * input)
   clib_error_t *volatile error;
   vlib_node_main_t *nm = &vm->node_main;
 
-  vm->queue_signal_callback = dummy_queue_signal_callback;
+  vm->queue_signal_callback = placeholder_queue_signal_callback;
 
-  /* Turn on event log. */
-  if (!vm->elog_main.event_ring_size)
-    vm->elog_main.event_ring_size = 128 << 10;
-  elog_init (&vm->elog_main, vm->elog_main.event_ring_size);
-  elog_enable_disable (&vm->elog_main, 1);
+  /* Reconfigure event log which is enabled very early */
+  if (vm->configured_elog_ring_size &&
+      vm->configured_elog_ring_size != vm->elog_main.event_ring_size)
+    elog_resize (&vm->elog_main, vm->configured_elog_ring_size);
   vl_api_set_elog_main (&vm->elog_main);
   (void) vl_api_set_elog_trace_api_messages (1);
 
@@ -2113,6 +1962,9 @@ vlib_main (vlib_main_t * volatile vm, unformat_input_t * input)
       goto done;
     }
 
+  /* Register node ifunction variants */
+  vlib_register_all_node_march_variants (vm);
+
   /* Register static nodes so that init functions may use them. */
   vlib_register_all_static_nodes (vm);
 
@@ -2222,6 +2074,7 @@ vlib_main (vlib_main_t * volatile vm, unformat_input_t * input)
   vlib_main_loop (vm);
 
 done:
+  vlib_worker_thread_barrier_sync (vm);
   /* Call all exit functions. */
   {
     clib_error_t *sub_error;
@@ -2229,6 +2082,7 @@ done:
     if (sub_error)
       clib_error_report (sub_error);
   }
+  vlib_worker_thread_barrier_release (vm);
 
   if (error)
     clib_error_report (error);
@@ -2236,270 +2090,18 @@ done:
   return 0;
 }
 
-int
-vlib_pcap_dispatch_trace_configure (vlib_pcap_dispatch_trace_args_t * a)
+vlib_main_t *
+vlib_get_main_not_inline (void)
 {
-  vlib_main_t *vm = vlib_get_main ();
-  pcap_main_t *pm = &vm->dispatch_pcap_main;
-  vlib_trace_main_t *tm;
-  vlib_trace_node_t *tn;
-
-  if (a->status)
-    {
-      if (vm->dispatch_pcap_enable)
-       {
-         int i;
-         vlib_cli_output
-           (vm, "pcap dispatch capture enabled: %d of %d pkts...",
-            pm->n_packets_captured, pm->n_packets_to_capture);
-         vlib_cli_output (vm, "capture to file %s", pm->file_name);
-
-         for (i = 0; i < vec_len (vm->dispatch_buffer_trace_nodes); i++)
-           {
-             vlib_cli_output (vm,
-                              "Buffer trace of %d pkts from %U enabled...",
-                              a->buffer_traces_to_capture,
-                              format_vlib_node_name, vm,
-                              vm->dispatch_buffer_trace_nodes[i]);
-           }
-       }
-      else
-       vlib_cli_output (vm, "pcap dispatch capture disabled");
-      return 0;
-    }
-
-  /* Consistency checks */
-
-  /* Enable w/ capture already enabled not allowed */
-  if (vm->dispatch_pcap_enable && a->enable)
-    return -7;                 /* VNET_API_ERROR_INVALID_VALUE */
-
-  /* Disable capture with capture already disabled, not interesting */
-  if (vm->dispatch_pcap_enable == 0 && a->enable == 0)
-    return -81;                        /* VNET_API_ERROR_VALUE_EXIST */
-
-  /* Change number of packets to capture while capturing */
-  if (vm->dispatch_pcap_enable && a->enable
-      && (pm->n_packets_to_capture != a->packets_to_capture))
-    return -8;                 /* VNET_API_ERROR_INVALID_VALUE_2 */
-
-  /* Independent of enable/disable, to allow buffer trace multi nodes */
-  if (a->buffer_trace_node_index != ~0)
-    {
-      /* *INDENT-OFF* */
-      foreach_vlib_main ((
-        {
-          tm = &this_vlib_main->trace_main;
-          tm->verbose = 0;  /* not sure this ever did anything... */
-          vec_validate (tm->nodes, a->buffer_trace_node_index);
-          tn = tm->nodes + a->buffer_trace_node_index;
-          tn->limit += a->buffer_traces_to_capture;
-          tm->trace_enable = 1;
-        }));
-      /* *INDENT-ON* */
-      vec_add1 (vm->dispatch_buffer_trace_nodes, a->buffer_trace_node_index);
-    }
-
-  if (a->enable)
-    {
-      /* Clean up from previous run, if any */
-      vec_free (pm->file_name);
-      vec_free (pm->pcap_data);
-      memset (pm, 0, sizeof (*pm));
-
-      vec_validate_aligned (vnet_trace_dummy, 2048, CLIB_CACHE_LINE_BYTES);
-      if (pm->lock == 0)
-       clib_spinlock_init (&(pm->lock));
-
-      if (a->filename == 0)
-       a->filename = format (0, "/tmp/dispatch.pcap%c", 0);
-
-      pm->file_name = (char *) a->filename;
-      pm->n_packets_captured = 0;
-      pm->packet_type = PCAP_PACKET_TYPE_vpp;
-      pm->n_packets_to_capture = a->packets_to_capture;
-      /* *INDENT-OFF* */
-      foreach_vlib_main (({this_vlib_main->dispatch_pcap_enable = 1;}));
-      /* *INDENT-ON* */
-    }
-  else
-    {
-      /* *INDENT-OFF* */
-      foreach_vlib_main (({this_vlib_main->dispatch_pcap_enable = 0;}));
-      /* *INDENT-ON* */
-      vec_reset_length (vm->dispatch_buffer_trace_nodes);
-      if (pm->n_packets_captured)
-       {
-         clib_error_t *error;
-         pm->n_packets_to_capture = pm->n_packets_captured;
-         vlib_cli_output (vm, "Write %d packets to %s, and stop capture...",
-                          pm->n_packets_captured, pm->file_name);
-         error = pcap_write (pm);
-         if (pm->flags & PCAP_MAIN_INIT_DONE)
-           pcap_close (pm);
-         /* Report I/O errors... */
-         if (error)
-           {
-             clib_error_report (error);
-             return -11;       /* VNET_API_ERROR_SYSCALL_ERROR_1 */
-           }
-         return 0;
-       }
-      else
-       return -6;              /* VNET_API_ERROR_NO_SUCH_ENTRY */
-    }
-
-  return 0;
+  return vlib_get_main ();
 }
 
-static clib_error_t *
-dispatch_trace_command_fn (vlib_main_t * vm,
-                          unformat_input_t * input, vlib_cli_command_t * cmd)
+elog_main_t *
+vlib_get_elog_main_not_inline ()
 {
-  unformat_input_t _line_input, *line_input = &_line_input;
-  vlib_pcap_dispatch_trace_args_t _a, *a = &_a;
-  u8 *filename = 0;
-  u32 max = 1000;
-  int rv;
-  int enable = 0;
-  int status = 0;
-  u32 node_index = ~0, buffer_traces_to_capture = 100;
-
-  /* Get a line of input. */
-  if (!unformat_user (input, unformat_line_input, line_input))
-    return 0;
-
-  while (unformat_check_input (line_input) != UNFORMAT_END_OF_INPUT)
-    {
-      if (unformat (line_input, "on %=", &enable, 1))
-       ;
-      else if (unformat (line_input, "enable %=", &enable, 1))
-       ;
-      else if (unformat (line_input, "off %=", &enable, 0))
-       ;
-      else if (unformat (line_input, "disable %=", &enable, 0))
-       ;
-      else if (unformat (line_input, "max %d", &max))
-       ;
-      else if (unformat (line_input, "packets-to-capture %d", &max))
-       ;
-      else if (unformat (line_input, "file %U", unformat_vlib_tmpfile,
-                        &filename))
-       ;
-      else if (unformat (line_input, "status %=", &status, 1))
-       ;
-      else if (unformat (line_input, "buffer-trace %U %d",
-                        unformat_vlib_node, vm, &node_index,
-                        &buffer_traces_to_capture))
-       ;
-      else
-       {
-         return clib_error_return (0, "unknown input `%U'",
-                                   format_unformat_error, line_input);
-       }
-    }
-
-  unformat_free (line_input);
-
-  /* no need for memset (a, 0, sizeof (*a)), set all fields here. */
-  a->filename = filename;
-  a->enable = enable;
-  a->status = status;
-  a->packets_to_capture = max;
-  a->buffer_trace_node_index = node_index;
-  a->buffer_traces_to_capture = buffer_traces_to_capture;
-
-  rv = vlib_pcap_dispatch_trace_configure (a);
-
-  switch (rv)
-    {
-    case 0:
-      break;
-
-    case -7:
-      return clib_error_return (0, "dispatch trace already enabled...");
-
-    case -81:
-      return clib_error_return (0, "dispatch trace already disabled...");
-
-    case -8:
-      return clib_error_return
-       (0, "can't change number of records to capture while tracing...");
-
-    case -11:
-      return clib_error_return (0, "I/O writing trace capture...");
-
-    case -6:
-      return clib_error_return (0, "No packets captured...");
-
-    default:
-      vlib_cli_output (vm, "WARNING: trace configure returned %d", rv);
-      break;
-    }
-  return 0;
+  return &vlib_global_main.elog_main;
 }
 
-/*?
- * This command is used to start or stop pcap dispatch trace capture, or show
- * the capture status.
- *
- * This command has the following optional parameters:
- *
- * - <b>on|off</b> - Used to start or stop capture.
- *
- * - <b>max <nn></b> - Depth of local buffer. Once '<em>nn</em>' number
- *   of packets have been received, buffer is flushed to file. Once another
- *   '<em>nn</em>' number of packets have been received, buffer is flushed
- *   to file, overwriting previous write. If not entered, value defaults
- *   to 100. Can only be updated if packet capture is off.
- *
- * - <b>file <name></b> - Used to specify the output filename. The file will
- *   be placed in the '<em>/tmp</em>' directory, so only the filename is
- *   supported. Directory should not be entered. If file already exists, file
- *   will be overwritten. If no filename is provided, '<em>/tmp/vpe.pcap</em>'
- *   will be used. Can only be updated if packet capture is off.
- *
- * - <b>status</b> - Displays the current status and configured attributes
- *   associated with a packet capture. If packet capture is in progress,
- *   '<em>status</em>' also will return the number of packets currently in
- *   the local buffer. All additional attributes entered on command line
- *   with '<em>status</em>' will be ignored and not applied.
- *
- * @cliexpar
- * Example of how to display the status of capture when off:
- * @cliexstart{pcap dispatch trace status}
- * max is 100, for any interface to file /tmp/vpe.pcap
- * pcap dispatch capture is off...
- * @cliexend
- * Example of how to start a dispatch trace capture:
- * @cliexstart{pcap dispatch trace on max 35 file dispatchTrace.pcap}
- * pcap dispatch capture on...
- * @cliexend
- * Example of how to start a dispatch trace capture with buffer tracing
- * @cliexstart{pcap dispatch trace on max 10000 file dispatchTrace.pcap buffer-trace dpdk-input 1000}
- * pcap dispatch capture on...
- * @cliexend
- * Example of how to display the status of a tx packet capture in progress:
- * @cliexstart{pcap tx trace status}
- * max is 35, dispatch trace to file /tmp/vppTest.pcap
- * pcap tx capture is on: 20 of 35 pkts...
- * @cliexend
- * Example of how to stop a tx packet capture:
- * @cliexstart{vppctl pcap dispatch trace off}
- * captured 21 pkts...
- * saved to /tmp/dispatchTrace.pcap...
- * @cliexend
-?*/
-/* *INDENT-OFF* */
-VLIB_CLI_COMMAND (pcap_dispatch_trace_command, static) = {
-    .path = "pcap dispatch trace",
-    .short_help =
-    "pcap dispatch trace [on|off] [max <nn>] [file <name>] [status]\n"
-    "              [buffer-trace <input-node-name> <nn>]",
-    .function = dispatch_trace_command_fn,
-};
-/* *INDENT-ON* */
-
 /*
  * fd.io coding-style-patch-verification: ON
  *