[ovs-dev] [PATCH] ofproto-dpif: fix issue with non-reversible actions on a patch ports

Eelco Chaudron echaudro at redhat.com
Tue Jun 15 07:15:36 UTC 2021



On 14 Jun 2021, at 20:15, Ilya Maximets wrote:

> On 6/14/21 4:28 PM, Eelco Chaudron wrote:
>> For patch ports, the is_last_action value is not propagated and is
>> always set to true. This causes non-reversible actions to modify the
>> packet, and the original content is not preserved when processing
>> the remaining actions.
>>
>> This patch propagates the is_last_action flag for patch port related
>> actions. It also includes a test case
>>
>> Signed-off-by: Eelco Chaudron <echaudro at redhat.com>
>> ---
>
> Hi, Eelco.  I didn't review the code, but this change fails a lot
> of tests in our CI: https://github.com/ovsrobot/ovs/actions/runs/936204455
> Basically it introduces extra clone() actions where was no clones
> before and where these clones are not really needed.

My bad! I did not run the full test set. Will investigate and fix where needed!

//Eelco



More information about the dev mailing list