On 15 Jul 2025, at 16:32, Ilya Maximets wrote:
> 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>
Thanks for the review Ilya! Applied to main and backported till 3.2.
//Eelco
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
