security: fix comments

Message ID 1651039306-149-1-git-send-email-anoobj@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: akhil goyal
Headers
Series security: fix comments |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-abi-testing success Testing PASS

Commit Message

Anoob Joseph April 27, 2022, 6:01 a.m. UTC
  Fix comments to reflect the field.

Fixes: ad7515a39f2a ("security: add SA lifetime configuration")
Cc: anoobj@marvell.com

Reported-by: Thomas Monjalon <thomas@monjalon.net>
Signed-off-by: Anoob Joseph <anoobj@marvell.com>
---
 lib/security/rte_security.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Morten Brørup April 27, 2022, 8:20 a.m. UTC | #1
> From: Anoob Joseph [mailto:anoobj@marvell.com]
> Sent: Wednesday, 27 April 2022 08.02
> 
> Fix comments to reflect the field.
> 
> Fixes: ad7515a39f2a ("security: add SA lifetime configuration")
> Cc: anoobj@marvell.com
> 
> Reported-by: Thomas Monjalon <thomas@monjalon.net>
> Signed-off-by: Anoob Joseph <anoobj@marvell.com>
> ---
>  lib/security/rte_security.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/security/rte_security.h b/lib/security/rte_security.h
> index 2a43cbe..675db94 100644
> --- a/lib/security/rte_security.h
> +++ b/lib/security/rte_security.h
> @@ -311,9 +311,9 @@ struct rte_security_ipsec_lifetime {
>  	uint64_t bytes_soft_limit;
>  	/**< Soft expiry limit in bytes */
>  	uint64_t packets_hard_limit;
> -	/**< Soft expiry limit in number of packets */
> +	/**< Hard expiry limit in number of packets */
>  	uint64_t bytes_hard_limit;
> -	/**< Soft expiry limit in bytes */
> +	/**< Hard expiry limit in bytes */
>  };
> 
>  /**
> --
> 2.7.4
> 

Reviewed-by: Morten Brørup <mb@smartsharesystems.com>
  
Akhil Goyal April 28, 2022, 7:28 a.m. UTC | #2
> Subject: [PATCH] security: fix comments
> 
> Fix comments to reflect the field.
> 
> Fixes: ad7515a39f2a ("security: add SA lifetime configuration")
> Cc: anoobj@marvell.com
> 
> Reported-by: Thomas Monjalon <thomas@monjalon.net>
> Signed-off-by: Anoob Joseph <anoobj@marvell.com>
Applied to dpdk-next-crypto

Cc: stable@dpdk.org
  

Patch

diff --git a/lib/security/rte_security.h b/lib/security/rte_security.h
index 2a43cbe..675db94 100644
--- a/lib/security/rte_security.h
+++ b/lib/security/rte_security.h
@@ -311,9 +311,9 @@  struct rte_security_ipsec_lifetime {
 	uint64_t bytes_soft_limit;
 	/**< Soft expiry limit in bytes */
 	uint64_t packets_hard_limit;
-	/**< Soft expiry limit in number of packets */
+	/**< Hard expiry limit in number of packets */
 	uint64_t bytes_hard_limit;
-	/**< Soft expiry limit in bytes */
+	/**< Hard expiry limit in bytes */
 };
 
 /**