DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] l2fwd-crypto: skip dev configure for masked devices
@ 2021-02-23  6:13 Hemant Agrawal
  2021-04-05 18:47 ` Akhil Goyal
  2021-04-09 13:09 ` [dpdk-dev] [PATCH v2] l2fwd-crypto: fix " Hemant Agrawal
  0 siblings, 2 replies; 4+ messages in thread
From: Hemant Agrawal @ 2021-02-23  6:13 UTC (permalink / raw)
  To: dev, declan.doherty; +Cc: Apeksha Gupta

From: Apeksha Gupta <apeksha.gupta@nxp.com>

The devices which are masked by cryptodev mask should not be initialized
and skipped while traversing the device list.

Signed-off-by: Apeksha Gupta <apeksha.gupta@nxp.com>
---
 examples/l2fwd-crypto/main.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/examples/l2fwd-crypto/main.c b/examples/l2fwd-crypto/main.c
index e7a1eea30..656b14036 100644
--- a/examples/l2fwd-crypto/main.c
+++ b/examples/l2fwd-crypto/main.c
@@ -2265,6 +2265,12 @@ initialize_cryptodevs(struct l2fwd_crypto_options *options, unsigned nb_ports,
 		if (enabled_cdevs[cdev_id] == 0)
 			continue;
 
+		if (check_cryptodev_mask(options, cdev_id) < 0)
+			continue;
+
+		if (check_capabilities(options, cdev_id) < 0)
+			continue;
+
 		retval = rte_cryptodev_socket_id(cdev_id);
 
 		if (retval < 0) {
-- 
2.17.1


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

* [dpdk-dev] [PATCH] l2fwd-crypto: skip dev configure for masked devices
  2021-02-23  6:13 [dpdk-dev] [PATCH] l2fwd-crypto: skip dev configure for masked devices Hemant Agrawal
@ 2021-04-05 18:47 ` Akhil Goyal
  2021-04-09 13:09 ` [dpdk-dev] [PATCH v2] l2fwd-crypto: fix " Hemant Agrawal
  1 sibling, 0 replies; 4+ messages in thread
From: Akhil Goyal @ 2021-04-05 18:47 UTC (permalink / raw)
  To: hemant.agrawal; +Cc: apeksha.gupta, declan.doherty, dev, gakhil

The devices which are masked by cryptodev mask should not be initialized
and skipped while traversing the device list.

Signed-off-by: Apeksha Gupta <apeksha.gupta@nxp.com>
---
 examples/l2fwd-crypto/main.c | 6 ++++++
 1 file changed, 6 insertions(+)

again a fixes tag is missing.
title should be
examples/l2fwd-crypto: fix dev configure for masked devices


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

* [dpdk-dev] [PATCH v2] l2fwd-crypto: fix dev configure for masked devices
  2021-02-23  6:13 [dpdk-dev] [PATCH] l2fwd-crypto: skip dev configure for masked devices Hemant Agrawal
  2021-04-05 18:47 ` Akhil Goyal
@ 2021-04-09 13:09 ` Hemant Agrawal
  2021-04-13 12:59   ` [dpdk-dev] [EXT] " Akhil Goyal
  1 sibling, 1 reply; 4+ messages in thread
From: Hemant Agrawal @ 2021-04-09 13:09 UTC (permalink / raw)
  To: dev, gakhil; +Cc: stable, Apeksha Gupta

From: Apeksha Gupta <apeksha.gupta@nxp.com>

The devices which are masked by cryptodev mask should not be initialized
and skipped while traversing the device list.

Fixes: 6ae3fb9df66e ("examples/l2fwd-crypto: fix session mempool size")
Cc: stable@dpdk.org

Signed-off-by: Apeksha Gupta <apeksha.gupta@nxp.com>
---
 examples/l2fwd-crypto/main.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/examples/l2fwd-crypto/main.c b/examples/l2fwd-crypto/main.c
index 0ee3e1470a..1016ceb841 100644
--- a/examples/l2fwd-crypto/main.c
+++ b/examples/l2fwd-crypto/main.c
@@ -2251,6 +2251,12 @@ initialize_cryptodevs(struct l2fwd_crypto_options *options, unsigned nb_ports,
 		if (enabled_cdevs[cdev_id] == 0)
 			continue;
 
+		if (check_cryptodev_mask(options, cdev_id) < 0)
+			continue;
+
+		if (check_capabilities(options, cdev_id) < 0)
+			continue;
+
 		retval = rte_cryptodev_socket_id(cdev_id);
 
 		if (retval < 0) {
-- 
2.17.1


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

* Re: [dpdk-dev] [EXT] [PATCH v2] l2fwd-crypto: fix dev configure for masked devices
  2021-04-09 13:09 ` [dpdk-dev] [PATCH v2] l2fwd-crypto: fix " Hemant Agrawal
@ 2021-04-13 12:59   ` Akhil Goyal
  0 siblings, 0 replies; 4+ messages in thread
From: Akhil Goyal @ 2021-04-13 12:59 UTC (permalink / raw)
  To: Hemant Agrawal, dev; +Cc: stable, Apeksha Gupta

> From: Apeksha Gupta <apeksha.gupta@nxp.com>
> 
> The devices which are masked by cryptodev mask should not be initialized
> and skipped while traversing the device list.
> 
> Fixes: 6ae3fb9df66e ("examples/l2fwd-crypto: fix session mempool size")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Apeksha Gupta <apeksha.gupta@nxp.com>
> ---
Acked-by: Akhil Goyal <gakhil@marvell.com>

Applied to dpdk-next-crypto

Thanks.

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

end of thread, other threads:[~2021-04-13 12:59 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-23  6:13 [dpdk-dev] [PATCH] l2fwd-crypto: skip dev configure for masked devices Hemant Agrawal
2021-04-05 18:47 ` Akhil Goyal
2021-04-09 13:09 ` [dpdk-dev] [PATCH v2] l2fwd-crypto: fix " Hemant Agrawal
2021-04-13 12:59   ` [dpdk-dev] [EXT] " Akhil Goyal

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/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 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

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


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