On Tue, Oct 03, 2023 at 11:33:04AM +0300, Roi Dayan via dev wrote:
> The cited commit fixed missing mirror packets by reset mirror when
> packets are modified but setting geneve options was also treated as
> a modified packet but should be treated as a part of set_tunnel
> which doesn't reset mirror.
>
> Fixes: 1437f51fea92 ("ofproto-dpif-upcall: Mirror packets that are modified.")
Hi Roi,
I don't think that hash appears in the master branch.
Perhaps the tag should be:
Fixes: feed7f677505 ("ofproto-dpif-upcall: Mirror packets that are modified.")
> Signed-off-by: Roi Dayan <[email protected]>
The above notwithstanding, this looks good to me.
Acked-by: Simon Horman <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
