> On Wed, 2021-05-19 at 17:35 +0200, Lorenzo Bianconi wrote:
> > In the current codebase, check_pkt_len is applied just for traffic
> > leaving the
> > ovn cluster. This series introduces the same capability for traffic
> > entering
> > the network from a gateway router or distributed gateway router port
> > in order
> > to send an ICMP error packet if the frame size is greater than the
> > configured
> > MTU.
> > 
> > Lorenzo Bianconi (3):
> >   northd: introduce build_check_pkt_len_flows_for_lrp routine
> >   northd: enable check_pkt_len for gw router
> >   northd: add chec_pkt_len lflows for ingress traffic
> 
> I know I said "check_pkt_len" a lot when talking about it internally,
> but the commit messages should probably reference check_pkt_larger()
> right?

yes, correct. I mixed the ovn action and the kernel one, I will fix it in v2.
Thanks.

> 
> Also small nit, commit message for last patch is missing a "k" in
> check.

ack, I will fix it in v2.

Regards,
Lorenzo

> 
> Thanks,
> Dan
> 
> > 
> >  northd/ovn-northd.c | 236 +++++++++++----------
> >  tests/ovn-northd.at | 150 +++++++-------
> >  tests/ovn.at        | 492
> > ++++++++++++++++++++++++++++++++++++++++++--
> >  3 files changed, 682 insertions(+), 196 deletions(-)
> > 
> 
> 
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to