On 02/02/2026 16:07, Eelco Chaudron wrote:
> The dpif-netdev code used a static variable 'first_set_config' to
> perform one-time initialization when dpif_netdev_set_config() was first
> called. Because this variable was global rather than per-datapath, the
> initialization would not be re-triggered if a datapath was destroyed and
> recreated.
>
> Replace the static variable with a per-datapath ovsthread_once
> structure, ensuring that each dp_netdev instance properly performs its
> one-time initialization regardless of other datapaths' lifecycle.
>
> Fixes: de3bbdc479a9 ("dpif-netdev: Add PMD load based sleeping.")
> Fixes: 7bd1867b6d6a ("dpif-offload-dpdk: Abstract rte_flow implementation
> from dpif-netdev.")
> Signed-off-by: Eelco Chaudron <[email protected]>
> ---
> lib/dpif-netdev.c | 16 ++++++++++------
> 1 file changed, 10 insertions(+), 6 deletions(-)
>
Thanks Eelco. LGTM and i did a quick test and working correctly.
Acked-by: Kevin Traynor <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
