From 9f3ed88982319c8b35da78cedcda9792e7ec3213 Mon Sep 17 00:00:00 2001 From: Amit Mehta Date: Wed, 10 Nov 2021 14:36:18 +0530 Subject: [PATCH] qcacmn: Remove csum_enabled flag check Currently, to set checksum enable flags in Tx descriptor we are checking the csum_enabled flag along with stack checksum offload request. In case of roaming from latency-critical connection to non-latency critical connection we request stack to calculate checksum and set csum_enabled flag to 0. For some packets which are already queued where the stack has not calculated checksum and csum_enabled flag is set to 0, we will not set the checksum enable flags in Tx descriptors for those packets. To fix the issue remove the csum_enabled flag check and directly check if the stack has calculated checksum for those packets or not. Change-Id: I8d7754afc3d0a33315b85b0113cd3062e5783e28 CRs-Fixed: 3070858 --- dp/wifi3.0/dp_tx.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/dp/wifi3.0/dp_tx.c b/dp/wifi3.0/dp_tx.c index 2bb79296f82b..ce555cea9100 100644 --- a/dp/wifi3.0/dp_tx.c +++ b/dp/wifi3.0/dp_tx.c @@ -1696,9 +1696,8 @@ dp_tx_hw_enqueue(struct dp_soc *soc, struct dp_vdev *vdev, hal_tx_desc_set_to_fw(hal_tx_desc_cached, 1); /* verify checksum offload configuration*/ - if (vdev->csum_enabled && - ((qdf_nbuf_get_tx_cksum(tx_desc->nbuf) == QDF_NBUF_TX_CKSUM_TCP_UDP) - || qdf_nbuf_is_tso(tx_desc->nbuf))) { + if ((qdf_nbuf_get_tx_cksum(tx_desc->nbuf) == QDF_NBUF_TX_CKSUM_TCP_UDP) + || qdf_nbuf_is_tso(tx_desc->nbuf)) { hal_tx_desc_set_l3_checksum_en(hal_tx_desc_cached, 1); hal_tx_desc_set_l4_checksum_en(hal_tx_desc_cached, 1); }