DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] driver: crypto: scheduler: fix session size computation
@ 2024-07-05  7:34 jhascoet
  2024-10-04  8:40 ` Ji, Kai
  0 siblings, 1 reply; 6+ messages in thread
From: jhascoet @ 2024-07-05  7:34 UTC (permalink / raw)
  To: dev
  Cc: jhascoet, stable, Julien Hascoet, Kai Ji, Anoob Joseph,
	Kevin O'Sullivan, Ciara Power, David Coyle

The crypto scheduler session size computation was taking
into account only the worker session sizes and not its own.

Fixes: e2af4e403c1 ("crypto/scheduler: support DOCSIS security protocol")
Cc: stable@dpdk.org

Signed-off-by: Julien Hascoet <jhascoet@gmail.com>
---
 drivers/crypto/scheduler/scheduler_pmd_ops.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/scheduler/scheduler_pmd_ops.c b/drivers/crypto/scheduler/scheduler_pmd_ops.c
index a18f7a08b0..6e43438469 100644
--- a/drivers/crypto/scheduler/scheduler_pmd_ops.c
+++ b/drivers/crypto/scheduler/scheduler_pmd_ops.c
@@ -185,7 +185,7 @@ scheduler_session_size_get(struct scheduler_ctx *sched_ctx,
 		uint8_t session_type)
 {
 	uint8_t i = 0;
-	uint32_t max_priv_sess_size = 0;
+	uint32_t max_priv_sess_size = sizeof(struct scheduler_session_ctx);
 
 	/* Check what is the maximum private session size for all workers */
 	for (i = 0; i < sched_ctx->nb_workers; i++) {
-- 
2.34.1


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH] driver: crypto: scheduler: fix session size computation
@ 2024-07-05 12:57 jhascoet
  0 siblings, 0 replies; 6+ messages in thread
From: jhascoet @ 2024-07-05 12:57 UTC (permalink / raw)
  To: dev
  Cc: Julien Hascoet, stable, Kai Ji, Kevin O'Sullivan,
	Ciara Power, David Coyle, Anoob Joseph

From: Julien Hascoet <ju.hascoet@gmail.com>

The crypto scheduler session size computation was taking
into account only the worker session sizes and not its own.

Fixes: e2af4e403c1 ("crypto/scheduler: support DOCSIS security protocol")
Cc: stable@dpdk.org

Signed-off-by: Julien Hascoet <ju.hascoet@gmail.com>
---
 drivers/crypto/scheduler/scheduler_pmd_ops.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/scheduler/scheduler_pmd_ops.c b/drivers/crypto/scheduler/scheduler_pmd_ops.c
index a18f7a08b0..6e43438469 100644
--- a/drivers/crypto/scheduler/scheduler_pmd_ops.c
+++ b/drivers/crypto/scheduler/scheduler_pmd_ops.c
@@ -185,7 +185,7 @@ scheduler_session_size_get(struct scheduler_ctx *sched_ctx,
 		uint8_t session_type)
 {
 	uint8_t i = 0;
-	uint32_t max_priv_sess_size = 0;
+	uint32_t max_priv_sess_size = sizeof(struct scheduler_session_ctx);
 
 	/* Check what is the maximum private session size for all workers */
 	for (i = 0; i < sched_ctx->nb_workers; i++) {
-- 
2.34.1


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH] driver: crypto: scheduler: fix session size computation
@ 2024-07-05  8:35 jhascoet
  2024-09-30 11:33 ` Ji, Kai
  0 siblings, 1 reply; 6+ messages in thread
From: jhascoet @ 2024-07-05  8:35 UTC (permalink / raw)
  To: dev
  Cc: jhascoet, stable, Kai Ji, Kevin O'Sullivan, David Coyle,
	Ciara Power, Anoob Joseph

From: jhascoet <jhascoet@gmail.com>

The crypto scheduler session size computation was taking
into account only the worker session sizes and not its own.

Fixes: e2af4e403c1 ("crypto/scheduler: support DOCSIS security protocol")
Cc: stable@dpdk.org

Signed-off-by: Julien Hascoet <jhascoet@gmail.com>
---
 drivers/crypto/scheduler/scheduler_pmd_ops.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/scheduler/scheduler_pmd_ops.c b/drivers/crypto/scheduler/scheduler_pmd_ops.c
index a18f7a08b0..6e43438469 100644
--- a/drivers/crypto/scheduler/scheduler_pmd_ops.c
+++ b/drivers/crypto/scheduler/scheduler_pmd_ops.c
@@ -185,7 +185,7 @@ scheduler_session_size_get(struct scheduler_ctx *sched_ctx,
 		uint8_t session_type)
 {
 	uint8_t i = 0;
-	uint32_t max_priv_sess_size = 0;
+	uint32_t max_priv_sess_size = sizeof(struct scheduler_session_ctx);
 
 	/* Check what is the maximum private session size for all workers */
 	for (i = 0; i < sched_ctx->nb_workers; i++) {
-- 
2.34.1


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH] driver: crypto: scheduler: fix session size computation
@ 2024-07-02  7:50 jhascoet
  0 siblings, 0 replies; 6+ messages in thread
From: jhascoet @ 2024-07-02  7:50 UTC (permalink / raw)
  To: dev
  Cc: jhascoet, stable, Kai Ji, Anoob Joseph, David Coyle,
	Kevin O'Sullivan, Ciara Power

The crypto scheduler session size computation was taking
into account only the worker session sizes and not its own.

Fixes: e2af4e403c1 ("crypto/scheduler: support DOCSIS security protocol")
Cc: stable@dpdk.org

Signed-off-by: Julien Hascoet <jhascoet@kalrayinc.com>
---
 drivers/crypto/scheduler/scheduler_pmd_ops.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/scheduler/scheduler_pmd_ops.c b/drivers/crypto/scheduler/scheduler_pmd_ops.c
index a18f7a08b0..6e43438469 100644
--- a/drivers/crypto/scheduler/scheduler_pmd_ops.c
+++ b/drivers/crypto/scheduler/scheduler_pmd_ops.c
@@ -185,7 +185,7 @@ scheduler_session_size_get(struct scheduler_ctx *sched_ctx,
 		uint8_t session_type)
 {
 	uint8_t i = 0;
-	uint32_t max_priv_sess_size = 0;
+	uint32_t max_priv_sess_size = sizeof(struct scheduler_session_ctx);
 
 	/* Check what is the maximum private session size for all workers */
 	for (i = 0; i < sched_ctx->nb_workers; i++) {
-- 
2.34.1


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

end of thread, other threads:[~2024-10-04  8:40 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-07-05  7:34 [PATCH] driver: crypto: scheduler: fix session size computation jhascoet
2024-10-04  8:40 ` Ji, Kai
  -- strict thread matches above, loose matches on Subject: below --
2024-07-05 12:57 jhascoet
2024-07-05  8:35 jhascoet
2024-09-30 11:33 ` Ji, Kai
2024-07-02  7:50 jhascoet

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).