On Thu, 17 Sept 2026 at 22:12, Stephen Hemminger <[email protected]> wrote: > diff --git a/drivers/net/enic/enic_main.c b/drivers/net/enic/enic_main.c > index 2696fa77d4..fb9a5754c9 100644 > --- a/drivers/net/enic/enic_main.c > +++ b/drivers/net/enic/enic_main.c > @@ -83,17 +83,15 @@ static void enic_log_q_error(struct enic *enic) > static void enic_clear_soft_stats(struct enic *enic) > { > struct enic_soft_stats *soft_stats = &enic->soft_stats; > - rte_atomic64_clear(&soft_stats->rx_nombuf); > - rte_atomic64_clear(&soft_stats->rx_packet_errors); > - rte_atomic64_clear(&soft_stats->tx_oversized); > + > + memset(soft_stats, 0, sizeof(*soft_stats)); > } > > static void enic_init_soft_stats(struct enic *enic) > { > struct enic_soft_stats *soft_stats = &enic->soft_stats; > - rte_atomic64_init(&soft_stats->rx_nombuf); > - rte_atomic64_init(&soft_stats->rx_packet_errors); > - rte_atomic64_init(&soft_stats->tx_oversized); > + > + memset(soft_stats, 0, sizeof(*soft_stats)); > enic_clear_soft_stats(enic); > } >
enic_init_soft_stats calls enic_clear_soft_stats, so a memset here is unncessary. -- David Marchand
