"dev" <dev-boun...@openvswitch.org> wrote on 08/08/2016 02:44:45 AM:
> From: Ben Pfaff <b...@ovn.org> > To: dev@openvswitch.org > Cc: Ben Pfaff <b...@ovn.org> > Date: 08/08/2016 02:46 AM > Subject: [ovs-dev] [PATCH 06/30] ofp-actions: Correct member name > for write_actions. > Sent by: "dev" <dev-boun...@openvswitch.org> > > For a variable-length action like write_actions, the member name is > supposed to be the name of the variable-length array at the end of the > action structure. It only makes a real difference if the beginning of the > array is not 64-bit aligned, so it did not matter in this case, but it's > better to get it right. > > Signed-off-by: Ben Pfaff <b...@ovn.org> > --- > include/openvswitch/ofp-actions.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/openvswitch/ofp-actions.h b/include/ > openvswitch/ofp-actions.h > index 42764b1..a0cefe3 100644 > --- a/include/openvswitch/ofp-actions.h > +++ b/include/openvswitch/ofp-actions.h > @@ -119,7 +119,7 @@ > /* Instructions. */ \ > OFPACT(METER, ofpact_meter, ofpact, "meter") \ > OFPACT(CLEAR_ACTIONS, ofpact_null, ofpact, "clear_actions") \ > - OFPACT(WRITE_ACTIONS, ofpact_nest, ofpact, "write_actions") \ > + OFPACT(WRITE_ACTIONS, ofpact_nest, actions, "write_actions") \ > OFPACT(WRITE_METADATA, ofpact_metadata, ofpact, "write_metadata") \ > OFPACT(GOTO_TABLE, ofpact_goto_table, ofpact, "goto_table") > > -- Since I don't have PS 5 in my mailbox, this Ack counts for both this patch set and PS 5... Acked-by: Ryan Moats <rmo...@us.ibm.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
