On 8/16/21 3:53 PM, Eli Britstein via dev wrote:
> Signed-off-by: Eli Britstein <[email protected]>
> ---
>  NEWS                      | 2 ++
>  lib/netdev-offload-dpdk.c | 5 -----
>  2 files changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/NEWS b/NEWS
> index 26920e215..80466c014 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -8,6 +8,8 @@ Post-v2.16.0
>         by default.  'other_config:dpdk-socket-limit' can be set equal to
>         the 'other_config:dpdk-socket-mem' to preserve the legacy memory
>         limiting behavior.
> +     * Add hardware offload support for matching IPv4/IPv6 frag types
> +       (experimental).
>  
>  
>  v2.16.0 - xx xxx xxxx
> diff --git a/lib/netdev-offload-dpdk.c b/lib/netdev-offload-dpdk.c
> index d79ad1bea..f6b1514b4 100644
> --- a/lib/netdev-offload-dpdk.c
> +++ b/lib/netdev-offload-dpdk.c
> @@ -1234,11 +1234,6 @@ parse_flow_match(struct netdev *netdev,
>          proto = spec->hdr.next_proto_id &
>                  mask->hdr.next_proto_id;
>      }
> -    /* If fragmented, then don't HW accelerate - for now. */
> -    if (match->wc.masks.nw_frag & match->flow.nw_frag) {
> -        return -1;
> -    }
> -    consumed_masks->nw_frag = 0;

As this is experimental, shouldn't it be disabled by default or at least
a possibility to disable it should be provided?

>  
>      /* IP v6 */
>      if (match->flow.dl_type == htons(ETH_TYPE_IPV6)) {
> 

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to