[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