On Wed, Jun 29, 2016 at 01:17:08AM -0700, Gurucharan Shetty wrote:
> Currently, the only use of stateful services in conntrack is
> OVN ACLs. In table ACL, we commit the packet to conntrack
> via ct_commit action.
> 
> As we introduce more stateful services, the ACL feature will
> have to share the conntrack module with others. As
> preparation for more stateful features like load balancing,
> this commit introduces a new stateful table
> that is responsible to commit packets to conntrack via
> ct_commit action. If ACL table needs to commit a packet,
> it sets 'reg0[1]' as 1. Stateful table in-turn will commit
> the packet if 'reg0[1]' is 1.
> 
> Signed-off-by: Gurucharan Shetty <g...@ovn.org>

s/initiater's/initiator's/ in the documentation.

The code introduces symbolic names, or at least macros, for reg0[0] and
reg0[1].  Do you think that the documentation would be improved by
giving these bits names for reference?  I do not know for sure, but
maybe it would make the documentation easier to understand.

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

Reply via email to