"dev" <[email protected]> wrote on 08/08/2016 02:44:45 AM:
> From: Ben Pfaff <[email protected]> > To: [email protected] > Cc: Ben Pfaff <[email protected]> > Date: 08/08/2016 02:46 AM > Subject: [ovs-dev] [PATCH 06/30] ofp-actions: Correct member name > for write_actions. > Sent by: "dev" <[email protected]> > > 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 <[email protected]> > --- > 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 <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
