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

Reply via email to