[v1,4/6] crypto/ccp: updating ccp crypto capabilities ops

Message ID 20191015070144.93172-1-asomalap@amd.com (mailing list archive)
State Changes Requested, archived
Delegated to: akhil goyal
Headers
Series [v1,1/6] doc: documentation update |

Checks

Context Check Description
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

AMARANATH SOMALAPURAM Oct. 15, 2019, 7:02 a.m. UTC
  From: Amaranath Somalapuram <asomalap@amd.com>

crypto capabilities ops for ccp initilzed incorrectly.
changing the correct values for the capabilities ops
Cc: stable@dpdk.org

Signed-off-by: Amaranath Somalapuram <asomalap@amd.com>
---
 drivers/crypto/ccp/ccp_pmd_ops.c | 44 ++++++++++++++++----------------
 1 file changed, 22 insertions(+), 22 deletions(-)
  

Comments

Akhil Goyal Oct. 15, 2019, 11:06 a.m. UTC | #1
Title: crypto/ccp: fix capabilities for digest size

> 
> From: Amaranath Somalapuram <asomalap@amd.com>
> 
> crypto capabilities ops for ccp initilzed incorrectly.
> changing the correct values for the capabilities ops
You are changing digest size only in this patch.
You can rephrase this as:

CCP can support varied digest sizes ranging from 1 to some max value
But the current code support only fixed max values.
This patch updates the minimum digest sizes to 1.

Also a Fixes line is missing

> Cc: stable@dpdk.org
> 
> Signed-off-by: Amaranath Somalapuram <asomalap@amd.com>
> ---
>  drivers/crypto/ccp/ccp_pmd_ops.c | 44 ++++++++++++++++----------------
>  1 file changed, 22 insertions(+), 22 deletions(-)
> 
> diff --git a/drivers/crypto/ccp/ccp_pmd_ops.c
> b/drivers/crypto/ccp/ccp_pmd_ops.c
> index d5041f0ec..768378033 100644
> --- a/drivers/crypto/ccp/ccp_pmd_ops.c
> +++ b/drivers/crypto/ccp/ccp_pmd_ops.c
> @@ -47,7 +47,7 @@
>  					 .increment = 1                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 20,                     \
> +					 .min = 1,                      \
>  					 .max = 20,                     \
>  					 .increment = 0                 \
>  				 },                                     \
> @@ -68,9 +68,9 @@
>  					 .increment = 0                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 28,                     \
> +					 .min = 1,                      \
>  					 .max = 28,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			 }, }                                           \
> @@ -89,9 +89,9 @@
>  					 .increment = 1                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 28,                     \
> +					 .min = 1,                      \
>  					 .max = 28,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			 }, }                                           \
> @@ -110,9 +110,9 @@
>  					 .increment = 0                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 28,                     \
> +					 .min = 1,                      \
>  					 .max = 28,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			 }, }                                           \
> @@ -131,9 +131,9 @@
>  					 .increment = 1                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 28,                     \
> +					 .min = 1,                      \
>  					 .max = 28,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			 }, }                                           \
> @@ -173,9 +173,9 @@
>  					 .increment = 1                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 32,                     \
> +					 .min = 1,                      \
>  					 .max = 32,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			 }, }                                           \
> @@ -215,9 +215,9 @@
>  					 .increment = 1                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 32,                     \
> +					 .min = 1,                      \
>  					 .max = 32,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			 }, }						\
> @@ -257,9 +257,9 @@
>  					 .increment = 1                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 48,                     \
> +					 .min = 1,                      \
>  					 .max = 48,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			 }, }                                           \
> @@ -299,9 +299,9 @@
>  					 .increment = 1                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 48,                     \
> +					 .min = 1,                      \
>  					 .max = 48,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			 }, }                                           \
> @@ -341,9 +341,9 @@
>  					 .increment = 1                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 64,                     \
> +					 .min = 1,                      \
>  					 .max = 64,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			 }, }                                           \
> @@ -383,9 +383,9 @@
>  					 .increment = 1                 \
>  				 },                                     \
>  				 .digest_size = {                       \
> -					 .min = 64,                     \
> +					 .min = 1,                      \
>  					 .max = 64,                     \
> -					 .increment = 0                 \
> +					 .increment = 1                 \
>  				 },                                     \
>  				 .aad_size = { 0 }                      \
>  			}, }                                            \
> @@ -536,7 +536,7 @@
>  					 .increment = 1			\
>  				 },					\
>  				 .digest_size = {			\
> -					 .min = 16,			\
> +					 .min = 1,			\
>  					 .max = 16,			\
>  					 .increment = 0			\
>  				 },					\
> --
> 2.17.1
  

Patch

diff --git a/drivers/crypto/ccp/ccp_pmd_ops.c b/drivers/crypto/ccp/ccp_pmd_ops.c
index d5041f0ec..768378033 100644
--- a/drivers/crypto/ccp/ccp_pmd_ops.c
+++ b/drivers/crypto/ccp/ccp_pmd_ops.c
@@ -47,7 +47,7 @@ 
 					 .increment = 1                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 20,                     \
+					 .min = 1,                      \
 					 .max = 20,                     \
 					 .increment = 0                 \
 				 },                                     \
@@ -68,9 +68,9 @@ 
 					 .increment = 0                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 28,                     \
+					 .min = 1,                      \
 					 .max = 28,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			 }, }                                           \
@@ -89,9 +89,9 @@ 
 					 .increment = 1                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 28,                     \
+					 .min = 1,                      \
 					 .max = 28,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			 }, }                                           \
@@ -110,9 +110,9 @@ 
 					 .increment = 0                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 28,                     \
+					 .min = 1,                      \
 					 .max = 28,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			 }, }                                           \
@@ -131,9 +131,9 @@ 
 					 .increment = 1                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 28,                     \
+					 .min = 1,                      \
 					 .max = 28,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			 }, }                                           \
@@ -173,9 +173,9 @@ 
 					 .increment = 1                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 32,                     \
+					 .min = 1,                      \
 					 .max = 32,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			 }, }                                           \
@@ -215,9 +215,9 @@ 
 					 .increment = 1                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 32,                     \
+					 .min = 1,                      \
 					 .max = 32,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			 }, }						\
@@ -257,9 +257,9 @@ 
 					 .increment = 1                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 48,                     \
+					 .min = 1,                      \
 					 .max = 48,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			 }, }                                           \
@@ -299,9 +299,9 @@ 
 					 .increment = 1                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 48,                     \
+					 .min = 1,                      \
 					 .max = 48,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			 }, }                                           \
@@ -341,9 +341,9 @@ 
 					 .increment = 1                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 64,                     \
+					 .min = 1,                      \
 					 .max = 64,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			 }, }                                           \
@@ -383,9 +383,9 @@ 
 					 .increment = 1                 \
 				 },                                     \
 				 .digest_size = {                       \
-					 .min = 64,                     \
+					 .min = 1,                      \
 					 .max = 64,                     \
-					 .increment = 0                 \
+					 .increment = 1                 \
 				 },                                     \
 				 .aad_size = { 0 }                      \
 			}, }                                            \
@@ -536,7 +536,7 @@ 
 					 .increment = 1			\
 				 },					\
 				 .digest_size = {			\
-					 .min = 16,			\
+					 .min = 1,			\
 					 .max = 16,			\
 					 .increment = 0			\
 				 },					\