crypto/scheduler: fix session size computation

julienhascoet · Akhil Goyal · commit ce1dc500be46 · 2024-10-09T19:25:20.000+05:30
The crypto scheduler session size computation was taking into account only the worker session sizes and not its own. Fixes: e2af4e4 ("crypto/scheduler: support DOCSIS security protocol") Cc: stable@dpdk.org Signed-off-by: Julien Hascoet <ju.hascoet@gmail.com> Acked-by: Kai Ji <kai.ji@intel.com>
diff --git a/.mailmap b/.mailmap
@@ -734,6 +734,7 @@ Julien Aube <julien_dpdk@jaube.fr>
 Julien Castets <jcastets@scaleway.com>
 Julien Courtat <julien.courtat@6wind.com>
 Julien Cretin <julien.cretin@trust-in-soft.com>
+Julien Hascoet <ju.hascoet@gmail.com>
 Julien Massonneau <julien.massonneau@6wind.com>
 Julien Meunier <julien.meunier@nokia.com> <julien.meunier@6wind.com>
 Július Milan <jmilan.dev@gmail.com>
diff --git 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++) {

-Original file line number
+Diff line change
 Julien Castets <[email protected]>
 Julien Courtat <[email protected]>
 Julien Cretin <[email protected]>
 +Julien Hascoet <[email protected]>
 Julien Massonneau <[email protected]>
 Julien Meunier <[email protected]> <[email protected]>
 Július Milan <[email protected]>
Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ scheduler_session_size_get(struct scheduler_ctx *sched_ctx,`
`185`	`185`	`uint8_t session_type)`
`186`	`186`	`{`
`187`	`187`	`uint8_t i = 0;`
`188`		`- uint32_t max_priv_sess_size = 0;`
	`188`	`+ uint32_t max_priv_sess_size = sizeof(struct scheduler_session_ctx);`
`189`	`189`
`190`	`190`	`/* Check what is the maximum private session size for all workers */`
`191`	`191`	`for (i = 0; i < sched_ctx->nb_workers; i++) {`