DPDK-dev Archive on lore.kernel.org
 help / color / Atom feed
* [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw
@ 2019-07-11  9:05 Bernard Iremonger
  2019-07-19 11:23 ` Ananyev, Konstantin
                   ` (3 more replies)
  0 siblings, 4 replies; 23+ messages in thread
From: Bernard Iremonger @ 2019-07-11  9:05 UTC (permalink / raw)
  To: dev, konstantin.ananyev, akhil.goyal; +Cc: Bernard Iremonger

All the functionality of the legacy code path in now available in
the librte_ipsec library. It is planned to deprecate the legacy
code path in the 19.11 release and remove the legacy code path in
the 20.02 release.

Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
---
 doc/guides/rel_notes/deprecation.rst | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index a7796f4..258fee8 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -82,3 +82,8 @@ Deprecation Notices
   to set new power environment if power environment was already initialized.
   In this case the function will return -1 unless the environment is unset first
   (using ``rte_power_unset_env``). Other function usage scenarios will not change.
+
+* ipsec-secgw: All the ipsec-secgw legacy code path functionality is now
+  available in the librte_ipsec library. It is planned to deprecate the legacy
+  code path in the ipsec-secgw application in the 19.11 release and remove it
+  in the 20.02 release.
-- 
2.7.4


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw
  2019-07-11  9:05 [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw Bernard Iremonger
@ 2019-07-19 11:23 ` Ananyev, Konstantin
  2019-07-19 14:55 ` Zhang, Roy Fan
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 23+ messages in thread
From: Ananyev, Konstantin @ 2019-07-19 11:23 UTC (permalink / raw)
  To: Iremonger, Bernard, dev, akhil.goyal

> 
> All the functionality of the legacy code path in now available in
> the librte_ipsec library. It is planned to deprecate the legacy
> code path in the 19.11 release and remove the legacy code path in
> the 20.02 release.
> 
> Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> ---
>  doc/guides/rel_notes/deprecation.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index a7796f4..258fee8 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -82,3 +82,8 @@ Deprecation Notices
>    to set new power environment if power environment was already initialized.
>    In this case the function will return -1 unless the environment is unset first
>    (using ``rte_power_unset_env``). Other function usage scenarios will not change.
> +
> +* ipsec-secgw: All the ipsec-secgw legacy code path functionality is now
> +  available in the librte_ipsec library. It is planned to deprecate the legacy
> +  code path in the ipsec-secgw application in the 19.11 release and remove it
> +  in the 20.02 release.
> --

Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>

> 2.7.4


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw
  2019-07-11  9:05 [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw Bernard Iremonger
  2019-07-19 11:23 ` Ananyev, Konstantin
@ 2019-07-19 14:55 ` Zhang, Roy Fan
  2019-07-24  9:47 ` Akhil Goyal
  2019-07-24 10:12 ` [dpdk-dev] [PATCH] " Bernard Iremonger
  3 siblings, 0 replies; 23+ messages in thread
From: Zhang, Roy Fan @ 2019-07-19 14:55 UTC (permalink / raw)
  To: Iremonger, Bernard, dev, Ananyev, Konstantin, akhil.goyal
  Cc: Iremonger, Bernard


> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Bernard Iremonger
> Sent: Thursday, July 11, 2019 6:06 PM
> To: dev@dpdk.org; Ananyev, Konstantin <konstantin.ananyev@intel.com>;
> akhil.goyal@nxp.com
> Cc: Iremonger, Bernard <bernard.iremonger@intel.com>
> Subject: [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw
> 
> All the functionality of the legacy code path in now available in the
> librte_ipsec library. It is planned to deprecate the legacy code path in the
> 19.11 release and remove the legacy code path in the 20.02 release.
> 
> Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> ---
>  doc/guides/rel_notes/deprecation.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> index a7796f4..258fee8 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -82,3 +82,8 @@ Deprecation Notices
>    to set new power environment if power environment was already
> initialized.
>    In this case the function will return -1 unless the environment is unset first
>    (using ``rte_power_unset_env``). Other function usage scenarios will not
> change.
> +
> +* ipsec-secgw: All the ipsec-secgw legacy code path functionality is
> +now
> +  available in the librte_ipsec library. It is planned to deprecate the
> +legacy
> +  code path in the ipsec-secgw application in the 19.11 release and
> +remove it
> +  in the 20.02 release.
> --
> 2.7.4

Acked-by: Fan Zhang <roy.fan.zhang@intel.com>

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw
  2019-07-11  9:05 [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw Bernard Iremonger
  2019-07-19 11:23 ` Ananyev, Konstantin
  2019-07-19 14:55 ` Zhang, Roy Fan
@ 2019-07-24  9:47 ` Akhil Goyal
  2019-07-24 10:12 ` [dpdk-dev] [PATCH] " Bernard Iremonger
  3 siblings, 0 replies; 23+ messages in thread
From: Akhil Goyal @ 2019-07-24  9:47 UTC (permalink / raw)
  To: Bernard Iremonger, dev, konstantin.ananyev

Hi Bernard,

> All the functionality of the legacy code path in now available in
> the librte_ipsec library. It is planned to deprecate the legacy
> code path in the 19.11 release and remove the legacy code path in
> the 20.02 release.
> 
> Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> ---
>  doc/guides/rel_notes/deprecation.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> index a7796f4..258fee8 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -82,3 +82,8 @@ Deprecation Notices
>    to set new power environment if power environment was already initialized.
>    In this case the function will return -1 unless the environment is unset first
>    (using ``rte_power_unset_env``). Other function usage scenarios will not
> change.
> +
> +* ipsec-secgw: All the ipsec-secgw legacy code path functionality is now
> +  available in the librte_ipsec library. It is planned to deprecate the legacy
> +  code path in the ipsec-secgw application in the 19.11 release and remove it
> +  in the 20.02 release.
> --
Library mode path was impacting the performance on NXP platforms for lookaside mode at the time the library was introduced. Not sure what is the state today. We will evaluate the performance numbers when the actual code will be submitted for review. If the numbers are good we can remove the legacy code path. Apart from this concern,
Acked-by: Akhil Goyal <akhil.goyal@nxp.com>

This patch is an RFC patch and generally RFC patches are not merged.
Please send the patch again without the RFC tag, if this patch need to be integrated in this release.
You can take the Acks.

Regards,
Akhil


^ permalink raw reply	[flat|nested] 23+ messages in thread

* [dpdk-dev] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-11  9:05 [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw Bernard Iremonger
                   ` (2 preceding siblings ...)
  2019-07-24  9:47 ` Akhil Goyal
@ 2019-07-24 10:12 ` " Bernard Iremonger
  2019-07-24 10:14   ` Akhil Goyal
  2019-07-26 14:45   ` [dpdk-dev] [EXT] " Anoob Joseph
  3 siblings, 2 replies; 23+ messages in thread
From: Bernard Iremonger @ 2019-07-24 10:12 UTC (permalink / raw)
  To: dev, konstantin.ananyev, akhil.goyal; +Cc: Bernard Iremonger

All the functionality of the legacy code path in now available in
the librte_ipsec library. It is planned to deprecate the legacy
code path in the 19.11 release and remove the legacy code path in
the 20.02 release.

Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
---
 doc/guides/rel_notes/deprecation.rst | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 37b8592..8efe234 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -78,3 +78,8 @@ Deprecation Notices
   to set new power environment if power environment was already initialized.
   In this case the function will return -1 unless the environment is unset first
   (using ``rte_power_unset_env``). Other function usage scenarios will not change.
+
+* ipsec-secgw: All the ipsec-secgw legacy code path functionality is now
+  available in the librte_ipsec library. It is planned to deprecate the legacy
+  code path in the ipsec-secgw application in the 19.11 release and remove it
+  in the 20.02 release.
-- 
2.7.4


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-24 10:12 ` [dpdk-dev] [PATCH] " Bernard Iremonger
@ 2019-07-24 10:14   ` Akhil Goyal
  2019-07-26 14:06     ` Akhil Goyal
  2019-07-26 14:45   ` [dpdk-dev] [EXT] " Anoob Joseph
  1 sibling, 1 reply; 23+ messages in thread
From: Akhil Goyal @ 2019-07-24 10:14 UTC (permalink / raw)
  To: Bernard Iremonger, dev, konstantin.ananyev, Anoob Joseph, Jerin Jacob

+Anoob, Jerin

> 
> All the functionality of the legacy code path in now available in
> the librte_ipsec library. It is planned to deprecate the legacy
> code path in the 19.11 release and remove the legacy code path in
> the 20.02 release.
> 
> Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> ---
>  doc/guides/rel_notes/deprecation.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> index 37b8592..8efe234 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -78,3 +78,8 @@ Deprecation Notices
>    to set new power environment if power environment was already initialized.
>    In this case the function will return -1 unless the environment is unset first
>    (using ``rte_power_unset_env``). Other function usage scenarios will not
> change.
> +
> +* ipsec-secgw: All the ipsec-secgw legacy code path functionality is now
> +  available in the librte_ipsec library. It is planned to deprecate the legacy
> +  code path in the ipsec-secgw application in the 19.11 release and remove it
> +  in the 20.02 release.
> --
> 2.7.4


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-24 10:14   ` Akhil Goyal
@ 2019-07-26 14:06     ` Akhil Goyal
  0 siblings, 0 replies; 23+ messages in thread
From: Akhil Goyal @ 2019-07-26 14:06 UTC (permalink / raw)
  To: Akhil Goyal, Bernard Iremonger, dev, konstantin.ananyev,
	Anoob Joseph, Jerin Jacob

Hi Anoob,

Could you please Ack this patch if there is no issue?

Thanks,
Akhil

> +Anoob, Jerin
> 
> >
> > All the functionality of the legacy code path in now available in
> > the librte_ipsec library. It is planned to deprecate the legacy
> > code path in the 19.11 release and remove the legacy code path in
> > the 20.02 release.
> >
> > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/doc/guides/rel_notes/deprecation.rst
> > b/doc/guides/rel_notes/deprecation.rst
> > index 37b8592..8efe234 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -78,3 +78,8 @@ Deprecation Notices
> >    to set new power environment if power environment was already initialized.
> >    In this case the function will return -1 unless the environment is unset first
> >    (using ``rte_power_unset_env``). Other function usage scenarios will not
> > change.
> > +
> > +* ipsec-secgw: All the ipsec-secgw legacy code path functionality is now
> > +  available in the librte_ipsec library. It is planned to deprecate the legacy
> > +  code path in the ipsec-secgw application in the 19.11 release and remove it
> > +  in the 20.02 release.
> > --
> > 2.7.4


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-24 10:12 ` [dpdk-dev] [PATCH] " Bernard Iremonger
  2019-07-24 10:14   ` Akhil Goyal
@ 2019-07-26 14:45   ` " Anoob Joseph
  2019-07-29  8:49     ` Akhil Goyal
  1 sibling, 1 reply; 23+ messages in thread
From: Anoob Joseph @ 2019-07-26 14:45 UTC (permalink / raw)
  To: Bernard Iremonger, dev, konstantin.ananyev, akhil.goyal
  Cc: Jerin Jacob Kollanukkaran, Narayana Prasad Raju Athreya

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Bernard Iremonger
> Sent: Wednesday, July 24, 2019 3:43 PM
> To: dev@dpdk.org; konstantin.ananyev@intel.com; akhil.goyal@nxp.com
> Cc: Bernard Iremonger <bernard.iremonger@intel.com>
> Subject: [EXT] [dpdk-dev] [PATCH] doc: deprecate legacy code path in ipsec-
> secgw
> 
> External Email
> 
> ----------------------------------------------------------------------
> All the functionality of the legacy code path in now available in the librte_ipsec
> library. It is planned to deprecate the legacy code path in the 19.11 release and
> remove the legacy code path in the 20.02 release.
> 
> Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> ---
>  doc/guides/rel_notes/deprecation.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
Acked-by: Anoob Joseph <anoobj@marvell.com>

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-26 14:45   ` [dpdk-dev] [EXT] " Anoob Joseph
@ 2019-07-29  8:49     ` Akhil Goyal
  2019-07-29 16:33       ` Thomas Monjalon
  0 siblings, 1 reply; 23+ messages in thread
From: Akhil Goyal @ 2019-07-29  8:49 UTC (permalink / raw)
  To: Anoob Joseph, Bernard Iremonger, dev, konstantin.ananyev
  Cc: Jerin Jacob Kollanukkaran, Narayana Prasad Raju Athreya



> >
> > ----------------------------------------------------------------------
> > All the functionality of the legacy code path in now available in the librte_ipsec
> > library. It is planned to deprecate the legacy code path in the 19.11 release and
> > remove the legacy code path in the 20.02 release.
> >
> > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> Acked-by: Anoob Joseph <anoobj@marvell.com>

Applied to dpdk-next-crypto

Thanks.

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-29  8:49     ` Akhil Goyal
@ 2019-07-29 16:33       ` Thomas Monjalon
  2019-07-29 20:07         ` Ananyev, Konstantin
  2019-07-30  5:55         ` Akhil Goyal
  0 siblings, 2 replies; 23+ messages in thread
From: Thomas Monjalon @ 2019-07-29 16:33 UTC (permalink / raw)
  To: Bernard Iremonger
  Cc: dev, Akhil Goyal, Anoob Joseph, konstantin.ananyev,
	Jerin Jacob Kollanukkaran, Narayana Prasad Raju Athreya

> > > All the functionality of the legacy code path in now available in the librte_ipsec
> > > library. It is planned to deprecate the legacy code path in the 19.11 release and
> > > remove the legacy code path in the 20.02 release.
> > >
> > > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > ---
> > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > >  1 file changed, 5 insertions(+)
> > >
> > Acked-by: Anoob Joseph <anoobj@marvell.com>
> 
> Applied to dpdk-next-crypto

Why do we have a deprecation notice for some code path in an example?
The deprecation notices are for the API.

I think you can drop the legacy code in 19.11,
and I don't merge this patch in master.



^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-29 16:33       ` Thomas Monjalon
@ 2019-07-29 20:07         ` Ananyev, Konstantin
  2019-07-30  5:55         ` Akhil Goyal
  1 sibling, 0 replies; 23+ messages in thread
From: Ananyev, Konstantin @ 2019-07-29 20:07 UTC (permalink / raw)
  To: Thomas Monjalon, Iremonger, Bernard
  Cc: dev, Akhil Goyal, Anoob Joseph, Jerin Jacob Kollanukkaran,
	Narayana Prasad Raju Athreya



> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas@monjalon.net]
> Sent: Monday, July 29, 2019 5:34 PM
> To: Iremonger, Bernard <bernard.iremonger@intel.com>
> Cc: dev@dpdk.org; Akhil Goyal <akhil.goyal@nxp.com>; Anoob Joseph <anoobj@marvell.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Jerin Jacob Kollanukkaran <jerinj@marvell.com>; Narayana Prasad Raju Athreya
> <pathreya@marvell.com>
> Subject: Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
> 
> > > > All the functionality of the legacy code path in now available in the librte_ipsec
> > > > library. It is planned to deprecate the legacy code path in the 19.11 release and
> > > > remove the legacy code path in the 20.02 release.
> > > >
> > > > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > ---
> > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > >  1 file changed, 5 insertions(+)
> > > >
> > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> >
> > Applied to dpdk-next-crypto
> 
> Why do we have a deprecation notice for some code path in an example?
> The deprecation notices are for the API.
> 
> I think you can drop the legacy code in 19.11,
> and I don't merge this patch in master.

Ok, if we don't need a formal deprecation note for that - even better.
Thanks
Konstantin


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-29 16:33       ` Thomas Monjalon
  2019-07-29 20:07         ` Ananyev, Konstantin
@ 2019-07-30  5:55         ` Akhil Goyal
  2019-07-30  7:09           ` Thomas Monjalon
  1 sibling, 1 reply; 23+ messages in thread
From: Akhil Goyal @ 2019-07-30  5:55 UTC (permalink / raw)
  To: Thomas Monjalon, Bernard Iremonger
  Cc: dev, Anoob Joseph, konstantin.ananyev, Jerin Jacob Kollanukkaran,
	Narayana Prasad Raju Athreya


> 
> > > > All the functionality of the legacy code path in now available in the
> librte_ipsec
> > > > library. It is planned to deprecate the legacy code path in the 19.11 release
> and
> > > > remove the legacy code path in the 20.02 release.
> > > >
> > > > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > ---
> > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > >  1 file changed, 5 insertions(+)
> > > >
> > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> >
> > Applied to dpdk-next-crypto
> 
> Why do we have a deprecation notice for some code path in an example?
> The deprecation notices are for the API.
> 
> I think you can drop the legacy code in 19.11,
> and I don't merge this patch in master.
> 

We are planning to remove the original code and replace it with IPSec library APIs which are still experimental. With this change there won't be any example of the legacy ipsec code path. Applications over DPDK take ipsec-secgw as an example and IPSec is a major use case for customers. There may also be performance differences in the two code paths. Atleast on NXP platforms I saw 5-7% drop when the patches were originally submitted. Not sure what is the current state.
I feel it is worth notifying the users that the original codepath is getting deprecated, so that they can plan to move to new IPSec APIs.

-Akhil


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-30  5:55         ` Akhil Goyal
@ 2019-07-30  7:09           ` Thomas Monjalon
  2019-07-30  7:20             ` Akhil Goyal
  0 siblings, 1 reply; 23+ messages in thread
From: Thomas Monjalon @ 2019-07-30  7:09 UTC (permalink / raw)
  To: Akhil Goyal
  Cc: Bernard Iremonger, dev, Anoob Joseph, konstantin.ananyev,
	Jerin Jacob Kollanukkaran, Narayana Prasad Raju Athreya

30/07/2019 07:55, Akhil Goyal:
> 
> > 
> > > > > All the functionality of the legacy code path in now available in the
> > librte_ipsec
> > > > > library. It is planned to deprecate the legacy code path in the 19.11 release
> > and
> > > > > remove the legacy code path in the 20.02 release.
> > > > >
> > > > > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > > > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > ---
> > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > >  1 file changed, 5 insertions(+)
> > > > >
> > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > >
> > > Applied to dpdk-next-crypto
> > 
> > Why do we have a deprecation notice for some code path in an example?
> > The deprecation notices are for the API.
> > 
> > I think you can drop the legacy code in 19.11,
> > and I don't merge this patch in master.
> > 
> 
> We are planning to remove the original code and replace it with IPSec library APIs which are still experimental. With this change there won't be any example of the legacy ipsec code path. Applications over DPDK take ipsec-secgw as an example and IPSec is a major use case for customers. There may also be performance differences in the two code paths. Atleast on NXP platforms I saw 5-7% drop when the patches were originally submitted. Not sure what is the current state.
> I feel it is worth notifying the users that the original codepath is getting deprecated, so that they can plan to move to new IPSec APIs.

The deprecation notice is not the right place for a change in an example.
What change is there in IPsec API? In which release?



^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-30  7:09           ` Thomas Monjalon
@ 2019-07-30  7:20             ` Akhil Goyal
  2019-07-30  7:54               ` Thomas Monjalon
  0 siblings, 1 reply; 23+ messages in thread
From: Akhil Goyal @ 2019-07-30  7:20 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: Bernard Iremonger, dev, Anoob Joseph, konstantin.ananyev,
	Jerin Jacob Kollanukkaran, Narayana Prasad Raju Athreya


> 
> 30/07/2019 07:55, Akhil Goyal:
> >
> > >
> > > > > > All the functionality of the legacy code path in now available in the
> > > librte_ipsec
> > > > > > library. It is planned to deprecate the legacy code path in the 19.11
> release
> > > and
> > > > > > remove the legacy code path in the 20.02 release.
> > > > > >
> > > > > > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > > > > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > ---
> > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > >  1 file changed, 5 insertions(+)
> > > > > >
> > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > >
> > > > Applied to dpdk-next-crypto
> > >
> > > Why do we have a deprecation notice for some code path in an example?
> > > The deprecation notices are for the API.
> > >
> > > I think you can drop the legacy code in 19.11,
> > > and I don't merge this patch in master.
> > >
> >
> > We are planning to remove the original code and replace it with IPSec library
> APIs which are still experimental. With this change there won't be any example
> of the legacy ipsec code path. Applications over DPDK take ipsec-secgw as an
> example and IPSec is a major use case for customers. There may also be
> performance differences in the two code paths. Atleast on NXP platforms I saw
> 5-7% drop when the patches were originally submitted. Not sure what is the
> current state.
> > I feel it is worth notifying the users that the original codepath is getting
> deprecated, so that they can plan to move to new IPSec APIs.
> 
> The deprecation notice is not the right place for a change in an example.
> What change is there in IPsec API? In which release?
> 

IPSec lib was introduced in 1902 release and a few enhancements are done thereafter.
Previously all IPSec related stuff was done in the application, now we have IPSec Lib which
perform similar work. There are changes both in datapath as well as control path.
User need to adapt to the recent changes, as we may no longer support/maintain the datapath/control
path which was done previously and there may be some conflict.

If deprecation notice is not the right place, then where should it be notified before actually making the
change.


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-30  7:20             ` Akhil Goyal
@ 2019-07-30  7:54               ` Thomas Monjalon
  2019-07-30  8:48                 ` Akhil Goyal
  0 siblings, 1 reply; 23+ messages in thread
From: Thomas Monjalon @ 2019-07-30  7:54 UTC (permalink / raw)
  To: Akhil Goyal
  Cc: Bernard Iremonger, dev, Anoob Joseph, konstantin.ananyev,
	Jerin Jacob Kollanukkaran, Narayana Prasad Raju Athreya

30/07/2019 09:20, Akhil Goyal:
> > 30/07/2019 07:55, Akhil Goyal:
> > > > > > > All the functionality of the legacy code path in now available
> > > > > > > in the librte_ipsec library.
> > > > > > > It is planned to deprecate the legacy code path in the 19.11
> > > > > > > release and remove the legacy code path in the 20.02 release.
> > > > > > >
> > > > > > > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > > > > > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > > ---
> > > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > > >  1 file changed, 5 insertions(+)
> > > > > > >
> > > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > > >
> > > > > Applied to dpdk-next-crypto
> > > >
> > > > Why do we have a deprecation notice for some code path in an example?
> > > > The deprecation notices are for the API.
> > > >
> > > > I think you can drop the legacy code in 19.11,
> > > > and I don't merge this patch in master.
> > >
> > > We are planning to remove the original code and replace it with IPSec
> > > library APIs which are still experimental.
> > > With this change there won't be any example of the legacy ipsec code path.

That's good to drop old code.
If someone still wants to look at it, it is in old releases.

> > > Applications over DPDK take ipsec-secgw as an example and IPSec
> > > is a major use case for customers. There may also be performance
> > > differences in the two code paths. Atleast on NXP platforms I saw
> > > 5-7% drop when the patches were originally submitted.
> > > Not sure what is the current state.

That's a different issue you need to solve in the library.

> > > I feel it is worth notifying the users that the original codepath is
> > > getting deprecated, so that they can plan to move to new IPSec APIs.

I hope they already planned to move when they saw the new library.

> > The deprecation notice is not the right place for a change in an example.
> > What change is there in IPsec API? In which release?
> 
> IPSec lib was introduced in 1902 release and a few enhancements are done thereafter.
> Previously all IPSec related stuff was done in the application, now we have IPSec Lib which
> perform similar work. There are changes both in datapath as well as control path.
> User need to adapt to the recent changes, as we may no longer support/maintain the datapath/control
> path which was done previously and there may be some conflict.

So the real DPDK change is to have a new library in 19.02.

> If deprecation notice is not the right place,
> then where should it be notified before actually making the change.

It has already been notified in "New Features" of 19.02
that there is an IPsec library. What do you want to notify more?
Again, the example is not supposed to be a real application.
If you want to maintain an IPsec application with better quality rules,
I suggest to start a new git repository for it.



^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-30  7:54               ` Thomas Monjalon
@ 2019-07-30  8:48                 ` Akhil Goyal
  2019-07-30  8:56                   ` Thomas Monjalon
  0 siblings, 1 reply; 23+ messages in thread
From: Akhil Goyal @ 2019-07-30  8:48 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: Bernard Iremonger, dev, Anoob Joseph, konstantin.ananyev,
	Jerin Jacob Kollanukkaran, Narayana Prasad Raju Athreya

> 
> 30/07/2019 09:20, Akhil Goyal:
> > > 30/07/2019 07:55, Akhil Goyal:
> > > > > > > > All the functionality of the legacy code path in now available
> > > > > > > > in the librte_ipsec library.
> > > > > > > > It is planned to deprecate the legacy code path in the 19.11
> > > > > > > > release and remove the legacy code path in the 20.02 release.
> > > > > > > >
> > > > > > > > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > > > > > > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > > > ---
> > > > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > > > >  1 file changed, 5 insertions(+)
> > > > > > > >
> > > > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > > > >
> > > > > > Applied to dpdk-next-crypto
> > > > >
> > > > > Why do we have a deprecation notice for some code path in an example?
> > > > > The deprecation notices are for the API.
> > > > >
> > > > > I think you can drop the legacy code in 19.11,
> > > > > and I don't merge this patch in master.
> > > >
> > > > We are planning to remove the original code and replace it with IPSec
> > > > library APIs which are still experimental.
> > > > With this change there won't be any example of the legacy ipsec code path.
> 
> That's good to drop old code.
> If someone still wants to look at it, it is in old releases.
> 
> > > > Applications over DPDK take ipsec-secgw as an example and IPSec
> > > > is a major use case for customers. There may also be performance
> > > > differences in the two code paths. Atleast on NXP platforms I saw
> > > > 5-7% drop when the patches were originally submitted.
> > > > Not sure what is the current state.
> 
> That's a different issue you need to solve in the library.
> 
> > > > I feel it is worth notifying the users that the original codepath is
> > > > getting deprecated, so that they can plan to move to new IPSec APIs.
> 
> I hope they already planned to move when they saw the new library.
> 
> > > The deprecation notice is not the right place for a change in an example.
> > > What change is there in IPsec API? In which release?
> >
> > IPSec lib was introduced in 1902 release and a few enhancements are done
> thereafter.
> > Previously all IPSec related stuff was done in the application, now we have
> IPSec Lib which
> > perform similar work. There are changes both in datapath as well as control
> path.
> > User need to adapt to the recent changes, as we may no longer
> support/maintain the datapath/control
> > path which was done previously and there may be some conflict.
> 
> So the real DPDK change is to have a new library in 19.02.
> 
> > If deprecation notice is not the right place,
> > then where should it be notified before actually making the change.
> 
> It has already been notified in "New Features" of 19.02
> that there is an IPsec library. What do you want to notify more?
> Again, the example is not supposed to be a real application.
> If you want to maintain an IPsec application with better quality rules,
> I suggest to start a new git repository for it.
> 

OK got your point, but in that case, I would say, legacy code shall not be removed
Until we have the ipsec lib as experimental.
User should have both the code paths as long as we have ipsec library experimental.

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-30  8:48                 ` Akhil Goyal
@ 2019-07-30  8:56                   ` Thomas Monjalon
  2019-07-30  9:03                     ` Akhil Goyal
  0 siblings, 1 reply; 23+ messages in thread
From: Thomas Monjalon @ 2019-07-30  8:56 UTC (permalink / raw)
  To: Akhil Goyal
  Cc: Bernard Iremonger, dev, Anoob Joseph, konstantin.ananyev,
	Jerin Jacob Kollanukkaran, Narayana Prasad Raju Athreya

30/07/2019 10:48, Akhil Goyal:
> > 30/07/2019 09:20, Akhil Goyal:
> > > > 30/07/2019 07:55, Akhil Goyal:
> > > > > > > > > All the functionality of the legacy code path in now available
> > > > > > > > > in the librte_ipsec library.
> > > > > > > > > It is planned to deprecate the legacy code path in the 19.11
> > > > > > > > > release and remove the legacy code path in the 20.02 release.
> > > > > > > > >
> > > > > > > > > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> > > > > > > > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > > > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > > > > ---
> > > > > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > > > > >  1 file changed, 5 insertions(+)
> > > > > > > > >
> > > > > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > > > > >
> > > > > > > Applied to dpdk-next-crypto
> > > > > >
> > > > > > Why do we have a deprecation notice for some code path in an example?
> > > > > > The deprecation notices are for the API.
> > > > > >
> > > > > > I think you can drop the legacy code in 19.11,
> > > > > > and I don't merge this patch in master.
> > > > >
> > > > > We are planning to remove the original code and replace it with IPSec
> > > > > library APIs which are still experimental.
> > > > > With this change there won't be any example of the legacy ipsec code path.
> > 
> > That's good to drop old code.
> > If someone still wants to look at it, it is in old releases.
> > 
> > > > > Applications over DPDK take ipsec-secgw as an example and IPSec
> > > > > is a major use case for customers. There may also be performance
> > > > > differences in the two code paths. Atleast on NXP platforms I saw
> > > > > 5-7% drop when the patches were originally submitted.
> > > > > Not sure what is the current state.
> > 
> > That's a different issue you need to solve in the library.
> > 
> > > > > I feel it is worth notifying the users that the original codepath is
> > > > > getting deprecated, so that they can plan to move to new IPSec APIs.
> > 
> > I hope they already planned to move when they saw the new library.
> > 
> > > > The deprecation notice is not the right place for a change in an example.
> > > > What change is there in IPsec API? In which release?
> > >
> > > IPSec lib was introduced in 1902 release and a few enhancements
> > > are done thereafter.
> > > Previously all IPSec related stuff was done in the application,
> > > now we have IPSec Lib which perform similar work.
> > > There are changes both in datapath as well as control path.
> > > User need to adapt to the recent changes, as we may no longer
> > > support/maintain the datapath/control path which was done previously
> > > and there may be some conflict.
> > 
> > So the real DPDK change is to have a new library in 19.02.
> > 
> > > If deprecation notice is not the right place,
> > > then where should it be notified before actually making the change.
> > 
> > It has already been notified in "New Features" of 19.02
> > that there is an IPsec library. What do you want to notify more?
> > Again, the example is not supposed to be a real application.
> > If you want to maintain an IPsec application with better quality rules,
> > I suggest to start a new git repository for it.
> 
> OK got your point, but in that case, I would say, legacy code shall not be removed
> Until we have the ipsec lib as experimental.
> User should have both the code paths as long as we have ipsec library experimental.

That's your take.
When do you plan to remove experimental status of IPsec library?



^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-30  8:56                   ` Thomas Monjalon
@ 2019-07-30  9:03                     ` Akhil Goyal
  2019-07-30  9:25                       ` Ananyev, Konstantin
  0 siblings, 1 reply; 23+ messages in thread
From: Akhil Goyal @ 2019-07-30  9:03 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: Bernard Iremonger, dev, Anoob Joseph, konstantin.ananyev,
	Jerin Jacob Kollanukkaran, Narayana Prasad Raju Athreya



> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Tuesday, July 30, 2019 2:27 PM
> To: Akhil Goyal <akhil.goyal@nxp.com>
> Cc: Bernard Iremonger <bernard.iremonger@intel.com>; dev@dpdk.org; Anoob
> Joseph <anoobj@marvell.com>; konstantin.ananyev@intel.com; Jerin Jacob
> Kollanukkaran <jerinj@marvell.com>; Narayana Prasad Raju Athreya
> <pathreya@marvell.com>
> Subject: Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-
> secgw
> 
> 30/07/2019 10:48, Akhil Goyal:
> > > 30/07/2019 09:20, Akhil Goyal:
> > > > > 30/07/2019 07:55, Akhil Goyal:
> > > > > > > > > > All the functionality of the legacy code path in now available
> > > > > > > > > > in the librte_ipsec library.
> > > > > > > > > > It is planned to deprecate the legacy code path in the 19.11
> > > > > > > > > > release and remove the legacy code path in the 20.02 release.
> > > > > > > > > >
> > > > > > > > > > Signed-off-by: Bernard Iremonger
> <bernard.iremonger@intel.com>
> > > > > > > > > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > > > > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > > > > > ---
> > > > > > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > > > > > >  1 file changed, 5 insertions(+)
> > > > > > > > > >
> > > > > > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > > > > > >
> > > > > > > > Applied to dpdk-next-crypto
> > > > > > >
> > > > > > > Why do we have a deprecation notice for some code path in an
> example?
> > > > > > > The deprecation notices are for the API.
> > > > > > >
> > > > > > > I think you can drop the legacy code in 19.11,
> > > > > > > and I don't merge this patch in master.
> > > > > >
> > > > > > We are planning to remove the original code and replace it with IPSec
> > > > > > library APIs which are still experimental.
> > > > > > With this change there won't be any example of the legacy ipsec code
> path.
> > >
> > > That's good to drop old code.
> > > If someone still wants to look at it, it is in old releases.
> > >
> > > > > > Applications over DPDK take ipsec-secgw as an example and IPSec
> > > > > > is a major use case for customers. There may also be performance
> > > > > > differences in the two code paths. Atleast on NXP platforms I saw
> > > > > > 5-7% drop when the patches were originally submitted.
> > > > > > Not sure what is the current state.
> > >
> > > That's a different issue you need to solve in the library.
> > >
> > > > > > I feel it is worth notifying the users that the original codepath is
> > > > > > getting deprecated, so that they can plan to move to new IPSec APIs.
> > >
> > > I hope they already planned to move when they saw the new library.
> > >
> > > > > The deprecation notice is not the right place for a change in an example.
> > > > > What change is there in IPsec API? In which release?
> > > >
> > > > IPSec lib was introduced in 1902 release and a few enhancements
> > > > are done thereafter.
> > > > Previously all IPSec related stuff was done in the application,
> > > > now we have IPSec Lib which perform similar work.
> > > > There are changes both in datapath as well as control path.
> > > > User need to adapt to the recent changes, as we may no longer
> > > > support/maintain the datapath/control path which was done previously
> > > > and there may be some conflict.
> > >
> > > So the real DPDK change is to have a new library in 19.02.
> > >
> > > > If deprecation notice is not the right place,
> > > > then where should it be notified before actually making the change.
> > >
> > > It has already been notified in "New Features" of 19.02
> > > that there is an IPsec library. What do you want to notify more?
> > > Again, the example is not supposed to be a real application.
> > > If you want to maintain an IPsec application with better quality rules,
> > > I suggest to start a new git repository for it.
> >
> > OK got your point, but in that case, I would say, legacy code shall not be
> removed
> > Until we have the ipsec lib as experimental.
> > User should have both the code paths as long as we have ipsec library
> experimental.
> 
> That's your take.
> When do you plan to remove experimental status of IPsec library?
> 
There have been addition of some functionality in this release cycle. I would say we
can wait for 1 release cycle for some fixes or changes which may be required.
If it looks stable in next release cycle, we can make formal in DPDK 2002.

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-30  9:03                     ` Akhil Goyal
@ 2019-07-30  9:25                       ` Ananyev, Konstantin
  2019-07-30 11:06                         ` Akhil Goyal
  0 siblings, 1 reply; 23+ messages in thread
From: Ananyev, Konstantin @ 2019-07-30  9:25 UTC (permalink / raw)
  To: Akhil Goyal, Thomas Monjalon
  Cc: Iremonger, Bernard, dev, Anoob Joseph, Jerin Jacob Kollanukkaran,
	Narayana Prasad Raju Athreya

> 
> > -----Original Message-----
> > From: Thomas Monjalon <thomas@monjalon.net>
> > Sent: Tuesday, July 30, 2019 2:27 PM
> > To: Akhil Goyal <akhil.goyal@nxp.com>
> > Cc: Bernard Iremonger <bernard.iremonger@intel.com>; dev@dpdk.org; Anoob
> > Joseph <anoobj@marvell.com>; konstantin.ananyev@intel.com; Jerin Jacob
> > Kollanukkaran <jerinj@marvell.com>; Narayana Prasad Raju Athreya
> > <pathreya@marvell.com>
> > Subject: Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-
> > secgw
> >
> > 30/07/2019 10:48, Akhil Goyal:
> > > > 30/07/2019 09:20, Akhil Goyal:
> > > > > > 30/07/2019 07:55, Akhil Goyal:
> > > > > > > > > > > All the functionality of the legacy code path in now available
> > > > > > > > > > > in the librte_ipsec library.
> > > > > > > > > > > It is planned to deprecate the legacy code path in the 19.11
> > > > > > > > > > > release and remove the legacy code path in the 20.02 release.
> > > > > > > > > > >
> > > > > > > > > > > Signed-off-by: Bernard Iremonger
> > <bernard.iremonger@intel.com>
> > > > > > > > > > > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > > > > > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > > > > > > ---
> > > > > > > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > > > > > > >  1 file changed, 5 insertions(+)
> > > > > > > > > > >
> > > > > > > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > > > > > > >
> > > > > > > > > Applied to dpdk-next-crypto
> > > > > > > >
> > > > > > > > Why do we have a deprecation notice for some code path in an
> > example?
> > > > > > > > The deprecation notices are for the API.
> > > > > > > >
> > > > > > > > I think you can drop the legacy code in 19.11,
> > > > > > > > and I don't merge this patch in master.
> > > > > > >
> > > > > > > We are planning to remove the original code and replace it with IPSec
> > > > > > > library APIs which are still experimental.
> > > > > > > With this change there won't be any example of the legacy ipsec code
> > path.
> > > >
> > > > That's good to drop old code.
> > > > If someone still wants to look at it, it is in old releases.
> > > >
> > > > > > > Applications over DPDK take ipsec-secgw as an example and IPSec
> > > > > > > is a major use case for customers. There may also be performance
> > > > > > > differences in the two code paths. Atleast on NXP platforms I saw
> > > > > > > 5-7% drop when the patches were originally submitted.
> > > > > > > Not sure what is the current state.
> > > >
> > > > That's a different issue you need to solve in the library.
> > > >
> > > > > > > I feel it is worth notifying the users that the original codepath is
> > > > > > > getting deprecated, so that they can plan to move to new IPSec APIs.
> > > >
> > > > I hope they already planned to move when they saw the new library.
> > > >
> > > > > > The deprecation notice is not the right place for a change in an example.
> > > > > > What change is there in IPsec API? In which release?
> > > > >
> > > > > IPSec lib was introduced in 1902 release and a few enhancements
> > > > > are done thereafter.
> > > > > Previously all IPSec related stuff was done in the application,
> > > > > now we have IPSec Lib which perform similar work.
> > > > > There are changes both in datapath as well as control path.
> > > > > User need to adapt to the recent changes, as we may no longer
> > > > > support/maintain the datapath/control path which was done previously
> > > > > and there may be some conflict.
> > > >
> > > > So the real DPDK change is to have a new library in 19.02.
> > > >
> > > > > If deprecation notice is not the right place,
> > > > > then where should it be notified before actually making the change.
> > > >
> > > > It has already been notified in "New Features" of 19.02
> > > > that there is an IPsec library. What do you want to notify more?
> > > > Again, the example is not supposed to be a real application.
> > > > If you want to maintain an IPsec application with better quality rules,
> > > > I suggest to start a new git repository for it.
> > >
> > > OK got your point, but in that case, I would say, legacy code shall not be
> > removed
> > > Until we have the ipsec lib as experimental.
> > > User should have both the code paths as long as we have ipsec library
> > experimental.

Not sure why it is needed?
Why DPDK sample app can't use DPDK experimental API as it is,
without some alternate code-path?

> >
> > That's your take.
> > When do you plan to remove experimental status of IPsec library?
> >
> There have been addition of some functionality in this release cycle. I would say we
> can wait for 1 release cycle for some fixes or changes which may be required.
> If it looks stable in next release cycle, we can make formal in DPDK 2002.

If we'll leave legacy code in 19.11, does it mean we'll have to
support it for next 2 years (LTS cycle)?
Konstantin  


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-30  9:25                       ` Ananyev, Konstantin
@ 2019-07-30 11:06                         ` Akhil Goyal
  2019-07-31 10:19                           ` Ananyev, Konstantin
  0 siblings, 1 reply; 23+ messages in thread
From: Akhil Goyal @ 2019-07-30 11:06 UTC (permalink / raw)
  To: Ananyev, Konstantin, Thomas Monjalon
  Cc: Iremonger, Bernard, dev, Anoob Joseph, Jerin Jacob Kollanukkaran,
	Narayana Prasad Raju Athreya

> > >
> > > 30/07/2019 10:48, Akhil Goyal:
> > > > > 30/07/2019 09:20, Akhil Goyal:
> > > > > > > 30/07/2019 07:55, Akhil Goyal:
> > > > > > > > > > > > All the functionality of the legacy code path in now available
> > > > > > > > > > > > in the librte_ipsec library.
> > > > > > > > > > > > It is planned to deprecate the legacy code path in the 19.11
> > > > > > > > > > > > release and remove the legacy code path in the 20.02
> release.
> > > > > > > > > > > >
> > > > > > > > > > > > Signed-off-by: Bernard Iremonger
> > > <bernard.iremonger@intel.com>
> > > > > > > > > > > > Acked-by: Konstantin Ananyev
> <konstantin.ananyev@intel.com>
> > > > > > > > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > > > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > > > > > > > ---
> > > > > > > > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > > > > > > > >  1 file changed, 5 insertions(+)
> > > > > > > > > > > >
> > > > > > > > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > > > > > > > >
> > > > > > > > > > Applied to dpdk-next-crypto
> > > > > > > > >
> > > > > > > > > Why do we have a deprecation notice for some code path in an
> > > example?
> > > > > > > > > The deprecation notices are for the API.
> > > > > > > > >
> > > > > > > > > I think you can drop the legacy code in 19.11,
> > > > > > > > > and I don't merge this patch in master.
> > > > > > > >
> > > > > > > > We are planning to remove the original code and replace it with
> IPSec
> > > > > > > > library APIs which are still experimental.
> > > > > > > > With this change there won't be any example of the legacy ipsec
> code
> > > path.
> > > > >
> > > > > That's good to drop old code.
> > > > > If someone still wants to look at it, it is in old releases.
> > > > >
> > > > > > > > Applications over DPDK take ipsec-secgw as an example and IPSec
> > > > > > > > is a major use case for customers. There may also be performance
> > > > > > > > differences in the two code paths. Atleast on NXP platforms I saw
> > > > > > > > 5-7% drop when the patches were originally submitted.
> > > > > > > > Not sure what is the current state.
> > > > >
> > > > > That's a different issue you need to solve in the library.
> > > > >
> > > > > > > > I feel it is worth notifying the users that the original codepath is
> > > > > > > > getting deprecated, so that they can plan to move to new IPSec
> APIs.
> > > > >
> > > > > I hope they already planned to move when they saw the new library.
> > > > >
> > > > > > > The deprecation notice is not the right place for a change in an
> example.
> > > > > > > What change is there in IPsec API? In which release?
> > > > > >
> > > > > > IPSec lib was introduced in 1902 release and a few enhancements
> > > > > > are done thereafter.
> > > > > > Previously all IPSec related stuff was done in the application,
> > > > > > now we have IPSec Lib which perform similar work.
> > > > > > There are changes both in datapath as well as control path.
> > > > > > User need to adapt to the recent changes, as we may no longer
> > > > > > support/maintain the datapath/control path which was done previously
> > > > > > and there may be some conflict.
> > > > >
> > > > > So the real DPDK change is to have a new library in 19.02.
> > > > >
> > > > > > If deprecation notice is not the right place,
> > > > > > then where should it be notified before actually making the change.
> > > > >
> > > > > It has already been notified in "New Features" of 19.02
> > > > > that there is an IPsec library. What do you want to notify more?
> > > > > Again, the example is not supposed to be a real application.
> > > > > If you want to maintain an IPsec application with better quality rules,
> > > > > I suggest to start a new git repository for it.
> > > >
> > > > OK got your point, but in that case, I would say, legacy code shall not be
> > > removed
> > > > Until we have the ipsec lib as experimental.
> > > > User should have both the code paths as long as we have ipsec library
> > > experimental.
> 
> Not sure why it is needed?
There is some performance gap. Original Idea for ipsec lib was to have
better performing APIs. How much gain is observed using these APIs on intel?
Do we have data? Atleast on NXP devices, these are not performing better.

> Why DPDK sample app can't use DPDK experimental API as it is,
> without some alternate code-path?

Sample app can use experimental APIs as is. There is no issue with that.
The intent is just to notify the users that it will be removed so that they
can be ready for change in their code. 

> 
> > >
> > > That's your take.
> > > When do you plan to remove experimental status of IPsec library?
> > >
> > There have been addition of some functionality in this release cycle. I would
> say we
> > can wait for 1 release cycle for some fixes or changes which may be required.
> > If it looks stable in next release cycle, we can make formal in DPDK 2002.
> 
> If we'll leave legacy code in 19.11, does it mean we'll have to
> support it for next 2 years (LTS cycle)?

The original intent of this patch was also to remove the legacy code in 2002 release and
not in 1911. Moreover, it is the application code that we need to remove. We can get that patch
backported to the 19.11.1 release as well. Then it would be same.

> Konstantin


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-30 11:06                         ` Akhil Goyal
@ 2019-07-31 10:19                           ` Ananyev, Konstantin
  2019-07-31 11:26                             ` Akhil Goyal
  0 siblings, 1 reply; 23+ messages in thread
From: Ananyev, Konstantin @ 2019-07-31 10:19 UTC (permalink / raw)
  To: Akhil Goyal, Thomas Monjalon
  Cc: Iremonger, Bernard, dev, Anoob Joseph, Jerin Jacob Kollanukkaran,
	Narayana Prasad Raju Athreya

Hi Akhil,

> > > >
> > > > 30/07/2019 10:48, Akhil Goyal:
> > > > > > 30/07/2019 09:20, Akhil Goyal:
> > > > > > > > 30/07/2019 07:55, Akhil Goyal:
> > > > > > > > > > > > > All the functionality of the legacy code path in now available
> > > > > > > > > > > > > in the librte_ipsec library.
> > > > > > > > > > > > > It is planned to deprecate the legacy code path in the 19.11
> > > > > > > > > > > > > release and remove the legacy code path in the 20.02
> > release.
> > > > > > > > > > > > >
> > > > > > > > > > > > > Signed-off-by: Bernard Iremonger
> > > > <bernard.iremonger@intel.com>
> > > > > > > > > > > > > Acked-by: Konstantin Ananyev
> > <konstantin.ananyev@intel.com>
> > > > > > > > > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > > > > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > > > > > > > > ---
> > > > > > > > > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > > > > > > > > >  1 file changed, 5 insertions(+)
> > > > > > > > > > > > >
> > > > > > > > > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > > > > > > > > >
> > > > > > > > > > > Applied to dpdk-next-crypto
> > > > > > > > > >
> > > > > > > > > > Why do we have a deprecation notice for some code path in an
> > > > example?
> > > > > > > > > > The deprecation notices are for the API.
> > > > > > > > > >
> > > > > > > > > > I think you can drop the legacy code in 19.11,
> > > > > > > > > > and I don't merge this patch in master.
> > > > > > > > >
> > > > > > > > > We are planning to remove the original code and replace it with
> > IPSec
> > > > > > > > > library APIs which are still experimental.
> > > > > > > > > With this change there won't be any example of the legacy ipsec
> > code
> > > > path.
> > > > > >
> > > > > > That's good to drop old code.
> > > > > > If someone still wants to look at it, it is in old releases.
> > > > > >
> > > > > > > > > Applications over DPDK take ipsec-secgw as an example and IPSec
> > > > > > > > > is a major use case for customers. There may also be performance
> > > > > > > > > differences in the two code paths. Atleast on NXP platforms I saw
> > > > > > > > > 5-7% drop when the patches were originally submitted.
> > > > > > > > > Not sure what is the current state.
> > > > > >
> > > > > > That's a different issue you need to solve in the library.
> > > > > >
> > > > > > > > > I feel it is worth notifying the users that the original codepath is
> > > > > > > > > getting deprecated, so that they can plan to move to new IPSec
> > APIs.
> > > > > >
> > > > > > I hope they already planned to move when they saw the new library.
> > > > > >
> > > > > > > > The deprecation notice is not the right place for a change in an
> > example.
> > > > > > > > What change is there in IPsec API? In which release?
> > > > > > >
> > > > > > > IPSec lib was introduced in 1902 release and a few enhancements
> > > > > > > are done thereafter.
> > > > > > > Previously all IPSec related stuff was done in the application,
> > > > > > > now we have IPSec Lib which perform similar work.
> > > > > > > There are changes both in datapath as well as control path.
> > > > > > > User need to adapt to the recent changes, as we may no longer
> > > > > > > support/maintain the datapath/control path which was done previously
> > > > > > > and there may be some conflict.
> > > > > >
> > > > > > So the real DPDK change is to have a new library in 19.02.
> > > > > >
> > > > > > > If deprecation notice is not the right place,
> > > > > > > then where should it be notified before actually making the change.
> > > > > >
> > > > > > It has already been notified in "New Features" of 19.02
> > > > > > that there is an IPsec library. What do you want to notify more?
> > > > > > Again, the example is not supposed to be a real application.
> > > > > > If you want to maintain an IPsec application with better quality rules,
> > > > > > I suggest to start a new git repository for it.
> > > > >
> > > > > OK got your point, but in that case, I would say, legacy code shall not be
> > > > removed
> > > > > Until we have the ipsec lib as experimental.
> > > > > User should have both the code paths as long as we have ipsec library
> > > > experimental.
> >
> > Not sure why it is needed?
> There is some performance gap.

Ok, and you think that keeping legacy code-path  till 20.02
will provide enough time to analyze the perf drop for NXP devices?

> Original Idea for ipsec lib was to have
> better performing APIs.

Not only.
Main reasons were to hide from user complexity of IPsec packet processing
and avoid necessity for each user to re-implement it. 
Also legacy code doesn't provide many important features
(replay window, ESN, multi-seg support, etc.)

> How much gain is observed using these APIs on intel?

It depends.
From my last observations for inline case library code-path is 10-15%
faster than legacy one. Though I think reason for that in not in the library
itself, but that legacy code handles inline traffic in suboptimal way.
For lookaside - in most cases library and legacy code performance is about the same,
except the case when we have a burst of packets where each packet belongs to different SA.
In that case new code-path is 3-5% slower.
As I understand, main reason is again not the library itself,
but that in new code we try to group packets that belongs 
to the same SA both before and after crypto-dev enqueue/dequeue.
Usually that helps, but for that case it just adds extra overhead (no grouping is possible). 

> Do we have data? Atleast on NXP devices, these are not performing better.
> 
> > Why DPDK sample app can't use DPDK experimental API as it is,
> > without some alternate code-path?
> 
> Sample app can use experimental APIs as is. There is no issue with that.
> The intent is just to notify the users that it will be removed so that they
> can be ready for change in their code.

Ok, but if we don't submit deprecation notice (as Thomas suggested) 
how they'll be notified?
Would there be a separate RN when we'll remove experimental tag
and legacy code? 

> 
> >
> > > >
> > > > That's your take.
> > > > When do you plan to remove experimental status of IPsec library?
> > > >
> > > There have been addition of some functionality in this release cycle. I would
> > say we
> > > can wait for 1 release cycle for some fixes or changes which may be required.
> > > If it looks stable in next release cycle, we can make formal in DPDK 2002.
> >
> > If we'll leave legacy code in 19.11, does it mean we'll have to
> > support it for next 2 years (LTS cycle)?
> 
> The original intent of this patch was also to remove the legacy code in 2002 release and
> not in 1911. Moreover, it is the application code that we need to remove. We can get that patch
> backported to the 19.11.1 release as well. Then it would be same.
> 





^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-31 10:19                           ` Ananyev, Konstantin
@ 2019-07-31 11:26                             ` Akhil Goyal
  2019-08-01  7:34                               ` Ananyev, Konstantin
  0 siblings, 1 reply; 23+ messages in thread
From: Akhil Goyal @ 2019-07-31 11:26 UTC (permalink / raw)
  To: Ananyev, Konstantin, Thomas Monjalon
  Cc: Iremonger, Bernard, dev, Anoob Joseph, Jerin Jacob Kollanukkaran,
	Narayana Prasad Raju Athreya

Hi Konstantin,
> 
> Hi Akhil,
> 
> > > > >
> > > > > 30/07/2019 10:48, Akhil Goyal:
> > > > > > > 30/07/2019 09:20, Akhil Goyal:
> > > > > > > > > 30/07/2019 07:55, Akhil Goyal:
> > > > > > > > > > > > > > All the functionality of the legacy code path in now
> available
> > > > > > > > > > > > > > in the librte_ipsec library.
> > > > > > > > > > > > > > It is planned to deprecate the legacy code path in the
> 19.11
> > > > > > > > > > > > > > release and remove the legacy code path in the 20.02
> > > release.
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > Signed-off-by: Bernard Iremonger
> > > > > <bernard.iremonger@intel.com>
> > > > > > > > > > > > > > Acked-by: Konstantin Ananyev
> > > <konstantin.ananyev@intel.com>
> > > > > > > > > > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > > > > > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > > > > > > > > > ---
> > > > > > > > > > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > > > > > > > > > >  1 file changed, 5 insertions(+)
> > > > > > > > > > > > > >
> > > > > > > > > > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > > > > > > > > > >
> > > > > > > > > > > > Applied to dpdk-next-crypto
> > > > > > > > > > >
> > > > > > > > > > > Why do we have a deprecation notice for some code path in
> an
> > > > > example?
> > > > > > > > > > > The deprecation notices are for the API.
> > > > > > > > > > >
> > > > > > > > > > > I think you can drop the legacy code in 19.11,
> > > > > > > > > > > and I don't merge this patch in master.
> > > > > > > > > >
> > > > > > > > > > We are planning to remove the original code and replace it with
> > > IPSec
> > > > > > > > > > library APIs which are still experimental.
> > > > > > > > > > With this change there won't be any example of the legacy ipsec
> > > code
> > > > > path.
> > > > > > >
> > > > > > > That's good to drop old code.
> > > > > > > If someone still wants to look at it, it is in old releases.
> > > > > > >
> > > > > > > > > > Applications over DPDK take ipsec-secgw as an example and
> IPSec
> > > > > > > > > > is a major use case for customers. There may also be
> performance
> > > > > > > > > > differences in the two code paths. Atleast on NXP platforms I
> saw
> > > > > > > > > > 5-7% drop when the patches were originally submitted.
> > > > > > > > > > Not sure what is the current state.
> > > > > > >
> > > > > > > That's a different issue you need to solve in the library.
> > > > > > >
> > > > > > > > > > I feel it is worth notifying the users that the original codepath is
> > > > > > > > > > getting deprecated, so that they can plan to move to new IPSec
> > > APIs.
> > > > > > >
> > > > > > > I hope they already planned to move when they saw the new library.
> > > > > > >
> > > > > > > > > The deprecation notice is not the right place for a change in an
> > > example.
> > > > > > > > > What change is there in IPsec API? In which release?
> > > > > > > >
> > > > > > > > IPSec lib was introduced in 1902 release and a few enhancements
> > > > > > > > are done thereafter.
> > > > > > > > Previously all IPSec related stuff was done in the application,
> > > > > > > > now we have IPSec Lib which perform similar work.
> > > > > > > > There are changes both in datapath as well as control path.
> > > > > > > > User need to adapt to the recent changes, as we may no longer
> > > > > > > > support/maintain the datapath/control path which was done
> previously
> > > > > > > > and there may be some conflict.
> > > > > > >
> > > > > > > So the real DPDK change is to have a new library in 19.02.
> > > > > > >
> > > > > > > > If deprecation notice is not the right place,
> > > > > > > > then where should it be notified before actually making the change.
> > > > > > >
> > > > > > > It has already been notified in "New Features" of 19.02
> > > > > > > that there is an IPsec library. What do you want to notify more?
> > > > > > > Again, the example is not supposed to be a real application.
> > > > > > > If you want to maintain an IPsec application with better quality rules,
> > > > > > > I suggest to start a new git repository for it.
> > > > > >
> > > > > > OK got your point, but in that case, I would say, legacy code shall not
> be
> > > > > removed
> > > > > > Until we have the ipsec lib as experimental.
> > > > > > User should have both the code paths as long as we have ipsec library
> > > > > experimental.
> > >
> > > Not sure why it is needed?
> > There is some performance gap.
> 
> Ok, and you think that keeping legacy code-path  till 20.02
> will provide enough time to analyze the perf drop for NXP devices?

Yes. We cannot commit for 19.11 timeframe to fix the performance issues in
the new code. You can send the patches, we can try our best to incorporate it in
19.11 but we cannot commit at this moment.

> 
> > Original Idea for ipsec lib was to have
> > better performing APIs.
> 
> Not only.
> Main reasons were to hide from user complexity of IPsec packet processing
> and avoid necessity for each user to re-implement it.
> Also legacy code doesn't provide many important features
> (replay window, ESN, multi-seg support, etc.)
> 
> > How much gain is observed using these APIs on intel?
> 
> It depends.
> From my last observations for inline case library code-path is 10-15%
> faster than legacy one. Though I think reason for that in not in the library
> itself, but that legacy code handles inline traffic in suboptimal way.
> For lookaside - in most cases library and legacy code performance is about the
> same,
> except the case when we have a burst of packets where each packet belongs to
> different SA.
> In that case new code-path is 3-5% slower.
> As I understand, main reason is again not the library itself,
> but that in new code we try to group packets that belongs
> to the same SA both before and after crypto-dev enqueue/dequeue.
> Usually that helps, but for that case it just adds extra overhead (no grouping is
> possible).
> 
> > Do we have data? Atleast on NXP devices, these are not performing better.
> >
> > > Why DPDK sample app can't use DPDK experimental API as it is,
> > > without some alternate code-path?
> >
> > Sample app can use experimental APIs as is. There is no issue with that.
> > The intent is just to notify the users that it will be removed so that they
> > can be ready for change in their code.
> 
> Ok, but if we don't submit deprecation notice (as Thomas suggested)
> how they'll be notified?
As Thomas said, there is no need for sending the deprecation notice for application.
So I have to hold my comment back.

> Would there be a separate RN when we'll remove experimental tag
> and legacy code?
There will be a RN entry when we remove the experimental tag and the legacy code.
> 
> >
> > >
> > > > >
> > > > > That's your take.
> > > > > When do you plan to remove experimental status of IPsec library?
> > > > >
> > > > There have been addition of some functionality in this release cycle. I
> would
> > > say we
> > > > can wait for 1 release cycle for some fixes or changes which may be
> required.
> > > > If it looks stable in next release cycle, we can make formal in DPDK 2002.
> > >
> > > If we'll leave legacy code in 19.11, does it mean we'll have to
> > > support it for next 2 years (LTS cycle)?
> >
> > The original intent of this patch was also to remove the legacy code in 2002
> release and
> > not in 1911. Moreover, it is the application code that we need to remove. We
> can get that patch
> > backported to the 19.11.1 release as well. Then it would be same.
> >
> 
> 
> 


^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [dpdk-dev] [EXT] [PATCH] doc: deprecate legacy code path in ipsec-secgw
  2019-07-31 11:26                             ` Akhil Goyal
@ 2019-08-01  7:34                               ` Ananyev, Konstantin
  0 siblings, 0 replies; 23+ messages in thread
From: Ananyev, Konstantin @ 2019-08-01  7:34 UTC (permalink / raw)
  To: Akhil Goyal, Thomas Monjalon
  Cc: Iremonger, Bernard, dev, Anoob Joseph, Jerin Jacob Kollanukkaran,
	Narayana Prasad Raju Athreya


Hi Akhil,
> >
> > > > > >
> > > > > > 30/07/2019 10:48, Akhil Goyal:
> > > > > > > > 30/07/2019 09:20, Akhil Goyal:
> > > > > > > > > > 30/07/2019 07:55, Akhil Goyal:
> > > > > > > > > > > > > > > All the functionality of the legacy code path in now
> > available
> > > > > > > > > > > > > > > in the librte_ipsec library.
> > > > > > > > > > > > > > > It is planned to deprecate the legacy code path in the
> > 19.11
> > > > > > > > > > > > > > > release and remove the legacy code path in the 20.02
> > > > release.
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > Signed-off-by: Bernard Iremonger
> > > > > > <bernard.iremonger@intel.com>
> > > > > > > > > > > > > > > Acked-by: Konstantin Ananyev
> > > > <konstantin.ananyev@intel.com>
> > > > > > > > > > > > > > > Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > > > > > > > > > > > Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> > > > > > > > > > > > > > > ---
> > > > > > > > > > > > > > >  doc/guides/rel_notes/deprecation.rst | 5 +++++
> > > > > > > > > > > > > > >  1 file changed, 5 insertions(+)
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > Acked-by: Anoob Joseph <anoobj@marvell.com>
> > > > > > > > > > > > >
> > > > > > > > > > > > > Applied to dpdk-next-crypto
> > > > > > > > > > > >
> > > > > > > > > > > > Why do we have a deprecation notice for some code path in
> > an
> > > > > > example?
> > > > > > > > > > > > The deprecation notices are for the API.
> > > > > > > > > > > >
> > > > > > > > > > > > I think you can drop the legacy code in 19.11,
> > > > > > > > > > > > and I don't merge this patch in master.
> > > > > > > > > > >
> > > > > > > > > > > We are planning to remove the original code and replace it with
> > > > IPSec
> > > > > > > > > > > library APIs which are still experimental.
> > > > > > > > > > > With this change there won't be any example of the legacy ipsec
> > > > code
> > > > > > path.
> > > > > > > >
> > > > > > > > That's good to drop old code.
> > > > > > > > If someone still wants to look at it, it is in old releases.
> > > > > > > >
> > > > > > > > > > > Applications over DPDK take ipsec-secgw as an example and
> > IPSec
> > > > > > > > > > > is a major use case for customers. There may also be
> > performance
> > > > > > > > > > > differences in the two code paths. Atleast on NXP platforms I
> > saw
> > > > > > > > > > > 5-7% drop when the patches were originally submitted.
> > > > > > > > > > > Not sure what is the current state.
> > > > > > > >
> > > > > > > > That's a different issue you need to solve in the library.
> > > > > > > >
> > > > > > > > > > > I feel it is worth notifying the users that the original codepath is
> > > > > > > > > > > getting deprecated, so that they can plan to move to new IPSec
> > > > APIs.
> > > > > > > >
> > > > > > > > I hope they already planned to move when they saw the new library.
> > > > > > > >
> > > > > > > > > > The deprecation notice is not the right place for a change in an
> > > > example.
> > > > > > > > > > What change is there in IPsec API? In which release?
> > > > > > > > >
> > > > > > > > > IPSec lib was introduced in 1902 release and a few enhancements
> > > > > > > > > are done thereafter.
> > > > > > > > > Previously all IPSec related stuff was done in the application,
> > > > > > > > > now we have IPSec Lib which perform similar work.
> > > > > > > > > There are changes both in datapath as well as control path.
> > > > > > > > > User need to adapt to the recent changes, as we may no longer
> > > > > > > > > support/maintain the datapath/control path which was done
> > previously
> > > > > > > > > and there may be some conflict.
> > > > > > > >
> > > > > > > > So the real DPDK change is to have a new library in 19.02.
> > > > > > > >
> > > > > > > > > If deprecation notice is not the right place,
> > > > > > > > > then where should it be notified before actually making the change.
> > > > > > > >
> > > > > > > > It has already been notified in "New Features" of 19.02
> > > > > > > > that there is an IPsec library. What do you want to notify more?
> > > > > > > > Again, the example is not supposed to be a real application.
> > > > > > > > If you want to maintain an IPsec application with better quality rules,
> > > > > > > > I suggest to start a new git repository for it.
> > > > > > >
> > > > > > > OK got your point, but in that case, I would say, legacy code shall not
> > be
> > > > > > removed
> > > > > > > Until we have the ipsec lib as experimental.
> > > > > > > User should have both the code paths as long as we have ipsec library
> > > > > > experimental.
> > > >
> > > > Not sure why it is needed?
> > > There is some performance gap.
> >
> > Ok, and you think that keeping legacy code-path  till 20.02
> > will provide enough time to analyze the perf drop for NXP devices?
> 
> Yes. We cannot commit for 19.11 timeframe to fix the performance issues in
> the new code. You can send the patches, we can try our best to incorporate it in
> 19.11 but we cannot commit at this moment.

Ok, then let's put 20.02 as a preliminary target for removal experimental tag and legacy code-path.
As you said, we can later consider to backport ipsec-secgw changes into 19.11.1 or so.
Konstantin


> 
> >
> > > Original Idea for ipsec lib was to have
> > > better performing APIs.
> >
> > Not only.
> > Main reasons were to hide from user complexity of IPsec packet processing
> > and avoid necessity for each user to re-implement it.
> > Also legacy code doesn't provide many important features
> > (replay window, ESN, multi-seg support, etc.)
> >
> > > How much gain is observed using these APIs on intel?
> >
> > It depends.
> > From my last observations for inline case library code-path is 10-15%
> > faster than legacy one. Though I think reason for that in not in the library
> > itself, but that legacy code handles inline traffic in suboptimal way.
> > For lookaside - in most cases library and legacy code performance is about the
> > same,
> > except the case when we have a burst of packets where each packet belongs to
> > different SA.
> > In that case new code-path is 3-5% slower.
> > As I understand, main reason is again not the library itself,
> > but that in new code we try to group packets that belongs
> > to the same SA both before and after crypto-dev enqueue/dequeue.
> > Usually that helps, but for that case it just adds extra overhead (no grouping is
> > possible).
> >
> > > Do we have data? Atleast on NXP devices, these are not performing better.
> > >
> > > > Why DPDK sample app can't use DPDK experimental API as it is,
> > > > without some alternate code-path?
> > >
> > > Sample app can use experimental APIs as is. There is no issue with that.
> > > The intent is just to notify the users that it will be removed so that they
> > > can be ready for change in their code.
> >
> > Ok, but if we don't submit deprecation notice (as Thomas suggested)
> > how they'll be notified?
> As Thomas said, there is no need for sending the deprecation notice for application.
> So I have to hold my comment back.
> 
> > Would there be a separate RN when we'll remove experimental tag
> > and legacy code?
> There will be a RN entry when we remove the experimental tag and the legacy code.
> >
> > >
> > > >
> > > > > >
> > > > > > That's your take.
> > > > > > When do you plan to remove experimental status of IPsec library?
> > > > > >
> > > > > There have been addition of some functionality in this release cycle. I
> > would
> > > > say we
> > > > > can wait for 1 release cycle for some fixes or changes which may be
> > required.
> > > > > If it looks stable in next release cycle, we can make formal in DPDK 2002.
> > > >
> > > > If we'll leave legacy code in 19.11, does it mean we'll have to
> > > > support it for next 2 years (LTS cycle)?
> > >
> > > The original intent of this patch was also to remove the legacy code in 2002
> > release and
> > > not in 1911. Moreover, it is the application code that we need to remove. We
> > can get that patch
> > > backported to the 19.11.1 release as well. Then it would be same.
> > >
> >
> >
> >


^ permalink raw reply	[flat|nested] 23+ messages in thread

end of thread, back to index

Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-11  9:05 [dpdk-dev] [RFC] doc: deprecate legacy code path in ipsec-secgw Bernard Iremonger
2019-07-19 11:23 ` Ananyev, Konstantin
2019-07-19 14:55 ` Zhang, Roy Fan
2019-07-24  9:47 ` Akhil Goyal
2019-07-24 10:12 ` [dpdk-dev] [PATCH] " Bernard Iremonger
2019-07-24 10:14   ` Akhil Goyal
2019-07-26 14:06     ` Akhil Goyal
2019-07-26 14:45   ` [dpdk-dev] [EXT] " Anoob Joseph
2019-07-29  8:49     ` Akhil Goyal
2019-07-29 16:33       ` Thomas Monjalon
2019-07-29 20:07         ` Ananyev, Konstantin
2019-07-30  5:55         ` Akhil Goyal
2019-07-30  7:09           ` Thomas Monjalon
2019-07-30  7:20             ` Akhil Goyal
2019-07-30  7:54               ` Thomas Monjalon
2019-07-30  8:48                 ` Akhil Goyal
2019-07-30  8:56                   ` Thomas Monjalon
2019-07-30  9:03                     ` Akhil Goyal
2019-07-30  9:25                       ` Ananyev, Konstantin
2019-07-30 11:06                         ` Akhil Goyal
2019-07-31 10:19                           ` Ananyev, Konstantin
2019-07-31 11:26                             ` Akhil Goyal
2019-08-01  7:34                               ` Ananyev, Konstantin

DPDK-dev Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/dpdk-dev/0 dpdk-dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dpdk-dev dpdk-dev/ https://lore.kernel.org/dpdk-dev \
		dev@dpdk.org dpdk-dev@archiver.kernel.org
	public-inbox-index dpdk-dev


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/ public-inbox