On Thu, Nov 15, 2012 at 10:31:04AM +0800, Cong Wang wrote:
> On Wed, Nov 14, 2012 at 11:43 PM, Ben Pfaff <b...@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 <b...@nicira.com>
> >> Signed-off-by: Cong Wang <xiyou.wangc...@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 <jpet...@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 <jpet...@nicira.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to