On Tue, Oct 15, 2013 at 12:40:38PM -0700, Jarno Rajahalme wrote: > Since at the datapath interface we do not have set actions for > individual fields, but larger sets of fields for a given protocol > layer, the set action will in practice only ever apply to exactly > matched flows for the given protocol layer. For example, if the > reg_load changes the IP TTL, the corresponding datapath action will > rewrite also the IP addresses and TOS byte. Since these other field > values may not be explicitly set, they depend on the incoming flow field > values, and are hence all of them are set in the wildcards masks, when > the action is committed to the datapath. For the rare case, where the > reg_load action does not actually change the value, and no other flow > field values are set (or loaded), the datapath action is skipped, and > no mask bits are set. Such a datapath flow should, however, be > dependent on the specific field value, so the corresponding wildcard > mask bits must be set, lest the datapath flow be applied to packets > containing some other value in the field and the field value remain > unchanged regardless of the incoming value. > > Signed-off-by: Jarno Rajahalme <[email protected]>
I applied both of these patches to master, thanks! Would you mind looking at doing backports to 2.0 and 1.11? Let me know what I need to do. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
