On 10/17/24 16:04, Mike Pattrick wrote:
> Previously a commit attempted to reset the mirror context when packets
> were modified. However, this commit erroneously also reset the mirror
> context when only a packet's metadata was modified. An intermediate
> commit corrected this for tunnel metadata, but now that correction is
> extended to other forms of metadata as well.
>
> Fixes: feed7f677505 ("ofproto-dpif-upcall: Mirror packets that are modified.")
> Reported-at: https://issues.redhat.com/browse/FDP-699
> Acked-by: Eelco Chaudron <[email protected]>
> Acked-by: Simon Horman <[email protected]>
> Signed-off-by: Mike Pattrick <[email protected]>
> ---
> v2:
> - Added extra whitespace
> - Moved N_IDS to never reached section
> - Added unit test
> v3:
> - Fixed broken unit test.
> v4:
> - Changed unit test to use m4_define and changed comment
> - Reordered switch statement
> ---
Thanks, Mike, Eelco and Simon!
Applied and backported down to 3.1.
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
