* [dpdk-dev] [PATCH] doc: security deprecation notice for session changes
@ 2018-11-14 11:23 Konstantin Ananyev
2018-11-14 11:32 ` Mohammad Abdul Awal
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Konstantin Ananyev @ 2018-11-14 11:23 UTC (permalink / raw)
To: dev; +Cc: akhil.goyal, declan.doherty, Konstantin Ananyev
Add 'uint64_t opaque_data' inside struct rte_security_session.
That allows upper layer to easily associate some user defined
data with the session.
Proposed new layout for:
struct rte_security_session {
void *sess_private_data;
/**< Private session material */
+ uint64_t opaque_data;
+ /**< Opaque user defined data */
};
Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
---
doc/guides/rel_notes/deprecation.rst | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 34b28234c..0cdc42842 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -55,3 +55,9 @@ Deprecation Notices
- ``rte_pdump_set_socket_dir`` will be removed;
- The parameter, ``path``, of ``rte_pdump_init`` will be removed;
- The enum ``rte_pdump_socktype`` will be removed.
+
+* security: ABI change:
+
+ New field ``uint64_t opaque_data`` is planned to add into
+ ``rte_security_session`` structure. That would allow upper layer to easily
+ associate/de-associate some user defined data with the security session.
--
2.17.1
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH] doc: security deprecation notice for session changes
2018-11-14 11:23 [dpdk-dev] [PATCH] doc: security deprecation notice for session changes Konstantin Ananyev
@ 2018-11-14 11:32 ` Mohammad Abdul Awal
2018-11-14 11:32 ` Zhang, Roy Fan
2018-11-14 12:39 ` Akhil Goyal
2 siblings, 0 replies; 6+ messages in thread
From: Mohammad Abdul Awal @ 2018-11-14 11:32 UTC (permalink / raw)
To: Konstantin Ananyev, dev; +Cc: akhil.goyal, declan.doherty
On 14/11/2018 11:23, Konstantin Ananyev wrote:
> Add 'uint64_t opaque_data' inside struct rte_security_session.
> That allows upper layer to easily associate some user defined
> data with the session.
> Proposed new layout for:
> struct rte_security_session {
> void *sess_private_data;
> /**< Private session material */
> + uint64_t opaque_data;
> + /**< Opaque user defined data */
> };
>
> Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> ---
> doc/guides/rel_notes/deprecation.rst | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 34b28234c..0cdc42842 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -55,3 +55,9 @@ Deprecation Notices
> - ``rte_pdump_set_socket_dir`` will be removed;
> - The parameter, ``path``, of ``rte_pdump_init`` will be removed;
> - The enum ``rte_pdump_socktype`` will be removed.
> +
> +* security: ABI change:
> +
> + New field ``uint64_t opaque_data`` is planned to add into
> + ``rte_security_session`` structure. That would allow upper layer to easily
> + associate/de-associate some user defined data with the security session.
Acked-by: Mohammad Abdul Awal <mohammad.abdul.awal@intel.com>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH] doc: security deprecation notice for session changes
2018-11-14 11:23 [dpdk-dev] [PATCH] doc: security deprecation notice for session changes Konstantin Ananyev
2018-11-14 11:32 ` Mohammad Abdul Awal
@ 2018-11-14 11:32 ` Zhang, Roy Fan
2018-11-14 12:39 ` Akhil Goyal
2 siblings, 0 replies; 6+ messages in thread
From: Zhang, Roy Fan @ 2018-11-14 11:32 UTC (permalink / raw)
To: Ananyev, Konstantin, dev
Cc: akhil.goyal, Doherty, Declan, Ananyev, Konstantin
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Konstantin
> Ananyev
> Sent: Wednesday, November 14, 2018 11:24 AM
> To: dev@dpdk.org
> Cc: akhil.goyal@nxp.com; Doherty, Declan <declan.doherty@intel.com>;
> Ananyev, Konstantin <konstantin.ananyev@intel.com>
> Subject: [dpdk-dev] [PATCH] doc: security deprecation notice for session
> changes
>
Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] doc: security deprecation notice for session changes
2018-11-14 11:23 [dpdk-dev] [PATCH] doc: security deprecation notice for session changes Konstantin Ananyev
2018-11-14 11:32 ` Mohammad Abdul Awal
2018-11-14 11:32 ` Zhang, Roy Fan
@ 2018-11-14 12:39 ` Akhil Goyal
2018-11-14 13:02 ` Ananyev, Konstantin
2018-11-24 17:59 ` Thomas Monjalon
2 siblings, 2 replies; 6+ messages in thread
From: Akhil Goyal @ 2018-11-14 12:39 UTC (permalink / raw)
To: Konstantin Ananyev, dev; +Cc: declan.doherty
On 11/14/2018 4:53 PM, Konstantin Ananyev wrote:
> Add 'uint64_t opaque_data' inside struct rte_security_session.
> That allows upper layer to easily associate some user defined
> data with the session.
> Proposed new layout for:
> struct rte_security_session {
> void *sess_private_data;
> /**< Private session material */
> + uint64_t opaque_data;
> + /**< Opaque user defined data */
> };
>
> Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> ---
Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
Does this also mean you have given the Ack for removing the experimental
tag from security library? Because otherwise there is no point of this
deprecation notice if the library is not formal.
> doc/guides/rel_notes/deprecation.rst | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 34b28234c..0cdc42842 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -55,3 +55,9 @@ Deprecation Notices
> - ``rte_pdump_set_socket_dir`` will be removed;
> - The parameter, ``path``, of ``rte_pdump_init`` will be removed;
> - The enum ``rte_pdump_socktype`` will be removed.
> +
> +* security: ABI change:
> +
> + New field ``uint64_t opaque_data`` is planned to add into
> + ``rte_security_session`` structure. That would allow upper layer to easily
> + associate/de-associate some user defined data with the security session.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH] doc: security deprecation notice for session changes
2018-11-14 12:39 ` Akhil Goyal
@ 2018-11-14 13:02 ` Ananyev, Konstantin
2018-11-24 17:59 ` Thomas Monjalon
1 sibling, 0 replies; 6+ messages in thread
From: Ananyev, Konstantin @ 2018-11-14 13:02 UTC (permalink / raw)
To: Akhil Goyal, dev; +Cc: Doherty, Declan
> -----Original Message-----
> From: Akhil Goyal [mailto:akhil.goyal@nxp.com]
> Sent: Wednesday, November 14, 2018 12:40 PM
> To: Ananyev, Konstantin <konstantin.ananyev@intel.com>; dev@dpdk.org
> Cc: Doherty, Declan <declan.doherty@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] doc: security deprecation notice for session changes
>
>
>
> On 11/14/2018 4:53 PM, Konstantin Ananyev wrote:
> > Add 'uint64_t opaque_data' inside struct rte_security_session.
> > That allows upper layer to easily associate some user defined
> > data with the session.
> > Proposed new layout for:
> > struct rte_security_session {
> > void *sess_private_data;
> > /**< Private session material */
> > + uint64_t opaque_data;
> > + /**< Opaque user defined data */
> > };
> >
> > Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > ---
> Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
>
> Does this also mean you have given the Ack for removing the experimental
> tag from security library? Because otherwise there is no point of this
> deprecation notice if the library is not formal.
For the whole library - yes.
Though I still suggest to keep 'experimental' for non-implemented functions
(rte_security_get_userdata()).
Hope that wouldn't block you guys.
Konstantin
> > doc/guides/rel_notes/deprecation.rst | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> > index 34b28234c..0cdc42842 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -55,3 +55,9 @@ Deprecation Notices
> > - ``rte_pdump_set_socket_dir`` will be removed;
> > - The parameter, ``path``, of ``rte_pdump_init`` will be removed;
> > - The enum ``rte_pdump_socktype`` will be removed.
> > +
> > +* security: ABI change:
> > +
> > + New field ``uint64_t opaque_data`` is planned to add into
> > + ``rte_security_session`` structure. That would allow upper layer to easily
> > + associate/de-associate some user defined data with the security session.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH] doc: security deprecation notice for session changes
2018-11-14 12:39 ` Akhil Goyal
2018-11-14 13:02 ` Ananyev, Konstantin
@ 2018-11-24 17:59 ` Thomas Monjalon
1 sibling, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2018-11-24 17:59 UTC (permalink / raw)
To: Konstantin Ananyev; +Cc: dev, Akhil Goyal, declan.doherty
14/11/2018 13:39, Akhil Goyal:
> On 11/14/2018 4:53 PM, Konstantin Ananyev wrote:
> > Add 'uint64_t opaque_data' inside struct rte_security_session.
> > That allows upper layer to easily associate some user defined
> > data with the session.
> > Proposed new layout for:
> > struct rte_security_session {
> > void *sess_private_data;
> > /**< Private session material */
> > + uint64_t opaque_data;
> > + /**< Opaque user defined data */
> > };
> >
> > Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > ---
> Acked-by: Mohammad Abdul Awal <mohammad.abdul.awal@intel.com>
> Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
Applied
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-11-24 17:59 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-14 11:23 [dpdk-dev] [PATCH] doc: security deprecation notice for session changes Konstantin Ananyev
2018-11-14 11:32 ` Mohammad Abdul Awal
2018-11-14 11:32 ` Zhang, Roy Fan
2018-11-14 12:39 ` Akhil Goyal
2018-11-14 13:02 ` Ananyev, Konstantin
2018-11-24 17:59 ` Thomas Monjalon
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).