On Thu, Jan 09, 2020 at 04:14:10PM +0100, Ilya Maximets wrote:
> On 09.01.2020 08:36, xiangxia.m....@gmail.com wrote:
> > From: Tonghao Zhang <xiangxia.m....@gmail.com>
> > 
> > For installing more than MAX_FLOWS (65536) flows to netdev datapath.
> > Add the ovs-appctl subcommand "dpif-netdev/pmd-set-max-flow" which
> > can change the flow number which netdev datapath support.
> > 
> > Signed-off-by: Tonghao Zhang <xiangxia.m....@gmail.com>
> 
> Hi.
> 
> I'm wondering why we need the flow limit on the datapath level at all?
> 
> MAX_FLOWS constant was there from the introduction of dpif-netdev,
> however, later new flow-limit mechanism was implemented that
> controls number of datapath flows in a dynamic way on ofproto level.
> 
> So, maybe we can just remove the limit and fully rely on ofproto
> to decide what flow limit we need?  There are no limitations for
> flow table size in dpif-netdev beside the artificial one.
> 'other_config:flow-limit' seems suitable to control this.
> 
> Ben, what do you think?

Hmm, that's a good point.

Tonghao, do you have a good reason to want to limit flows at this level?
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to