On Thu, May 19, 2016 at 01:02:33PM -0700, Gurucharan Shetty wrote: > OVS NAT currently cannot do snat and dnat in the same zone. > So we need two zones per gateway router. > > Signed-off-by: Gurucharan Shetty <g...@ovn.org>
We're running out of registers quickly, but we're also using a full 32-bit register when we only need 16 bits, so there's considerable room to economize later if necessary. Please update ovn-architecture(7) to mention the new connection tracking zones (the existing one is already mentioned). There are a couple of instances of + char *dnat = xasprintf(UUID_FMT"_%s", + UUID_ARGS(&binding->datapath->header_.uuid), + "dnat"); + char *snat = xasprintf(UUID_FMT"_%s", + UUID_ARGS(&binding->datapath->header_.uuid), + "snat"); or similar. Do you think it would be worth having a helper function (or two) so that they are harder to get out-of-sync? Acked-by: Ben Pfaff <b...@ovn.org> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
