[ovs-dev] [PATCH] ovs-ctl.in: increase the limit of fd

Ben Pfaff blp at nicira.com
Thu Nov 15 03:06:25 UTC 2012


On Thu, Nov 15, 2012 at 10:31:04AM +0800, Cong Wang wrote:
> On Wed, Nov 14, 2012 at 11:43 PM, Ben Pfaff <blp at nicira.com> wrote:
> > On Wed, Nov 14, 2012 at 08:58:18PM +0800, Cong Wang wrote:
> >> We can't create more than 248 bridges with the current limit 5000,
> >> so increase it to 6000 so that at least 256+ bridges could be created.
> >>
> >> Cc: Ben Pfaff <blp at nicira.com>
> >> Signed-off-by: Cong Wang <xiyou.wangcong at gmail.com>
> >
> > Are you sure?  I would have guessed that even 1000 file descriptors was
> > enough on current master.
> 
> I am sure for 1.7.0, it is very easy to reproduce the problem.
> Run `for i in {1..256}; do ovs-vsctl add-br ovsbr$i ; done`

If you wanted to submit this for 1.7, you should have mentioned it
somewhere.

> Just not sure if the master branch improves this by some
> other way. If it does, please point me the commit?

I believe that the following commit fixes the problem on master.  I have
not tested it.

commit acf60855126bcfa79ea22d7846af5f2efe26cd30
Author: Justin Pettit <jpettit at nicira.com>
Date:   Tue Oct 30 17:41:22 2012 -0700

    ofproto-dpif: Use a single underlying datapath across multiple bridges.
    
    This commit switches to using a single backing datapath (called
    "ovs-datapath") for all bridges of that datapath's type.  Previously,
    resources couldn't be shared across bridges, since each was in its own
    datapath.  This change will allow sharing of tunnels and cheaper patch
    ports to be added in the future.
    
    Since bridges share a common datapath, the ovs-dpctl commands won't
    provide bridge-specific information.  Users wishing to have that
    information should use the new "ovs-appctl dpif/*" commands as
    documented in ovs-vswitchd(8).
    
    Signed-off-by: Justin Pettit <jpettit at nicira.com>



More information about the dev mailing list