On 4 Oct 2023, at 12:09, 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: feed7f677505 ("ofproto-dpif-upcall: Mirror packets that are modified.")
> Signed-off-by: Roi Dayan <[email protected]>
> Acked-by: Simon Horman <[email protected]>
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
- [ovs-dev] [PATCH v2 0/3] Fix redundant mirror with tunne... Roi Dayan via dev
- [ovs-dev] [PATCH v2 1/3] ofproto-dpif-upcall: Fix r... Roi Dayan via dev
- Re: [ovs-dev] [PATCH v2 1/3] ofproto-dpif-upcal... Eelco Chaudron
- [ovs-dev] [PATCH v2 2/3] tests/tunnel.at: Add genev... Roi Dayan via dev
- Re: [ovs-dev] [PATCH v2 2/3] tests/tunnel.at: A... Simon Horman
- Re: [ovs-dev] [PATCH v2 2/3] tests/tunnel.at: A... Eelco Chaudron
- Re: [ovs-dev] [PATCH v2 2/3] tests/tunnel.a... Roi Dayan via dev
- Re: [ovs-dev] [PATCH v2 2/3] tests/tunn... Eelco Chaudron
- Re: [ovs-dev] [PATCH v2 2/3] tests... Roi Dayan via dev
- Re: [ovs-dev] [PATCH v2 2/3] t... Eelco Chaudron
- [ovs-dev] [PATCH v2 3/3] tests: Update some tests t... Roi Dayan via dev
- Re: [ovs-dev] [PATCH v2 3/3] tests: Update some... Simon Horman
- Re: [ovs-dev] [PATCH v2 3/3] tests: Update some... Eelco Chaudron
