diff mbox series

[18/25] crypto/cnxk: fix inflight cnt calculation

Message ID	1638859858-734-19-git-send-email-anoobj@marvell.com (mailing list archive)
State	Superseded, archived
Delegated to:	akhil goyal
Headers	From: Anoob Joseph <anoobj@marvell.com> To: Akhil Goyal <gakhil@marvell.com>, Jerin Jacob <jerinj@marvell.com> CC: Anoob Joseph <anoobj@marvell.com>, Archana Muniganti <marchana@marvell.com>, Tejasree Kondoj <ktejasree@marvell.com>, <dev@dpdk.org> Subject: [PATCH 18/25] crypto/cnxk: fix inflight cnt calculation Date: Tue, 7 Dec 2021 12:20:51 +0530 Message-ID: <1638859858-734-19-git-send-email-anoobj@marvell.com> In-Reply-To: <1638859858-734-1-git-send-email-anoobj@marvell.com> References: <1638859858-734-1-git-send-email-anoobj@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain Precedence: list Errors-To: dev-bounces@dpdk.org
Series	New features and improvements in cnxk crypto PMD \| [00/25] New features and improvements in cnxk crypto PMD [01/25] common/cnxk: define minor opcodes for MISC opcode [02/25] common/cnxk: add aes-xcbc key derive [03/25] common/cnxk: add bit fields for params [04/25] common/cnxk: fix reset of fields [05/25] common/cnxk: verify input args [06/25] crypto/cnxk: only enable queues that are allocated [07/25] crypto/cnxk: add lookaside IPsec AES-CBC-HMAC-SHA256 support [08/25] crypto/cnxk: clear session data before populating [09/25] crypto/cnxk: update max sec crypto caps [10/25] crypto/cnxk: write CPT CTX through microcode op [11/25] crypto/cnxk: support cnxk lookaside IPsec HMAC-SHA384/512 [12/25] crypto/cnxk: account for CPT CTX updates and flush delays [13/25] crypto/cnxk: use struct sizes for ctx writes [14/25] crypto/cnxk: add security session stats get [15/25] crypto/cnxk: add skip for unsupported cases [16/25] crypto/cnxk: add context reload for IV [17/25] crypto/cnxk: handle null chained ops [18/25] crypto/cnxk: fix inflight cnt calculation [19/25] crypto/cnxk: use atomics to access cpt res [20/25] crypto/cnxk: add more info on command timeout [21/25] crypto/cnxk: support lookaside IPsec AES-CTR [22/25] crypto/cnxk: fix extend tail calculation [23/25] crypto/cnxk: add aes xcbc and null cipher [24/25] crypto/cnxk: add copy and set DF [25/25] crypto/cnxk: add aes cmac

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Anoob Joseph Dec. 7, 2021, 6:50 a.m. UTC

  Inflight count calculation is updated to cover wrap around cases where
head can become smaller than tail.


Reported-by: Kiran Kumar K <kirankumark@marvell.com>
Signed-off-by: Anoob Joseph <anoobj@marvell.com>
---
 drivers/crypto/cnxk/cnxk_cryptodev_ops.h | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff mbox series

Patch

diff --git a/drivers/crypto/cnxk/cnxk_cryptodev_ops.h b/drivers/crypto/cnxk/cnxk_cryptodev_ops.h
index ca363bb..0336ae1 100644
--- a/drivers/crypto/cnxk/cnxk_cryptodev_ops.h
+++ b/drivers/crypto/cnxk/cnxk_cryptodev_ops.h
@@ -156,7 +156,11 @@  pending_queue_retreat(uint64_t *index, const uint64_t mask, uint64_t nb_entry)
 static __rte_always_inline uint64_t
 pending_queue_infl_cnt(uint64_t head, uint64_t tail, const uint64_t mask)
 {
-	return (head - tail) & mask;
+	/*
+	 * Mask is nb_desc - 1. Add nb_desc to head and mask to account for
+	 * cases when tail > head, which happens during wrap around.
+	 */
+	return ((head + mask + 1) - tail) & mask;
 }
 
 static __rte_always_inline uint64_t