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

[Joe Stringer]

On 17 August 2017 at 00:17, Roi Dayan <roid at mellanox.com> wrote:
>
>
> On 17/08/2017 08:32, Roi Dayan wrote:
>>
>>
>>
>> On 17/08/2017 01:17, Joe Stringer wrote:
>>>
>>> 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.
>>>
>>
>
> In the past you had an issue that cls_flower was not configured
> in your kernel. could be the same issue now?

Maybe it is, I changed which setup I was testing in. I'll double-check.