On 2/12/24 07:00, Dumitru Ceara wrote:
On 2/6/24 10:15, Ales Musil wrote:
The fields that are not directly supported by OvS were encoded
via additional controller action that changed the required value.
This was most notably needed for ICMP need frag messages.
Encode the field value loads as note action instead. This allows
us to find the note and act accordingly in the first controller
action without the need to send it to pinctrl again, parse and
change the packet. This way we can also encode any future fields
that might be needed as this method should be flexible enough.
This change is completely transparent to the user and shouldn't
cause any disruptions.
Signed-off-by: Ales Musil <amu...@redhat.com>
---
v3: Rebase on top of current main.
Fix some cosmetic stuff pointed out by Dumitru.
Add back the handler for old way to keep the ICMP
error working during upgrade.
v2: Fix the wrong checksum for the ICMP packet.
---
Thanks for the update, Ales! Looks good to me:
Acked-by: Dumitru Ceara <dce...@redhat.com>
Regards,
Dumitru
Thanks Ales and Dumitru,
I pushed this change to main.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
