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 <[email protected]> > --- > 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 <[email protected]> Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
