On 27 May 2026, at 0:11, Ilya Maximets wrote:
> Current code tracks is_last_action through many translation paths, but
> not through the NORMAL pipeline. This results in unnecessary clone()
> actions emitted in the end of the pipeline. For example, this is
> happening for a common case of hash() + recirc() after a tunnel push.
> Can be avoided by propagating the flag along the way.
>
> Mirrors are left as-is with the last action always treated as 'false'
> for now. The benefits for mirrors are not clear, as normally the
> output for them is a simple port and not something like a patch port
> that can recurse and produce all sorts of actions. Updating the code
> for mirrors to track the last action is also a bit cumbersome given
> the mirrors bitmask and the potential multiple outputs per mirror.
>
> Port addition in the patch port test is split into multiple
> transactions to ensure that flood order doesn't depend on the order
> of ports within the database update.
Thanks Ilya, this patch looks good to me.
Acked-by: Eelco Chaudron <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
