On 8/18/25 7:28 PM, Yury Norov wrote:
> From: "Yury Norov (NVIDIA)" <yury.no...@gmail.com>
>
> Due to legacy reasons, openswitch code opencodes for_each_cpu() to make
> sure that CPU0 is always considered.
>
> Since commit c4b2bf6b4a35 ("openvswitch: Optimize operations for OvS
> flow_stats."), the corresponding flow->cpu_used_mask is initialized
> such that CPU0 is explicitly set.
>
> So, switch the code to using plain for_each_cpu().
>
> Suggested-by: Ilya Maximets <i.maxim...@ovn.org>
> Signed-off-by: Yury Norov (NVIDIA) <yury.no...@gmail.com>
> ---
> net/openvswitch/flow.c | 12 ++++--------
> net/openvswitch/flow_table.c | 7 +++----
> 2 files changed, 7 insertions(+), 12 deletions(-)
>
> v1: https://lore.kernel.org/all/20250814195838.388693-1-yury.no...@gmail.com/
> v2:
> - always include CPU0 (Ilya);
nit: The subject prefix should've been [PATCH net-next v2].
The change itself looks correct to me. Thanks!
Acked-by: Ilya Maximets <i.maxim...@ovn.org>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
