On 10/15/2020 1:41 PM, Gregory Etelson wrote:
Tunnel Offload API provides hardware independent, unified model
to offload tunneled traffic. Key model elements are:
  - apply matches to both outer and inner packet headers
    during entire offload procedure;
  - restore outer header of partially offloaded packet;
  - model is implemented as a set of helper functions.

  v2:
  * documentation updates
  * MLX5 PMD implementation for tunnel offload
  * testpmd updates for tunnel offload

  v3:
  * documentation updates
  * MLX5 PMD updates
  * testpmd updates

  v4:
  * updated patch: allow negative values in flow rule types

v5:
  * rebase to next-net

Eli Britstein (1):
   ethdev: tunnel offload model

Gregory Etelson (2):
   ethdev: allow negative values in flow rule types
   app/testpmd: add commands for tunnel offload API


32 build fails [1], can you please check?

Also there are a few typos [2].


[1]
../app/test-pmd/config.c: In function ‘port_flow_tunnel_list’:
../app/test-pmd/config.c:1580:18: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘uint64_t’ {aka ‘long long unsigned int’} [-Werror=format=]

1580 |    printf(" id=%lu", flt->tunnel.tun_id);
     |                ~~^   ~~~~~~~~~~~~~~~~~~
     |                  |              |
     |                  |              uint64_t {aka long long unsigned int}
     |                  long unsigned int
     |                %llu


[2]
### ethdev: allow negative values in flow rule types

WARNING:TYPO_SPELLING: 'becase' may be misspelled - perhaps 'because'?
#17:
actions do not fit into this scheme becase PMD type values are

total: 0 errors, 1 warnings, 72 lines checked

### app/testpmd: add commands for tunnel offload API

WARNING:TYPO_SPELLING: 'argumens' may be misspelled - perhaps 'arguments'?
#87: FILE: app/test-pmd/cmdline_flow.c:79:
+       /* Tunnel argumens. */

Reply via email to