On 5/18/23 12:56, Ihar Hrachyshka wrote:
> The new tables will be used in a later patch as follows:
> 
> table=37, OFTABLE_OUTPUT_INIT: becomes an initial entry point into the
> egress pipeline that serves a semantic goal. (Not doing any actual
> processing at the moment.)
> 
> table=38, OFTABLE_OUTPUT_LARGE_PKT_DETECT: detect "too-big" IP packets
> and mark them for later processing in table=39.
> 
> table=39, OFTABLE_OUTPUT_LARGE_PKT_PROCESS: process "too-big" IP packets
> detected in table=38 by sending ICMPv4 Fragmentation Needed / ICMPv6 Too
> Big errors back to the originating port.
> 
> All previous table indices shifted by 3 (old table=37 becomes table=40).
> Otherwise, no changes to existing tables and flows introduced.
> 
> Acked-by: Dumitru Ceara <dce...@redhat.com>
> Signed-off-by: Ihar Hrachyshka <ihrac...@redhat.com>
> ---
>  controller/lflow.c       |   4 +-
>  controller/lflow.h       |  49 ++++---
>  controller/physical.c    |  77 +++++++----
>  controller/pinctrl.c     |   8 +-
>  ovn-architecture.7.xml   |  71 +++++-----
>  tests/ovn-controller.at  | 174 ++++++++++++------------

This patch needs a few minor changes to tests/ovn-controller.at now that
46dacbb755dd ("northd: Break ACLs into two stages.") went in.

With those taken care of, my ack stands.

Regards,
Dumitru

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to