[ovs-dev] [PATCH] dpif-netdev: Fix crash when PACKET_OUT is metered

[Tony van der Peet]

Thanks Ilya. For what it's worth, besides running the OVS unit tests, I put
this new code through our (enhanced) version of oftest (500 test cases)
including a couple I wrote just for this situation.

Tony

On Thu, Jun 17, 2021 at 8:05 AM Ilya Maximets <i.maximets at ovn.org> wrote:

> On 6/16/21 2:04 AM, Tony van der Peet wrote:
> > From: Tony van der Peet <tony.vanderpeet at gmail.com>
> >
> > When a PACKET_OUT has output port of OFPP_TABLE, and the rule
> > table includes a meter and this causes the packet to be deleted,
> > stop the packet from being deleted twice by cloning it and setting
> > it up to be stolen in execution.
> >
> > Add a test to verify this condition.
> >
> > Signed-off-by: Tony van der Peet <tony.vanderpeet at gmail.com>
>
> Thanks for the patch!  OVS seems to work fine with this change,
> but for some reason several OVN unit tests are failing if it
> uses OVS with this change applied.
>
> Trying to figure out why...
>
> Best regards, Ilya Maximets.
>