[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openvswitch-dev
Subject:    [ovs-dev] [of1.1 resend 4/7] OPENFLOW-1.1+: Remove items that I've verified to be implemented alread
From:       blp () nicira ! com (Ben Pfaff)
Date:       2013-06-29 4:08:49
Message-ID: 20130629040849.GC8935 () nicira ! com
[Download RAW message or body]

Thank you for the reviews.  I applied patches 1, 2, and 4 to master.

On Sat, Jun 29, 2013 at 11:25:01AM +0900, Joe Stringer wrote:
> Looks good.
> 
> On Sat, Jun 29, 2013 at 8:27 AM, Ben Pfaff <blp at nicira.com> wrote:
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> >  OPENFLOW-1.1+ |   12 ------------
> >  1 files changed, 0 insertions(+), 12 deletions(-)
> >
> > diff --git a/OPENFLOW-1.1+ b/OPENFLOW-1.1+
> > index fb39873..d08a46e 100644
> > --- a/OPENFLOW-1.1+
> > +++ b/OPENFLOW-1.1+
> > @@ -62,10 +62,6 @@ probably incomplete.
> >        so the value is not necessarily meaningful.  We might have to
> >        just fix it as the same as in_port.
> >
> > -    * On OF1.1+ flow_mods, updates by MODIFY are now much better
> > -      specified.  Check that OVS implements the new behavior, fix it
> > -      if not.
> > -
> >      * OFPT_TABLE_MOD stats.  This is new in OF1.1, so we need to
> >        implement it.  It should be implemented so that the default OVS
> >        behavior does not change.
> > @@ -94,16 +90,8 @@ end of the OF1.2 spec.  I didn't compare the specs carefully yet.)
> >
> >      * OFPT_FLOW_MOD:
> >
> > -        * MODIFY and MODIFY_STRICT commands now never insert new flows
> > -          in the table.  We will still need variations that do,
> > -          though, both to support older OpenFlow protocols and to get
> > -          sensible behavior for the internal implementation of the
> > -          NXAST_LEARN action.
> > -
> >          * New flag OFPFF_RESET_COUNTS.
> >
> > -        * New cookie field behavior.
> > -
> >          * Add ability to delete flow in all tables.
> >
> >          * Update DESIGN to describe OF1.2 behavior also.
> > --
> > 1.7.2.5
> >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev

[prev in list] [next in list] [prev in thread] [next in thread]