Looks good. --Justin
On Feb 22, 2011, at 4:28 PM, Ben Pfaff wrote: > Multipath actions only support registers as destinations, but this was > defined by reference in nicira-ext.h and the referenced text changed. > --- > include/openflow/nicira-ext.h | 5 +++-- > 1 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/include/openflow/nicira-ext.h b/include/openflow/nicira-ext.h > index 88efdf6..95c8989 100644 > --- a/include/openflow/nicira-ext.h > +++ b/include/openflow/nicira-ext.h > @@ -509,8 +509,9 @@ OFP_ASSERT(sizeof(struct nx_action_note) == 16); > * Some algorithms use 'arg' as an additional argument. > * > * 3. Stores 'link' in dst[ofs:ofs+n_bits]. The format and semantics of > - * 'dst' and 'ofs_nbits' are identical to those for the NXAST_REG_LOAD > - * action; refer to the description of that action for details. > + * 'dst' and 'ofs_nbits' are similar to those for the NXAST_REG_LOAD > + * action, except that 'dst' must be NXM_NX_REG(idx) for 'idx' in the > + * switch's supported range. > * > * The switch will reject actions that have an unknown 'fields', or an unknown > * 'algorithm', or in which ofs+n_bits is greater than the width of 'dst', or > -- > 1.7.1 > > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev_openvswitch.org _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
