Re: [dpdk-dev] [PATCH v8] eal: make lcore_config private

[David Marchand <david.marchand@redhat.com>]

On Wed, Oct 2, 2019 at 9:40 PM Stephen Hemminger
<stephen@networkplumber.org> wrote:
> +struct lcore_config {
> +       pthread_t thread_id;       /**< pthread identifier */
> +       int pipe_master2slave[2];  /**< communication pipe with master */
> +       int pipe_slave2master[2];  /**< communication pipe with master */
> +
> +       lcore_function_t * volatile f; /**< function to call */
> +       void * volatile arg;       /**< argument of function */
> +       volatile int ret;          /**< return value of function */
> +
> +       uint32_t core_id;          /**< core number on socket for this lcore */
> +       uint32_t core_index;       /**< relative index, starting from 0 */
> +       uint16_t socket_id;        /**< physical socket id for this lcore */
> +       uint8_t core_role;         /**< role of core eg: OFF, RTE, SERVICE */
> +       uint8_t detected;          /**< true if lcore was detected */
> +       volatile enum rte_lcore_state_t state; /**< lcore state */
> +       rte_cpuset_t cpuset;       /**< cpu set which the lcore affinity to */
> +};

There are still changes on the core_id, core_index, socket_id that I
am not confortable with (at this point).

I prepared a series for -rc1 on ABI changes in EAL (that I will send shortly).
I took your patch without the changes on core_id, core_index and socket_id.


We can discuss those changes later, thanks.

-- 
David Marchand