On 17 August 2017 at 00:17, Roi Dayan <r...@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 <r...@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 <r...@mellanox.com> >>>> Reviewed-by: Paul Blakey <pa...@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. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev