Code Review / vpp.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | review | tree
raw | inline | side by side
session: API cleanup
[vpp.git] / src / vcl / vcl_cfg.c
diff --git a/src/vcl/vcl_cfg.c b/src/vcl/vcl_cfg.c
index 12a4fd8..3be6de0 100644 (file)
--- a/src/vcl/vcl_cfg.c
+++ b/src/vcl/vcl_cfg.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2018 Cisco and/or its affiliates.
+ * Copyright (c) 2018-2019 Cisco and/or its affiliates.
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this
  * You may obtain a copy of the License at:
@@ -228,7 +228,7 @@ vppcom_cfg_read_file (char *conf_fname)
   int fd;
   unformat_input_t _input, *input = &_input;
   unformat_input_t _line_input, *line_input = &_line_input;
-  u8 vc_cfg_input = 0, *chroot_path;
+  u8 vc_cfg_input = 0;
   struct stat s;
   u32 uid, gid, q_len;
 
@@ -282,19 +282,12 @@ vppcom_cfg_read_file (char *conf_fname)
              VCFG_DBG (0, "VCL<%d>: configured max-workers %u", getpid (),
                        vcl_cfg->max_workers);
            }
-         else if (unformat (line_input, "api-prefix %s", &chroot_path))
+         else if (unformat (line_input, "api-prefix %s",
+                            &vcl_cfg->vpp_api_chroot))
            {
-             vec_terminate_c_string (chroot_path);
-             if (vcl_cfg->vpp_api_filename)
-               vec_free (vcl_cfg->vpp_api_filename);
-             vcl_cfg->vpp_api_filename = format (0, "/%s-vpe-api%c",
-                                                 chroot_path, 0);
-             vl_set_memory_root_path ((char *) chroot_path);
-
-             VCFG_DBG (0, "VCL<%d>: configured api-prefix (%s) and api "
-                       "filename (%s)", getpid (), chroot_path,
-                       vcl_cfg->vpp_api_filename);
-             chroot_path = 0;  /* Don't vec_free() it! */
+             vec_terminate_c_string (vcl_cfg->vpp_api_chroot);
+             VCFG_DBG (0, "VCL<%d>: configured api-prefix (%s) ", getpid (),
+                       vcl_cfg->vpp_api_chroot);
            }
          else if (unformat (line_input, "api-socket-name %s",
                             &vcl_cfg->vpp_api_socket_name))
@@ -343,84 +336,84 @@ vppcom_cfg_read_file (char *conf_fname)
                        getpid (), vcl_cfg->segment_size,
                        vcl_cfg->segment_size);
            }
