[ovs-dev] [slow path 01/11] ofproto: Don't limit flows in OpenFlow tables by default.

Justin Pettit jpettit at nicira.com
Tue May 8 20:13:51 UTC 2012


Looks good.

--Justin


On May 5, 2012, at 11:10 AM, Ben Pfaff wrote:

> This bug is ordinarily not exposed because bridge_configure_tables() in
> bridge.c configures the max number of flows soon after an ofproto is
> created.  But an upcoming commit will make construct() in ofproto-dpif.c
> try to create some built-in flows before bridge gets control, so we need
> to allow creating flows immediately upon initialization.
> 
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
> ofproto/ofproto.c |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index 4b2cbc9..8e3c042 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -3874,6 +3874,7 @@ oftable_init(struct oftable *table)
> {
>     memset(table, 0, sizeof *table);
>     classifier_init(&table->cls);
> +    table->max_flows = UINT_MAX;
> }
> 
> /* Destroys 'table', including its classifier and eviction groups.
> -- 
> 1.7.2.5
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev




More information about the dev mailing list