Sorry, I hadn't got around to it yet. I'll repost.
On 4 March 2014 09:46, Ben Pfaff <b...@nicira.com> wrote: > 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
