On 17 August 2017 at 11:07, Joe Stringer <[email protected]> wrote: > On 17 August 2017 at 00:17, Roi Dayan <[email protected]> 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 <[email protected]> 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 <[email protected]> >>>>> Reviewed-by: Paul Blakey <[email protected]> >>>> >>>> >>>> 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.
Managed to figure it out, looks good. Applied to master. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
