[PATCH v4 04/14] net/sxe: add link, flow ctrl, mac ops, mtu ops function

Stephen Hemminger stephen at networkplumber.org
Wed Jul 9 18:09:30 CEST 2025


On Wed,  9 Jul 2025 04:43:04 -0400
Jie Liu <liujie5 at linkdatatechnology.com> wrote:

> +	RTE_ATOMIC(bool)link_thread_running;
> +	RTE_ATOMIC(bool)is_stopping;

Despite what checkpatch says, need space here:
	RTE_ATOMIC(bool) is_stopping;

> +	rte_thread_t link_thread_tid;
> +	rte_atomic32_t link_thread_running;

Don't use old rte atomics, instead use RTE_ATOMIC() consistently

> +	rte_atomic32_t is_stopping;
> +	pthread_t link_thread_tid;

Two struct elements with same name, how does that work?

> +	bool is_stopped;


More information about the dev mailing list