[ovs-dev] [PATCH] tun-metadata: Always set option present when copying data.

Jesse Gross jesse at nicira.com
Fri Aug 21 21:28:27 UTC 2015


On Fri, Aug 21, 2015 at 1:28 PM, Ben Pfaff <blp at nicira.com> wrote:
> On Thu, Aug 13, 2015 at 07:59:29PM -0700, Jesse Gross wrote:
>> Whenever we write into a tunnel option field, we also need to mark
>> it as significant. If we don't, then the data will later be ignored.
>>
>> We currently do this in every case except for flow metadata. This causes
>> us to not correctly serialize the tunnel metadata for Packet Ins to the
>> controller.
>>
>> Rather than separately writing the data and marking the options as present,
>> it is better to combine the two steps to ensure that one can never be
>> done without the other.
>>
>> Signed-off-by: Jesse Gross <jesse at nicira.com>
>
> Acked-by: Ben Pfaff <blp at nicira.com>

Thanks, applied to master.



More information about the dev mailing list