On 2022-07-06 3:34 PM, Roi Dayan wrote:
On 2022-07-06 1:26 PM, Eelco Chaudron wrote:
On 5 Jul 2022, at 0:45, Ilya Maximets wrote:
TCA_FLOWER_KEY_ENC_UDP_DST_PORT_MASK is supported by TC, and OVS may
create a masked match on this field.
This change is important as we're not clearing the masks which wasn't
really used, so if OVS requests match on ports, we should use the
mask and clear, otherwise offloading will fail.
Signed-off-by: Ilya Maximets <[email protected]>
Changes look good to me, and all system-traffic.at tests that were
passing without the change are still passing, including the failing
ERSPAN ones.
Acked-by: Eelco Chaudron <[email protected]>
can we postpone to merge this?
After this patch simple vxlan rules won't have enc dst port on decap
rules and currently mlx5 driver must have enc dst port to do the
offload of the decap rule.
I'm checking about this limitation and if we can remove it but
if we can postpone a bit so we won't break users with our nics..
thanks,
Roi
I finished with the testing and checking I needed.
I don't have a problem with the patch. if something else will raise i'll
start a different discussion.
Acked-by: Roi Dayan <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev