On Oct 17, 2013, at 12:40 PM, Ben Pfaff <b...@nicira.com> wrote: > On Wed, Oct 16, 2013 at 04:16:08PM -0700, Jarno Rajahalme wrote: >> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> > > Why is saving space here worthwhile?
The difference in setting an IPv4 address, any register, port number, etc., is 8 bytes vs. 24 bytes (i.e, factor of three). I thought this could be significant for cases where there are a lot of actions. There is one constructor for them, and rest of the code usually does not need to care about it, other than using an accessor to get the data pointer. Otherwise on the source level the data members can be accessed quite much like with the union mf_value, so I thought it turned out quite nice :-) However, I explicitly put this in as a separate patch so that you can say what you think. Jarno _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev