Justin, are you planning to apply this then? Or, Joe, are you planning to post a new version? Just trying to figure out what's happening.
On Thu, Feb 27, 2014 at 10:59:00AM -0800, Justin Pettit wrote: > I think it's clear enough, since it explicitly mentions datapath > flows. There's a FAQ entry that describes the different types of > flows that Ben can point to, which always makes him happy. > > --Justin > > > On Feb 27, 2014, at 10:55 AM, Joe Stringer <joestrin...@nicira.com> wrote: > > > I agree, it would be helpful to have a sentence like that. > > > > This could be particularly misleading given that it is relating to datapath > > flows, which is completely different from OpenFlow rule timeouts. Do you > > think that this is clear enough with the current description? > > > > > > On 27 February 2014 10:50, Justin Pettit <jpet...@nicira.com> wrote: > > On Feb 27, 2014, at 10:37 AM, Joe Stringer <joestrin...@nicira.com> wrote: > > > > > + The maximum idle time in milliseconds for flows to be cached > > > in the > > > + datapath. A lower value may improve flow setup performance, > > > but > > > + decrease the number of cached flows in the datapath. > > > Conversely, a > > > + higher value allows more flows to be maintained in the cache > > > at the > > > + expense of flow setup performance. > > > > We get a lot of questions about locking particular flows in the kernel, and > > I'm afraid this description might give the impression that this will be the > > actual time it's kept in the kernel. (The name of the variable indicates > > that that's not the case, but I think most people will miss that.) What > > about adding a sentence like the following to the end? > > > > "Note that ovs-vswitchd may expire flows more quickly than the configured > > value based on system load and other factors." > > > > --Justin > > > > > > > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
