On Tue, May 13, 2014 at 09:55:45AM +0300, Alexandru Copot wrote:
> On Tue, May 13, 2014 at 4:18 AM, Ben Pfaff <b...@nicira.com> wrote:
> > On Sun, May 11, 2014 at 09:39:32PM +0300, Alexandru Copot wrote:
> >> On Sat, May 10, 2014 at 5:40 AM, Ben Pfaff <b...@nicira.com> wrote:
> >> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> >>
> >> >
> >> > diff --git a/lib/ofp-msgs.h b/lib/ofp-msgs.h
> >> > index df7569e..b548f6b 100644
> >> > --- a/lib/ofp-msgs.h
> >> > +++ b/lib/ofp-msgs.h
> >> > @@ -162,8 +162,10 @@ enum ofpraw {
> >> >
> >> > /* OFPT 1.0 (12): struct ofp_port_status, struct ofp10_phy_port. */
> >> > OFPRAW_OFPT10_PORT_STATUS,
> >> > - /* OFPT 1.1+ (12): struct ofp_port_status, struct ofp11_port. */
> >> > + /* OFPT 1.1-1.3 (12): struct ofp_port_status, struct ofp11_port. */
> >> > OFPRAW_OFPT11_PORT_STATUS,
> >> > + /* OFPT 1.4+ (12): struct ofp_port_status, struct ofp14_port,
> >> > uint8_t[8][]. */
> >> > + OFPRAW_OFPT14_PORT_STATUS,
> >> >
> >> > /* OFPT 1.0 (13): struct ofp10_packet_out, uint8_t[]. */
> >> > OFPRAW_OFPT10_PACKET_OUT,
> >> > @@ -360,8 +362,10 @@ enum ofpraw {
> >> >
> >> > /* OFPST 1.0 (13): struct ofp10_phy_port[]. */
> >> > OFPRAW_OFPST10_PORT_DESC_REPLY,
> >> > - /* OFPST 1.1+ (13): struct ofp11_port[]. */
> >> > + /* OFPST 1.1-1.3 (13): struct ofp11_port[]. */
> >> > OFPRAW_OFPST11_PORT_DESC_REPLY,
> >> > + /* OFPST 1.4+ (13): uint8_t[8][]. */
> >> > + OFPRAW_OFPST14_PORT_DESC_REPLY,
> >> >
> >> > /* Nicira extension messages.
> >> > *
> >> > @@ -471,7 +475,8 @@ enum ofptype {
> >> > * OFPRAW_OFPT11_FLOW_REMOVED.
> >> > * OFPRAW_NXT_FLOW_REMOVED. */
> >> > OFPTYPE_PORT_STATUS, /* OFPRAW_OFPT10_PORT_STATUS.
> >> > - * OFPRAW_OFPT11_PORT_STATUS. */
> >> > + * OFPRAW_OFPT11_PORT_STATUS.
> >> > + * OFPRAW_OFPT14_PORT_STATUS. */
> >> >
> >>
> >> Now that there is OFPRAW_OFPT14_PORT_STATUS, encode_port_status()
> >> should use it for OF1.4 instead of OFPRAW_OFPT11_PORT_STATUS. Otherwise,
> >> now we get an abort.
> >
> > Noticing something minor must mean that you are paying pretty good
> > attention to the patches. Which ones have you reviewed? Do you expect
> > to have more comments?
>
> It was easy to notice because the switch crashes when connecting
> to an OF1.4 controller. I mostly read the patches regarding 1.4 features,
> until 14/20 and they seem good, couldn't find anything else.
Fair enough. (I did not know that OF1.4 controllers existed yet.)
> > I ask because getting reviews is all that is blocking these patches. I
> > think that they are decently written, so I'm just waiting for someone to
> > look them over.
>
> I'll try to look over the rest of them.
Thanks!
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
