> +/**
> + * Iterate over each lcore id's value for an lcore variable.
> + *
> + * @param lcore_id
> + * An <code>unsigned int</code> variable successively set to the
> + * lcore id of every valid lcore id (up to @c RTE_MAX_LCORE).
> + * @param value
> + * A pointer variable successively set to point to lcore variable
> + * value instance of the current lcore id being processed.
> + * @param handle
> + * The lcore variable handle.
> + */
> +#define RTE_LCORE_VAR_FOREACH_VALUE(lcore_id, value, handle) \
> + for (lcore_id = (((value) = RTE_LCORE_VAR_LCORE_VALUE(0, handle)), 0); \
> + lcore_id < RTE_MAX_LCORE; \
> + lcore_id++, (value) = RTE_LCORE_VAR_LCORE_VALUE(lcore_id, handle))
> +
I think we need a '()' around references to lcore_id:
for ((lcore_id) = (((value) = RTE_LCORE_VAR_LCORE_VALUE(0, handle)), 0); \
(lcore_id) < RTE_MAX_LCORE;
\
(lcore_id)++, (value) = RTE_LCORE_VAR_LCORE_VALUE(lcore_id,
handle))
