eal: remove unused config fields
authorDavid Marchand <david.marchand@6wind.com>
Fri, 18 Apr 2014 12:58:08 +0000 (14:58 +0200)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Mon, 5 May 2014 16:02:56 +0000 (18:02 +0200)
There is no need for a 'magic' field in struct rte_config, as this part of the
structure is local to each process. All threads of a process are synchronised
because of the run_once atomic.
So remove this field, as it is only adding confusion when reading code that
references 'magic' field from struct rte_mem_config.

Besides, there is no reference about the 'version' field, so remove it as well.

Signed-off-by: David Marchand <david.marchand@6wind.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
lib/librte_eal/bsdapp/eal/eal.c
lib/librte_eal/common/include/rte_eal.h
lib/librte_eal/linuxapp/eal/eal.c

index 93bd50b..5fe2936 100644 (file)
@@ -280,8 +280,6 @@ eal_proc_type_detect(void)
 static void
 rte_config_init(void)
 {
-       /* set the magic in configuration structure */
-       rte_config.magic = RTE_MAGIC;
        rte_config.process_type = (internal_config.process_type == RTE_PROC_AUTO) ?
                        eal_proc_type_detect() : /* for auto, detect the type */
                        internal_config.process_type; /* otherwise use what's already set */
index 658b4f7..8fac05e 100644 (file)
@@ -71,10 +71,6 @@ enum rte_proc_type_t {
  * The global RTE configuration structure.
  */
 struct rte_config {
-       uint32_t version; /**< Configuration [structure] version. */
-       uint32_t magic;   /**< Magic number - Sanity check. */
-
-
        uint32_t master_lcore;       /**< Id of the master lcore */
        uint32_t lcore_count;        /**< Number of available logical cores. */
        enum rte_lcore_role_t lcore_role[RTE_MAX_LCORE]; /**< State of cores. */
index 39951fc..d7a59de 100644 (file)
@@ -289,8 +289,6 @@ eal_proc_type_detect(void)
 static void
 rte_config_init(void)
 {
-       /* set the magic in configuration structure */
-       rte_config.magic = RTE_MAGIC;
        rte_config.process_type = (internal_config.process_type == RTE_PROC_AUTO) ?
                        eal_proc_type_detect() : /* for auto, detect the type */
                        internal_config.process_type; /* otherwise use what's already set */