Re: [ovs-dev] [PATCH 00/11] tests: Add system-traffic.at tests to check-offloads.

2023-02-02 Thread Eelco Chaudron
Please ignore this version, it’s missing a version number and changes to two 
files :(

//Eelco


On 2 Feb 2023, at 8:37, Eelco Chaudron wrote:

> This series makes it possible to include system-traffic.at tests into
> "make check-offloads" tests.
>
> The last patch of the series explains which tests are still not passing
> and might need some more work.
>
> I'll try to work on the remaining failing test cases or find someone
> who can work on them.
>
> These tests where executed on a Fedora37 machine with the kernel
> 6.1.5-200.fc37.x86_64 installed.
>
> v9:
>   - Exclude "recirc_id sharing not supported" warning from the log.
>   - Reworked to use skip based macro rather than skip list.
>   - Fixes some spellings.
>   - Removed patches for issues no longer existing when using the
> latest kernel/OVS.
> v8:
>   - Re-based on top of latest OVS master.
> v7:
>   - Removed left over merge comment, and re-run all tests.
> v6:
>   - Added ACKs from v5
>   - Changed 'netdev-offload-tc: If the flow has not been used, report
> it as such.' to also work on hardware offloaded flows.
> v5:
>   - Include all patches, v4 went out with missing two patches :(
> v4:
>   - Fix rename from system-traffic.at to sym-traffic.at in patch 11
> v3:
>   - Fixed missing MACRO's in patches 4, 6 and 10.
> v2:
>   - Fix commit message on last patch
>   - Moved handling of system-traffic.at tests to a separate file
> system-offloads.at
>   - Re-based to the latest ovs master branch
>   - Added Roi's ACKs
>
> Eelco Chaudron (11):
>   tests: Include working system-traffic tests into the 
> system-offloads-testsuite.
>   test: Do not use MPLS implicit null label in test cases.
>   netdev-offload-tc: Fix tc conntrack force commit support.
>   test: Flush datapath when changing rules on the fly.
>   netdev-offload-tc: Conntrack ALGs are not supported with tc.
>   test: tc does not support conntrack timeout, skip the related test.
>   test: Fix 'conntrack - Multiple ICMP traverse' for tc case.
>   odp-util: Make odp_flow_key_from_flow__ nlattr order the same as the 
> kernel.
>   netdev-offload-tc: If the flow has not been used, report it as such.
>   tests: Fix reading of OpenFlow byte counters in GRE test cases.
>   tests: Comment currently failing TC system-traffic tests.
>
>
>  Documentation/howto/tc-offload.rst| 11 +++
>  lib/netdev-offload-tc.c   | 17 -
>  lib/odp-util.c| 21 +++---
>  lib/tc.c  | 14 +++-
>  tests/automake.mk |  3 +-
>  tests/dpif-netdev.at  | 28 
>  tests/mcast-snooping.at   |  4 +-
>  tests/nsh.at  | 10 +--
>  tests/odp.at  | 84 +++
>  tests/ofproto-dpif.at | 30 
>  tests/ofproto-macros.at   |  1 +
>  tests/packet-type-aware.at| 22 +++---
>  tests/pmd.at  |  2 +-
>  tests/system-offloads-testsuite-macros.at | 69 +++
>  tests/system-offloads-testsuite.at|  3 +
>  tests/system-traffic.at   | 37 ++
>  tests/tunnel-push-pop-ipv6.at |  2 +-
>  tests/tunnel-push-pop.at  |  2 +-
>  tests/tunnel.at   |  2 +-
>  19 files changed, 239 insertions(+), 123 deletions(-)
>  create mode 100644 tests/system-offloads-testsuite-macros.at
>
> ___
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


[ovs-dev] [PATCH 00/11] tests: Add system-traffic.at tests to check-offloads.

2023-02-01 Thread Eelco Chaudron
This series makes it possible to include system-traffic.at tests into
"make check-offloads" tests.

The last patch of the series explains which tests are still not passing
and might need some more work.

I'll try to work on the remaining failing test cases or find someone
who can work on them.

These tests where executed on a Fedora37 machine with the kernel
6.1.5-200.fc37.x86_64 installed.

v9:
  - Exclude "recirc_id sharing not supported" warning from the log.
  - Reworked to use skip based macro rather than skip list.
  - Fixes some spellings.
  - Removed patches for issues no longer existing when using the
latest kernel/OVS.
v8:
  - Re-based on top of latest OVS master.
v7:
  - Removed left over merge comment, and re-run all tests.
v6:
  - Added ACKs from v5
  - Changed 'netdev-offload-tc: If the flow has not been used, report
it as such.' to also work on hardware offloaded flows.
v5:
  - Include all patches, v4 went out with missing two patches :(
v4:
  - Fix rename from system-traffic.at to sym-traffic.at in patch 11
v3:
  - Fixed missing MACRO's in patches 4, 6 and 10.
v2:
  - Fix commit message on last patch
  - Moved handling of system-traffic.at tests to a separate file
system-offloads.at
  - Re-based to the latest ovs master branch
  - Added Roi's ACKs

Eelco Chaudron (11):
  tests: Include working system-traffic tests into the 
system-offloads-testsuite.
  test: Do not use MPLS implicit null label in test cases.
  netdev-offload-tc: Fix tc conntrack force commit support.
  test: Flush datapath when changing rules on the fly.
  netdev-offload-tc: Conntrack ALGs are not supported with tc.
  test: tc does not support conntrack timeout, skip the related test.
  test: Fix 'conntrack - Multiple ICMP traverse' for tc case.
  odp-util: Make odp_flow_key_from_flow__ nlattr order the same as the 
kernel.
  netdev-offload-tc: If the flow has not been used, report it as such.
  tests: Fix reading of OpenFlow byte counters in GRE test cases.
  tests: Comment currently failing TC system-traffic tests.


 Documentation/howto/tc-offload.rst| 11 +++
 lib/netdev-offload-tc.c   | 17 -
 lib/odp-util.c| 21 +++---
 lib/tc.c  | 14 +++-
 tests/automake.mk |  3 +-
 tests/dpif-netdev.at  | 28 
 tests/mcast-snooping.at   |  4 +-
 tests/nsh.at  | 10 +--
 tests/odp.at  | 84 +++
 tests/ofproto-dpif.at | 30 
 tests/ofproto-macros.at   |  1 +
 tests/packet-type-aware.at| 22 +++---
 tests/pmd.at  |  2 +-
 tests/system-offloads-testsuite-macros.at | 69 +++
 tests/system-offloads-testsuite.at|  3 +
 tests/system-traffic.at   | 37 ++
 tests/tunnel-push-pop-ipv6.at |  2 +-
 tests/tunnel-push-pop.at  |  2 +-
 tests/tunnel.at   |  2 +-
 19 files changed, 239 insertions(+), 123 deletions(-)
 create mode 100644 tests/system-offloads-testsuite-macros.at

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev