Re: [PATCH net v2 2/2] selftests: openvswitch: add tests for tunnel vport refcounting

[Aaron Conole]

Ilya Maximets <i.maximets@xxxxxxx> writes:

> There were a few issues found with the tunnel vport types around the
> vport destruction code.  Add some basic tests, so at least we know that
> they can be properly added and removed without obvious issues.
>
> The test creates OVS datapath, adds a non-LWT tunnel port, makes sure
> they are created, and then removes the datapath and waits for all the
> ports to be gone.
>
> The dpctl script had a few bugs in the none-lwt tunnel creation code,
> so fixing them as well to make the testing possible:
> - The type of the --lwt option changed in order to properly disable it.
> - Removed byte order conversion for the port numbers, as the value
>   supposed to be in the host order.
> - Added missing 'gre' choice for the tunnel type.
>
> Signed-off-by: Ilya Maximets <i.maximets@xxxxxxx>
> ---

Looks good to me.  Thanks for the test.

Acked-by: Aaron Conole <aconole@xxxxxxxxxx>