[ovs-dev] [branch-1.11] ofproto-dpif: Configure datapath max-idle through ovs-vsctl.

Alex Wang alexw at nicira.com
Mon Jun 23 17:36:44 UTC 2014


Thx, I already pushed them,

Forgot to mention in this thread.

Thanks,
Alex Wang,


On Mon, Jun 23, 2014 at 10:33 AM, Ben Pfaff <blp at nicira.com> wrote:

> OK. I understand now. Do you still need any reviews?
>
> On Mon, Jun 23, 2014 at 10:30 AM, Alex Wang <alexw at nicira.com> wrote:
> > Hey Ben,
> >
> > Thanks for pointing it out,  I actually do not have a clear judge on
> this.
> > I think it is necessary in this case,
> >
> > For branch-2.0, we are before the revalidator-as-a-thread era.  So, it is
> > pretty
> > different from the implementation in commit 72310b04.
> >
> > For branch-1.11, 1.10, we need to configure the option on a different
> table.
> >
> > Thanks,
> > Alex Wang,
> >
> >
> >
> >
> > On Mon, Jun 23, 2014 at 10:23 AM, Ben Pfaff <blp at nicira.com> wrote:
> >>
> >> On Thu, Jun 19, 2014 at 09:20:57PM -0700, Alex Wang wrote:
> >> > This patch adds a new configuration option, "max-idle" to the
> >> > Bridge "other-config" column. This sets how long datapath flows,
> >> > for the configured bridge, are cached in the datapath before
> >> > ovs-vswitchd thread expires them.
> >> >
> >> > This commit is a backport of commit 72310b04 (upcall: Configure
> >> > datapath max-idle through ovs-vsctl.).
> >> >
> >> > Signed-off-by: Alex Wang <alexw at nicira.com>
> >>
> >> Were there significant changes in the backports?  It's unusual to post
> >> all the versions of a patch to backport, because backports are usually
> >> straightforward, so that getting a review on each of them consumes
> >> time for both the poster and the reviewer without much reduction of
> >> risk.
> >
> >
>
>
>
> --
> "I don't normally do acked-by's.  I think it's my way of avoiding
> getting blamed when it all blows up."               Andrew Morton
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20140623/f8d9e968/attachment-0005.html>


More information about the dev mailing list