Commit f9a10440 authored by Raed Salem's avatar Raed Salem Committed by Saeed Mahameed
Browse files

net/mlx5e: IPSEC RX, enable checksum complete 



Currently in Rx data path IPsec crypto offloaded packets uses
csum_none flag, so checksum is handled by the stack, this naturally
have some performance/cpu utilization impact on such flows. As Nvidia
NIC starting from ConnectX6DX provides checksum complete value out of
the box also for such flows there is no sense in taking csum_none path,
furthermore the stack (xfrm) have the method to handle checksum complete
corrections for such flows i.e. IPsec trailer removal and consequently
checksum value adjustment.

Because of the above and in addition the ConnectX6DX is the first HW
which supports IPsec crypto offload then it is safe to report csum
complete for IPsec offloaded traffic.

Fixes: b2ac7541 ("net/mlx5e: IPsec: Add Connect-X IPsec Rx data path offload")
Signed-off-by: default avatarRaed Salem <raeds@nvidia.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@nvidia.com>
parent 35306eb2

drivers/net/ethernet/mellanox/mlx5/core/en_rx.c

+1 −6
Original line number Diff line number Diff line
@@ -1001,14 +1001,9 @@ static inline void mlx5e_handle_csum(struct net_device *netdev, 
		goto csum_unnecessary;



	if (likely(is_last_ethertype_ip(skb, &network_depth, &proto))) {

		u8 ipproto = get_ip_proto(skb, network_depth, proto);



		if (unlikely(ipproto == IPPROTO_SCTP))

		if (unlikely(get_ip_proto(skb, network_depth, proto) == IPPROTO_SCTP))

			goto csum_unnecessary;



		if (unlikely(mlx5_ipsec_is_rx_flow(cqe)))

			goto csum_none;



		stats->csum_complete++;

		skb->ip_summed = CHECKSUM_COMPLETE;

		skb->csum = csum_unfold((__force __sum16)cqe->check_sum);