On 7/15/25 3:12 PM, Eelco Chaudron wrote:
> When a meter action is encountered and stored in the auxiliary
> structure, and subsequently, a non-meter action is processed
> within a nested list during callback execution, an infinite
> loop is triggered.
>
> This patch maintains the current behavior but stores all
> required meter actions in an ofpbuf for deferred execution.
>
> Reportde-at:
> https://patchwork.ozlabs.org/project/openvswitch/patch/20250506022337.3242-1-danieldin...@gmail.com/
* Reported-at
> Fixes: 076caa2fb077 ("ofproto: Meter translation.")
> Signed-off-by: Eelco Chaudron <echau...@redhat.com>
>
> ---
> v3: - Fix some review nits.
> v2: - Removed redundant extra newlines.
> - Update comment to reflect reality
> - Use the add_of_ports MACRO
> - Use 'ovs-ofctl compose-packet --bare' for packet construction
> - Fix some grep commands.
> ---
> lib/dpif.c | 65 +++++++++++++++++--------------------------
> tests/ofproto-dpif.at | 36 ++++++++++++++++++++++++
> 2 files changed, 61 insertions(+), 40 deletions(-)
Beside the typo in the tag, this version looks good 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
