[ovs-dev] [PATCH] ovs-ctl: Restore ability to create 256 bridges.

[Ethan Jackson]

Looks good, thanks.

Ethan

On Thu, Jan 26, 2012 at 15:58, Ben Pfaff <blp at nicira.com> wrote:
> OVS is supposed to support at least 256 bridges simultaneously, but the
> file descriptor count was a little too low for that.
>
> Bug #8218.
> Reported-by: Luca Giraudo <lgiraudo at nicira.com>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
>  utilities/ovs-ctl.in |    9 +++++----
>  1 files changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/utilities/ovs-ctl.in b/utilities/ovs-ctl.in
> index d1df6a8..9bdb3df 100755
> --- a/utilities/ovs-ctl.in
> +++ b/utilities/ovs-ctl.in
> @@ -1,5 +1,5 @@
>  #! /bin/sh
> -# Copyright (C) 2009, 2010, 2011 Nicira Networks, Inc.
> +# Copyright (C) 2009, 2010, 2011, 2012 Nicira Networks, Inc.
>  #
>  # Licensed under the Apache License, Version 2.0 (the "License");
>  # you may not use this file except in compliance with the License.
> @@ -203,9 +203,10 @@ start () {
>     if daemon_is_running ovs-vswitchd; then
>        log_success_msg "ovs-vswitchd is already running"
>     else
> -       # Increase the limit on the number of open file descriptors since
> -       # ovs-vswitchd needs a few per bridge
> -       ulimit -n 4096
> +       # Increase the limit on the number of open file descriptors.
> +       # ovs-vswitchd needs 16 per datapath, plus a few extra, so this
> +       # should allow for 256 (or more) bridges.
> +       ulimit -n 5000
>
>        # Start ovs-vswitchd.
>        set ovs-vswitchd unix:"$DB_SOCK"
> --
> 1.7.2.5
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev