On Thu, Nov 15, 2012 at 11:15:54AM +0800, Cong Wang wrote:
> On Thu, Nov 15, 2012 at 11:06 AM, Ben Pfaff <b...@nicira.com> wrote:
> > 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.
>
> I didn't know master branch has a fix.
>
> >
> >> 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.
> >
>
> I will test it, not sure if it is proper to backport such a large patch
> to 1.7. If not, please consider my patch.
Your patch is much more appropriate for branches other than master.
(This fix was not the primary purpose of the patch applied to master.
It was only a side effect.)
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
