patches for DPDK stable branches
 help / color / mirror / Atom feed
* [PATCH] crypto/mlx5: fix max number of QPs capability
@ 2021-11-21 12:59 Raja Zidane
  2021-11-23 18:42 ` [EXT] " Akhil Goyal
  0 siblings, 1 reply; 2+ messages in thread
From: Raja Zidane @ 2021-11-21 12:59 UTC (permalink / raw)
  To: dev; +Cc: Matan Azrad, stable

An indirect mkey is created for each descriptor in a QP, number of
descriptors per QP is configured by the user on QP setup callback.
In mlx cryptodev autotest, the max number of QPs (provided by the driver)
is created, and due to mkey resource limits, QPs creation fail which leads
to the test failing.
Since there is no capability of max number of descriptors provided to
the user, we can't give an exact number of max QPs available.
Reduce the max number of QPs to 128, which supports standard descriptors
numbers, including the 4K number provided in the test.

Fixes: 6152534e211e ("crypto/mlx5: support queue pairs operations")
Cc: stable@dpdk.org

Signed-off-by: Raja Zidane <rzidane@nvidia.com>
---
Acked-by: Matan Azrad <matan@nvidia.com>
 drivers/crypto/mlx5/mlx5_crypto.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/mlx5/mlx5_crypto.c b/drivers/crypto/mlx5/mlx5_crypto.c
index 90e0df7fb2..421c23748a 100644
--- a/drivers/crypto/mlx5/mlx5_crypto.c
+++ b/drivers/crypto/mlx5/mlx5_crypto.c
@@ -20,7 +20,7 @@
 
 #define MLX5_CRYPTO_DRIVER_NAME crypto_mlx5
 #define MLX5_CRYPTO_LOG_NAME pmd.crypto.mlx5
-#define MLX5_CRYPTO_MAX_QPS 1024
+#define MLX5_CRYPTO_MAX_QPS 128
 #define MLX5_CRYPTO_MAX_SEGS 56
 
 #define MLX5_CRYPTO_FEATURE_FLAGS \
-- 
2.17.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [EXT] [PATCH] crypto/mlx5: fix max number of QPs capability
  2021-11-21 12:59 [PATCH] crypto/mlx5: fix max number of QPs capability Raja Zidane
@ 2021-11-23 18:42 ` Akhil Goyal
  0 siblings, 0 replies; 2+ messages in thread
From: Akhil Goyal @ 2021-11-23 18:42 UTC (permalink / raw)
  To: Raja Zidane, dev; +Cc: Matan Azrad, stable

> An indirect mkey is created for each descriptor in a QP, number of
> descriptors per QP is configured by the user on QP setup callback.
> In mlx cryptodev autotest, the max number of QPs (provided by the driver)
> is created, and due to mkey resource limits, QPs creation fail which leads
> to the test failing.
> Since there is no capability of max number of descriptors provided to
> the user, we can't give an exact number of max QPs available.
> Reduce the max number of QPs to 128, which supports standard descriptors
> numbers, including the 4K number provided in the test.
> 
> Fixes: 6152534e211e ("crypto/mlx5: support queue pairs operations")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Raja Zidane <rzidane@nvidia.com>
> ---
> Acked-by: Matan Azrad <matan@nvidia.com>

Applied to dpdk-next-crypto
Thanks.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-11-23 18:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-21 12:59 [PATCH] crypto/mlx5: fix max number of QPs capability Raja Zidane
2021-11-23 18:42 ` [EXT] " Akhil Goyal

patches for DPDK stable branches

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.dpdk.org/stable/0 stable/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 stable stable/ http://inbox.dpdk.org/stable \
		stable@dpdk.org
	public-inbox-index stable

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.stable


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git