On Mon, Dec 31, 2012 at 12:58 PM, Ben Pfaff <[email protected]> wrote:
> On Mon, Dec 31, 2012 at 12:48:04PM -0800, Jesse Gross wrote:
>> The datapath doesn't carry over tunnel metadata between input and
>> output because such an operation doesn't generally make sense.  In
>> order to keep the flow information in sync, userspace needs to do
>> this as well.  However, userspace was clearing out only the base
>> flow, which means that it later generated an action with the original
>> tunnel metadata.  If a packet is both being sent and received on a
>> tunnel then this will cause the output tunnel's configuration to be
>> overriden with incorrect information.  This was recently exposed
>> when the ability to interpret tunnel metadata was added to userspace.
>>
>> Reported-by: Anand Krishnamurthy <[email protected]>
>> Reported-by: Saul St. John <[email protected]>
>> Signed-off-by: Jesse Gross <[email protected]>
>
> Looks good to me, thanks.

Thanks, I pushed both of these patches to master and the first one to
branch-1.9.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to