On 3/21/23 19:05, Mike Pattrick wrote:
> From: Flavio Leitner <f...@sysclose.org>
> 
> The netdev receiving packets is supposed to provide the flags
> indicating if the IP checksum was verified and it is GOOD or BAD,
> otherwise the stack will check when appropriate by software.
> 
> If the packet comes with good checksum, then postpone the
> checksum calculation to the egress device if needed.
> 
> When encapsulate a packet with that flag, set the checksum
> of the inner IP header since that is not yet supported.
> 
> Calculate the IP checksum when the packet is going to be sent over
> a device that doesn't support the feature.
> 
> Linux devices don't support IP checksum offload alone, so the
> support is not enabled.
> 
> Signed-off-by: Flavio Leitner <f...@sysclose.org>
> Co-authored-by: Mike Pattrick <m...@redhat.com>
> Signed-off-by: Mike Pattrick <m...@redhat.com>
> ---
> Since v9:
>  - Removed duplicative field tx_ip_csum_offload from netdev-dpdk.c
>  - Left rx_csum_offload field as it is not duplicative
>  - Moved system-userspace-offload.at tests to dpif-netdev.at
>  - Various visual changes
>  - Extended miniflow_extract changes into avx512 code

Hi, Mike.  This patch fails Intel CI on autovalidation of
avx512 actions.  Could you, please, check?

Nit:  It would be better if you have a cover letter for the
patch set and send patches in reply to the cover letter instead
of each patch in reply to a previous one.  And there is no
need to send new versions in reply to the old ones. :)

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to