[ovs-dev] [PATCH 1/2] tests/system-offloads-traffic.at: add sanity check

Joe Stringer joe at ovn.org
Wed Aug 16 22:17:30 UTC 2017


On 16 August 2017 at 05:14, Roi Dayan <roid at mellanox.com> wrote:
> Doing dump-flows also altering the netdev ports list.
> So doing it pre the actual test is adding a check to
> make sure we don't break the that list.
>
> Signed-off-by: Roi Dayan <roid at mellanox.com>
> Reviewed-by: Paul Blakey <paulb at mellanox.com>

I'm actually not sure what the requirements are to run these offload
tests. I tried running them on a 4.4 kernel, and the first test passed
while the second failed; I assume that this is because 4.4's TC flower
support is not new enough.

Then I tried with a 4.12 kernel and neither test passed, and I have
extra flows being reported in the dump-flows output.

I believe that I understand what this patch is trying to achieve, but
I don't know how I'm supposed to validate it.


More information about the dev mailing list