[v2,2/3] security: hide internal API

Message ID 20211008204516.3497060-2-gakhil@marvell.com (mailing list archive)
State Superseded, archived
Delegated to: akhil goyal
Headers
Series [v2,1/3] cryptodev: remove LIST_END enumerators |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Akhil Goyal Oct. 8, 2021, 8:45 p.m. UTC
  rte_security_dynfield_register() is an internal
API to be used by the driver, hence moving it to internal.

Signed-off-by: Akhil Goyal <gakhil@marvell.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
---
v2: no change

 lib/security/rte_security_driver.h | 2 +-
 lib/security/version.map           | 7 ++++++-
 2 files changed, 7 insertions(+), 2 deletions(-)
  

Comments

Ray Kinsella Oct. 12, 2021, 8:50 a.m. UTC | #1
On 08/10/2021 21:45, Akhil Goyal wrote:
> rte_security_dynfield_register() is an internal
> API to be used by the driver, hence moving it to internal.
> 
> Signed-off-by: Akhil Goyal <gakhil@marvell.com>
> Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> ---
> v2: no change
> 
>  lib/security/rte_security_driver.h | 2 +-
>  lib/security/version.map           | 7 ++++++-
>  2 files changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/security/rte_security_driver.h b/lib/security/rte_security_driver.h
> index 938373205c..b0253e962e 100644
> --- a/lib/security/rte_security_driver.h
> +++ b/lib/security/rte_security_driver.h
> @@ -89,7 +89,7 @@ typedef int (*security_session_stats_get_t)(void *device,
>  		struct rte_security_session *sess,
>  		struct rte_security_stats *stats);
>  
> -__rte_experimental
> +__rte_internal
>  int rte_security_dynfield_register(void);
>  
>  /**
> diff --git a/lib/security/version.map b/lib/security/version.map
> index a1f46bfd27..edf4887e12 100644
> --- a/lib/security/version.map
> +++ b/lib/security/version.map
> @@ -16,7 +16,12 @@ EXPERIMENTAL {
>  	__rte_security_get_userdata;
>  	__rte_security_set_pkt_metadata;
>  	rte_security_dynfield_offset;
> -	rte_security_dynfield_register;
>  	rte_security_session_stats_get;
>  	rte_security_session_update;
>  };
> +
> +INTERNAL {
> +	global:
> +
> +	rte_security_dynfield_register;
> +};
> 
Acked-by: Ray Kinsella <mdr@ashroe.eu>
  

Patch

diff --git a/lib/security/rte_security_driver.h b/lib/security/rte_security_driver.h
index 938373205c..b0253e962e 100644
--- a/lib/security/rte_security_driver.h
+++ b/lib/security/rte_security_driver.h
@@ -89,7 +89,7 @@  typedef int (*security_session_stats_get_t)(void *device,
 		struct rte_security_session *sess,
 		struct rte_security_stats *stats);
 
-__rte_experimental
+__rte_internal
 int rte_security_dynfield_register(void);
 
 /**
diff --git a/lib/security/version.map b/lib/security/version.map
index a1f46bfd27..edf4887e12 100644
--- a/lib/security/version.map
+++ b/lib/security/version.map
@@ -16,7 +16,12 @@  EXPERIMENTAL {
 	__rte_security_get_userdata;
 	__rte_security_set_pkt_metadata;
 	rte_security_dynfield_offset;
-	rte_security_dynfield_register;
 	rte_security_session_stats_get;
 	rte_security_session_update;
 };
+
+INTERNAL {
+	global:
+
+	rte_security_dynfield_register;
+};