Rather than having a single function which attempts to reset multiple internal config structures, just init each structure in the function which configures it. Use structure assignment for initialization rather than a series of assignment statements.
Signed-off-by: Bruce Richardson <[email protected]> --- lib/eal/common/eal_common_options.c | 85 ++++++----------------------- lib/eal/common/eal_internal_cfg.h | 1 - lib/eal/freebsd/eal.c | 2 - lib/eal/linux/eal.c | 7 +-- 4 files changed, 21 insertions(+), 74 deletions(-) diff --git a/lib/eal/common/eal_common_options.c b/lib/eal/common/eal_common_options.c index bd08d29e1d..292ac7378e 100644 --- a/lib/eal/common/eal_common_options.c +++ b/lib/eal/common/eal_common_options.c @@ -466,72 +466,6 @@ eal_get_hugefile_prefix(void) return HUGEFILE_PREFIX_DEFAULT; } -void -eal_reset_internal_config(void) -{ - struct eal_user_cfg *user_cfg = eal_get_user_configuration(); - struct eal_platform_info *platform_info = eal_get_platform_info(); - struct eal_runtime_state *runtime_state = eal_get_runtime_state(); - int i; - - TAILQ_INIT(&user_cfg->devopt_list); - TAILQ_INIT(&user_cfg->plugin_list); - STAILQ_INIT(&user_cfg->trace_patterns); - TAILQ_INIT(&runtime_state->loaded_plugins); - user_cfg->memory = 0; - user_cfg->force_nrank = 0; - user_cfg->force_nchannel = 0; - user_cfg->force_numa = false; - for (i = 0; i < RTE_MAX_NUMA_NODES; i++) - user_cfg->numa_mem[i] = 0; - user_cfg->force_numa_limits = false; - for (i = 0; i < RTE_MAX_NUMA_NODES; i++) - user_cfg->numa_limit[i] = 0; - user_cfg->process_type = RTE_PROC_PRIMARY; - user_cfg->no_hugetlbfs = false; - user_cfg->no_pci = false; - user_cfg->hugefile_prefix = NULL; - user_cfg->hugepage_dir = NULL; - user_cfg->hugepage_file.unlink_before_mapping = false; - user_cfg->hugepage_file.unlink_existing = true; - /* zero out hugedir descriptors */ - for (i = 0; i < MAX_HUGEPAGE_SIZES; i++) { - memset(&platform_info->hugepage_info[i], 0, - sizeof(platform_info->hugepage_info[0])); - platform_info->hugepage_info[i].lock_descriptor = -1; - } - user_cfg->base_virtaddr = 0; - - /* if set to NONE, interrupt mode is determined automatically */ - user_cfg->vfio_intr_mode = RTE_INTR_MODE_NONE; - memset(user_cfg->vfio_vf_token, 0, - sizeof(user_cfg->vfio_vf_token)); - - user_cfg->no_auto_probing = false; - -#ifdef RTE_LIBEAL_USE_HPET - user_cfg->no_hpet = false; -#else - user_cfg->no_hpet = true; -#endif - user_cfg->vmware_tsc_map = false; - user_cfg->no_shconf = false; - user_cfg->in_memory = false; - user_cfg->create_uio_dev = false; - user_cfg->no_telemetry = false; - user_cfg->iova_mode = RTE_IOVA_DC; - user_cfg->user_mbuf_pool_ops_name = NULL; - CPU_ZERO(&runtime_state->ctrl_cpuset); - runtime_state->init_complete = 0; - CPU_ZERO(&user_cfg->service_cpuset); - for (i = 0; i < RTE_MAX_LCORE; i++) { - free(user_cfg->lcore_cpusets[i]); - user_cfg->lcore_cpusets[i] = NULL; - } - user_cfg->max_simd_bitwidth.bitwidth = RTE_VECT_DEFAULT_SIMD_BITWIDTH; - user_cfg->max_simd_bitwidth.forced = 0; -} - static int eal_plugin_path_add(const char *path) { @@ -1860,6 +1794,24 @@ int eal_parse_args(void) { struct eal_user_cfg *user_cfg = eal_get_user_configuration(); + + /* + * Initialise user_cfg to defaults. Fields not listed here are zero, + * false or NULL, which is the correct default (RTE_PROC_PRIMARY, + * RTE_INTR_MODE_NONE, RTE_IOVA_DC, etc. are all defined as 0). + */ + *user_cfg = (struct eal_user_cfg){ + .devopt_list = TAILQ_HEAD_INITIALIZER(user_cfg->devopt_list), + .plugin_list = TAILQ_HEAD_INITIALIZER(user_cfg->plugin_list), + .trace_patterns = STAILQ_HEAD_INITIALIZER(user_cfg->trace_patterns), + .hugepage_file.unlink_existing = true, + .main_lcore = -1, +#ifndef RTE_LIBEAL_USE_HPET + .no_hpet = true, +#endif + .max_simd_bitwidth.bitwidth = RTE_VECT_DEFAULT_SIMD_BITWIDTH, + }; + bool remap_lcores = (args.remap_lcore_ids != NULL); struct arg_list_elem *arg; uint16_t lcore_id_base = 0; @@ -1986,7 +1938,6 @@ eal_parse_args(void) return -1; } } - user_cfg->main_lcore = -1; if (args.main_lcore != NULL && eal_parse_main_lcore(args.main_lcore) < 0) return -1; diff --git a/lib/eal/common/eal_internal_cfg.h b/lib/eal/common/eal_internal_cfg.h index 239fe2a7ac..979c1320e8 100644 --- a/lib/eal/common/eal_internal_cfg.h +++ b/lib/eal/common/eal_internal_cfg.h @@ -213,6 +213,5 @@ struct eal_runtime_state { struct eal_user_cfg *eal_get_user_configuration(void); struct eal_platform_info *eal_get_platform_info(void); struct eal_runtime_state *eal_get_runtime_state(void); -void eal_reset_internal_config(void); #endif /* EAL_INTERNAL_CFG_H */ diff --git a/lib/eal/freebsd/eal.c b/lib/eal/freebsd/eal.c index 120425d425..13bbd8b868 100644 --- a/lib/eal/freebsd/eal.c +++ b/lib/eal/freebsd/eal.c @@ -444,8 +444,6 @@ rte_eal_init(int argc, char **argv) goto err_out; } - eal_reset_internal_config(); - if (rte_eal_cpu_init() < 0) { rte_eal_init_alert("Cannot detect lcores."); rte_errno = ENOTSUP; diff --git a/lib/eal/linux/eal.c b/lib/eal/linux/eal.c index 3f2ad98425..328c74ae4d 100644 --- a/lib/eal/linux/eal.c +++ b/lib/eal/linux/eal.c @@ -410,8 +410,9 @@ eal_hugedirs_unlock(void) int i; for (i = 0; i < MAX_HUGEPAGE_SIZES; i++) { - /* skip uninitialized */ - if (platform_info->hugepage_info[i].lock_descriptor < 0) + /* skip uninitialized or unlocked entries */ + if (platform_info->hugepage_info[i].hugepage_sz == 0 || + platform_info->hugepage_info[i].lock_descriptor < 0) continue; /* unlock hugepage file */ flock(platform_info->hugepage_info[i].lock_descriptor, LOCK_UN); @@ -606,8 +607,6 @@ rte_eal_init(int argc, char **argv) goto err_out; } - eal_reset_internal_config(); - if (rte_eal_cpu_init() < 0) { rte_eal_init_alert("Cannot detect lcores."); rte_errno = ENOTSUP; -- 2.51.0
