On 9 September 2016 at 13:59, Ben Pfaff <b...@ovn.org> wrote:

> On Tue, Aug 23, 2016 at 02:46:18AM -0700, Gurucharan Shetty wrote:
> > Load-balancers in gateway routers lets us load-balance
> > north-south traffic.
> >
> > This commit adds a new table called "DEFRAG" in the
> > logical router pipeline to defragment packets and to track them.
> > Once the packet is tracked, new connections get a group id as
> > an action. The group in turn chooses a DNAT action.  Established
> > connections go through the DNAT table for a regular DNAT.
> >
> > Signed-off-by: Gurucharan Shetty <g...@ovn.org>
>
> Thanks for working on this.
>
> I always thought that, for packets, the opposite of fragmentation was
> reassembly, but I hear the word "defragmentation", as used here, more
> and more these days.  I don't know whether it's considered an "official"
> term.
>
I think I picked it up from Justin's comments in build_acl() and probably
from team meetings. I am used it now!


>
> In the ovn-northd manpage, the DNAT section has a single bullet point
> with lots of paragraphs.  Probably, each paragraph should be a bullet
> point, because each one describes a category of flows.
>
I did that and pushed this. Thank you so much for the reviews!


>
> Acked-by: Ben Pfaff <b...@ovn.org>
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to