Re: [ovs-dev] [of1.1 resend 7/7] Add basic OpenFlow 1.1 protocol support.

2013-07-08 Thread Ben Pfaff
On Thu, Jul 04, 2013 at 12:34:46PM +0900, Joe Stringer wrote: > On Sat, Jun 29, 2013 at 8:27 AM, Ben Pfaff wrote: > > @@ -1514,7 +1593,17 @@ ofputil_decode_flow_mod(struct ofputil_flow_mod *fm, > > > > /* Translate the message. */ > > fm->priority = ntohs(ofm->priority); > > -

Re: [ovs-dev] [of1.1 resend 7/7] Add basic OpenFlow 1.1 protocol support.

2013-07-04 Thread Joe Stringer
Glad to help out. Enjoy your vacation! On Thu, Jul 4, 2013 at 8:59 PM, Ben Pfaff wrote: > Thanks a lot for reviewing these patches. I've wanted to get in the OpenFlow > 1.1 support for some time (it seems silly to support both earlier and later > versions but not 1.1 itself). > > I'm currently on

Re: [ovs-dev] [of1.1 resend 7/7] Add basic OpenFlow 1.1 protocol support.

2013-07-04 Thread Ben Pfaff
Thanks a lot for reviewing these patches. I've wanted to get in the OpenFlow 1.1 support for some time (it seems silly to support both earlier and later versions but not 1.1 itself). I'm currently on vacation but I'll get to responding to your comments and applying these patches next week. On Jul

Re: [ovs-dev] [of1.1 resend 7/7] Add basic OpenFlow 1.1 protocol support.

2013-07-03 Thread Joe Stringer
On Sat, Jun 29, 2013 at 8:27 AM, Ben Pfaff wrote: > @@ -1514,7 +1593,17 @@ ofputil_decode_flow_mod(struct ofputil_flow_mod *fm, > > /* Translate the message. */ > fm->priority = ntohs(ofm->priority); > -if (ofm->command == OFPFC_ADD) { > +if (oh->version == OFP11_

[ovs-dev] [of1.1 resend 7/7] Add basic OpenFlow 1.1 protocol support.

2013-06-28 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- FAQ| 16 +++--- NEWS |2 + lib/ofp-util.c | 125 +++ lib/ofp-util.h | 19 ++- tests/learn.at |2 +- tests/ofp-print.at |