-         else if (unformat (line_input, "segment-size %d",
+         else if (unformat (line_input, "segment-size %u",
                             &vcl_cfg->segment_size))
            {
-             VCFG_DBG (0, "VCL<%d>: configured segment_size %d (0x%x)",
+             VCFG_DBG (0, "VCL<%d>: configured segment_size %u (0x%x)",
                        getpid (), vcl_cfg->segment_size,
                        vcl_cfg->segment_size);
            }
          else if (unformat (line_input, "add-segment-size 0x%x",
                             &vcl_cfg->add_segment_size))
            {
-             VCFG_DBG (0, "VCL<%d>: configured add_segment_size 0x%x (%d)",
+             VCFG_DBG (0, "VCL<%d>: configured add_segment_size 0x%x (%u)",
                        getpid (), vcl_cfg->add_segment_size,
                        vcl_cfg->add_segment_size);
            }
-         else if (unformat (line_input, "add-segment-size %d",
+         else if (unformat (line_input, "add-segment-size %u",
                             &vcl_cfg->add_segment_size))
            {
-             VCFG_DBG (0, "VCL<%d>: configured add_segment_size %d (0x%x)",
+             VCFG_DBG (0, "VCL<%d>: configured add_segment_size %u (0x%x)",
                        getpid (), vcl_cfg->add_segment_size,
                        vcl_cfg->add_segment_size);
            }
-         else if (unformat (line_input, "preallocated-fifo-pairs %d",
+         else if (unformat (line_input, "preallocated-fifo-pairs %u",
                             &vcl_cfg->preallocated_fifo_pairs))
            {
-             VCFG_DBG (0, "VCL<%d>: configured preallocated_fifo_pairs %d "
+             VCFG_DBG (0, "VCL<%d>: configured preallocated_fifo_pairs %u "
                        "(0x%x)", getpid (), vcl_cfg->preallocated_fifo_pairs,
                        vcl_cfg->preallocated_fifo_pairs);
            }
-         else if (unformat (line_input, "rx-fifo-size 0x%lx",
+         else if (unformat (line_input, "rx-fifo-size 0x%x",
                             &vcl_cfg->rx_fifo_size))
            {
-             VCFG_DBG (0, "VCL<%d>: configured rx_fifo_size 0x%x (%d)",
+             VCFG_DBG (0, "VCL<%d>: configured rx_fifo_size 0x%x (%u)",
                        getpid (), vcl_cfg->rx_fifo_size,
                        vcl_cfg->rx_fifo_size);
            }
-         else if (unformat (line_input, "rx-fifo-size %d",
+         else if (unformat (line_input, "rx-fifo-size %u",
                             &vcl_cfg->rx_fifo_size))
            {
-             VCFG_DBG (0, "VCL<%d>: configured rx_fifo_size %d (0x%x)",
+             VCFG_DBG (0, "VCL<%d>: configured rx_fifo_size %u (0x%x)",
                        getpid (), vcl_cfg->rx_fifo_size,
                        vcl_cfg->rx_fifo_size);
            }
-         else if (unformat (line_input, "tx-fifo-size 0x%lx",
+         else if (unformat (line_input, "tx-fifo-size 0x%x",
                             &vcl_cfg->tx_fifo_size))
            {
-             VCFG_DBG (0, "VCL<%d>: configured tx_fifo_size 0x%x (%d)",
+             VCFG_DBG (0, "VCL<%d>: configured tx_fifo_size 0x%x (%u)",
                        getpid (), vcl_cfg->tx_fifo_size,
                        vcl_cfg->tx_fifo_size);
            }
-         else if (unformat (line_input, "tx-fifo-size %ld",
+         else if (unformat (line_input, "tx-fifo-size %u",
                             &vcl_cfg->tx_fifo_size))
            {
-             VCFG_DBG (0, "VCL<%d>: configured tx_fifo_size %d (0x%x)",
+             VCFG_DBG (0, "VCL<%d>: configured tx_fifo_size %u (0x%x)",
                        getpid (), vcl_cfg->tx_fifo_size,
                        vcl_cfg->tx_fifo_size);
            }
-         else if (unformat (line_input, "event-queue-size 0x%lx",
+         else if (unformat (line_input, "event-queue-size 0x%x",
                             &vcl_cfg->event_queue_size))
            {
-             VCFG_DBG (0, "VCL<%d>: configured event_queue_size 0x%x (%d)",
+             VCFG_DBG (0, "VCL<%d>: configured event_queue_size 0x%x (%u)",
                        getpid (), vcl_cfg->event_queue_size,
                        vcl_cfg->event_queue_size);
            }
-         else if (unformat (line_input, "event-queue-size %ld",
+         else if (unformat (line_input, "event-queue-size %u",
                             &vcl_cfg->event_queue_size))
            {
-             VCFG_DBG (0, "VCL<%d>: configured event_queue_size %d (0x%x)",
+             VCFG_DBG (0, "VCL<%d>: configured event_queue_size %u (0x%x)",
                        getpid (), vcl_cfg->event_queue_size,
                        vcl_cfg->event_queue_size);
            }
-         else if (unformat (line_input, "listen-queue-size 0x%lx",
+         else if (unformat (line_input, "listen-queue-size 0x%x",
                             &vcl_cfg->listen_queue_size))
            {
              VCFG_DBG (0, "VCL<%d>: configured listen_queue_size 0x%x (%u)",
                        getpid (), vcl_cfg->listen_queue_size,
                        vcl_cfg->listen_queue_size);
            }
-         else if (unformat (line_input, "listen-queue-size %ld",
+         else if (unformat (line_input, "listen-queue-size %u",
                             &vcl_cfg->listen_queue_size))
            {
              VCFG_DBG (0, "VCL<%d>: configured listen_queue_size %u (0x%x)",
@@ -472,9 +465,8 @@ vppcom_cfg_read_file (char *conf_fname)
          else if (unformat (line_input, "namespace-secret %lu",
                             &vcl_cfg->namespace_secret))
            {
-             VCFG_DBG (0,
-                       "VCL<%d>: configured namespace_secret %llu (0x%llx)",
-                       getpid (),
+             VCFG_DBG (0, "VCL<%d>: configured namespace_secret %llu "
+                       "(0x%llx)", getpid (),
                        (unsigned long long) vcl_cfg->namespace_secret,
                        (unsigned long long) vcl_cfg->namespace_secret);
            }
@@ -500,6 +492,12 @@ vppcom_cfg_read_file (char *conf_fname)
              VCFG_DBG (0, "VCL<%d>: configured with mq with eventfd",
                        getpid ());
            }
+         else if (unformat (line_input, "tls-engine %u",
+                            &vcl_cfg->tls_engine))
+           {
+             VCFG_DBG (0, "VCL<%d>: configured tls-engine %u (0x%x)",
+                       getpid (), vcl_cfg->tls_engine, vcl_cfg->tls_engine);
+           }
          else if (unformat (line_input, "}"))
            {
              vc_cfg_input = 0;
@@ -558,17 +556,15 @@ vppcom_cfg (vppcom_cfg_t * vcl_cfg)
   vppcom_cfg_heapsize (conf_fname);
   vppcom_cfg_read_file (conf_fname);
 
+  /* Regrab cfg after heap initialization */
+  vcl_cfg = &vcm->cfg;
   env_var_str = getenv (VPPCOM_ENV_API_PREFIX);
   if (env_var_str)
     {
-      if (vcl_cfg->vpp_api_filename)
-       vec_free (vcl_cfg->vpp_api_filename);
-      vcl_cfg->vpp_api_filename = format (0, "/%s-vpe-api%c", env_var_str, 0);
-      vl_set_memory_root_path ((char *) env_var_str);
-
-      VCFG_DBG (0, "VCL<%d>: configured api prefix (%s) and filename (%s) "
-               "from " VPPCOM_ENV_API_PREFIX "!", getpid (), env_var_str,
-               vcl_cfg->vpp_api_filename);
+      vcl_cfg->vpp_api_chroot = format (0, "%s", env_var_str);
+      vec_terminate_c_string (vcl_cfg->vpp_api_chroot);
+      VCFG_DBG (0, "VCL<%d>: configured api prefix (%s) from "
+               VPPCOM_ENV_API_PREFIX "!", getpid (), env_var_str);
     }
   env_var_str = getenv (VPPCOM_ENV_APP_NAMESPACE_ID);
   if (env_var_str)
@@ -580,8 +576,7 @@ vppcom_cfg (vppcom_cfg_t * vcl_cfg)
       clib_memcpy (vcm->cfg.namespace_id, env_var_str, ns_id_vec_len);
 
       VCFG_DBG (0, "VCL<%d>: configured namespace_id (%s) from "
-               VPPCOM_ENV_APP_NAMESPACE_ID "!", getpid (),
-               (char *) vcm->cfg.namespace_id);
+               VPPCOM_ENV_APP_NAMESPACE_ID "!", getpid (), env_var_str);
     }
   env_var_str = getenv (VPPCOM_ENV_APP_NAMESPACE_SECRET);
   if (env_var_str)
Vector Packet Processing