On 1/30/2020 1:28 PM, Ilya Maximets wrote:
On 14.01.2020 14:21, Eli Britstein wrote:
l3_ofs should be set all Ethernet packets, not just IPv4/IPv6 ones.
For example for ARP over VLAN tagged packets, it may cause wrong
processing like in changing the VLAN ID action. Fix it.
Fixes: aab96ec4d81e ("dpif-netdev: retrieve flow directly from the flow mark")
Signed-off-by: Eli Britstein <el...@mellanox.com>
Reviewed-by: Roi Dayan <r...@mellanox.com>
---
Thanks, Eli. This seems correct to me. I'm traveling now, will be able
to check on next week if Ben will not get to this earlier.
BTW, it might be good to have a unit test for this. We already have
a couple of such partial offloading tests.
Could you please point me to where? Note that in order to hit this issue
we really need a supporting HW.
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
