Add support for write-actions
authorSimon Horman <horms@verge.net.au>
Fri, 11 Oct 2013 04:23:29 +0000 (13:23 +0900)
committerBen Pfaff <blp@nicira.com>
Mon, 14 Oct 2013 22:20:03 +0000 (15:20 -0700)
commit7fdb60a758d76fecf5165a9d0bf1df9271699141
tree73cb60038bd795e2b176bc551ee7026c7909ed03
parent455ecd775d46f96fed8f307f44958a2a07ebcfff
Add support for write-actions

Implementation note:

All actions which modify a field are added to the action set
at the point where "set" actions should be added. In general
modifying a field many times is the same as only modifying it
the last time so the implementation simply adds all set actions to
the action set in the order they are specified. However, this breaks
down if two actions modify different portions of the same field.

Some examples.

1. load acting a subfield
2. mod_vlan_vid, mod_vlan_pcp

If this is considered to be a problem one possible solution would be to
either disallow all set actions other than set_field in write_actions.
Another possible solution is prohibit problematic the actions listed above
in write actions.

Signed-off-by: Simon Horman <horms@verge.net.au>
[blp@nicira.com simplified and edited the code]
Signed-off-by: Ben Pfaff <blp@nicira.com>
NEWS
OPENFLOW-1.1+
lib/ofp-actions.c
lib/ofp-actions.h
lib/ofp-parse.c
ofproto/ofproto-dpif-xlate.c
tests/ofp-actions.at
tests/ofproto-dpif.at
utilities/ovs-ofctl.8.in