DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators
@ 2020-07-29 14:46 Arek Kusztal
  2020-07-29 14:54 ` Kusztal, ArkadiuszX
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: Arek Kusztal @ 2020-07-29 14:46 UTC (permalink / raw)
  To: dev
  Cc: akhil.goyal, fiona.trahe, anoobj, shallyv, declan.doherty,
	roy.fan.zhang, konstantin.ananyev, Arek Kusztal

Enumerators RTE_CRYPTO_CIPHER_LIST_END, RTE_CRYPTO_AUTH_LIST_END,
RTE_CRYPTO_AEAD_LIST_END will be removed to prevent some problems
that may arise when adding new algorithms.

Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@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 a58a179..34b0e3c 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -142,3 +142,9 @@ Deprecation Notices
   Python 2 support will be completely removed in 20.11.
   In 20.08, explicit deprecation warnings will be displayed when running
   scripts with Python 2.
+
+* cryptodev: ``RTE_CRYPTO_AEAD_LIST_END`` from ``enum rte_crypto_aead_algorithm``,
+  ``RTE_CRYPTO_CIPHER_LIST_END`` from ``enum rte_crypto_cipher_algorithm`` and
+  ``RTE_CRYPTO_AUTH_LIST_END`` from ``enum rte_crypto_auth_algorithm`` will be
+  removed.
+  
\ No newline at end of file
-- 
2.1.0


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

* Re: [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators
  2020-07-29 14:46 [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators Arek Kusztal
@ 2020-07-29 14:54 ` Kusztal, ArkadiuszX
  2020-07-29 15:18 ` Bruce Richardson
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 8+ messages in thread
From: Kusztal, ArkadiuszX @ 2020-07-29 14:54 UTC (permalink / raw)
  To: dev
  Cc: akhil.goyal, Trahe, Fiona, anoobj, shallyv, Doherty, Declan,
	Zhang, Roy Fan, Ananyev, Konstantin

Hi All,

This is proposal to remove list_end enumerators from algorithms enums.
I do not expect many changes in current list of algorithms but it may happen (ChachaPoly in 20.02, GMAC move to AEAD proposal) so relying on list_end enumerator may be incorrect approach as this value may change across versions. It is used in some parts of DPDK code to specify array size so this would have to be changed with this approach.

Sorry if I not included someone in cc list.

> -----Original Message-----
> From: Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com>
> Sent: Wednesday, July 29, 2020 4:47 PM
> To: dev@dpdk.org
> Cc: akhil.goyal@nxp.com; Trahe, Fiona <fiona.trahe@intel.com>;
> anoobj@marvell.com; shallyv@marvell.com; Doherty, Declan
> <declan.doherty@intel.com>; Zhang, Roy Fan <roy.fan.zhang@intel.com>;
> Ananyev, Konstantin <konstantin.ananyev@intel.com>; Kusztal, ArkadiuszX
> <arkadiuszx.kusztal@intel.com>
> Subject: [PATCH] [RFC] doc: announce removal of crypto list end
> enumerators
> 
> Enumerators RTE_CRYPTO_CIPHER_LIST_END,
> RTE_CRYPTO_AUTH_LIST_END, RTE_CRYPTO_AEAD_LIST_END will be
> removed to prevent some problems that may arise when adding new
> algorithms.
> 
> Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@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 a58a179..34b0e3c 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -142,3 +142,9 @@ Deprecation Notices
>    Python 2 support will be completely removed in 20.11.
>    In 20.08, explicit deprecation warnings will be displayed when running
>    scripts with Python 2.
> +
> +* cryptodev: ``RTE_CRYPTO_AEAD_LIST_END`` from ``enum
> +rte_crypto_aead_algorithm``,
> +  ``RTE_CRYPTO_CIPHER_LIST_END`` from ``enum
> +rte_crypto_cipher_algorithm`` and
> +  ``RTE_CRYPTO_AUTH_LIST_END`` from ``enum
> rte_crypto_auth_algorithm``
> +will be
> +  removed.
> +
> \ No newline at end of file
> --
> 2.1.0


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

* Re: [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators
  2020-07-29 14:46 [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators Arek Kusztal
  2020-07-29 14:54 ` Kusztal, ArkadiuszX
@ 2020-07-29 15:18 ` Bruce Richardson
  2020-07-29 15:22 ` Trahe, Fiona
  2020-07-29 15:24 ` David Marchand
  3 siblings, 0 replies; 8+ messages in thread
From: Bruce Richardson @ 2020-07-29 15:18 UTC (permalink / raw)
  To: Arek Kusztal
  Cc: dev, akhil.goyal, fiona.trahe, anoobj, shallyv, declan.doherty,
	roy.fan.zhang, konstantin.ananyev

On Wed, Jul 29, 2020 at 04:46:51PM +0200, Arek Kusztal wrote:
> Enumerators RTE_CRYPTO_CIPHER_LIST_END, RTE_CRYPTO_AUTH_LIST_END,
> RTE_CRYPTO_AEAD_LIST_END will be removed to prevent some problems
> that may arise when adding new algorithms.
> 
> Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>

Since these seem to cause us ABI problems:

Acked-by: Bruce Richardson <bruce.richardson@intel.com>


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

* Re: [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators
  2020-07-29 14:46 [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators Arek Kusztal
  2020-07-29 14:54 ` Kusztal, ArkadiuszX
  2020-07-29 15:18 ` Bruce Richardson
@ 2020-07-29 15:22 ` Trahe, Fiona
  2020-07-29 15:24 ` David Marchand
  3 siblings, 0 replies; 8+ messages in thread
From: Trahe, Fiona @ 2020-07-29 15:22 UTC (permalink / raw)
  To: Kusztal, ArkadiuszX, dev
  Cc: akhil.goyal, anoobj, shallyv, Doherty, Declan, Zhang, Roy Fan,
	Ananyev, Konstantin



> -----Original Message-----
> From: Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com>
> Sent: Wednesday, July 29, 2020 3:47 PM
> To: dev@dpdk.org
> Cc: akhil.goyal@nxp.com; Trahe, Fiona <fiona.trahe@intel.com>; anoobj@marvell.com;
> shallyv@marvell.com; Doherty, Declan <declan.doherty@intel.com>; Zhang, Roy Fan
> <roy.fan.zhang@intel.com>; Ananyev, Konstantin <konstantin.ananyev@intel.com>; Kusztal, ArkadiuszX
> <arkadiuszx.kusztal@intel.com>
> Subject: [PATCH] [RFC] doc: announce removal of crypto list end enumerators
> 
> Enumerators RTE_CRYPTO_CIPHER_LIST_END, RTE_CRYPTO_AUTH_LIST_END,
> RTE_CRYPTO_AEAD_LIST_END will be removed to prevent some problems
> that may arise when adding new algorithms.
> 
> Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>
Acked-by: Fiona Trahe <fiona.trahe@intel.com> 


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

* Re: [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators
  2020-07-29 14:46 [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators Arek Kusztal
                   ` (2 preceding siblings ...)
  2020-07-29 15:22 ` Trahe, Fiona
@ 2020-07-29 15:24 ` David Marchand
  2020-07-29 15:32   ` Trahe, Fiona
  3 siblings, 1 reply; 8+ messages in thread
From: David Marchand @ 2020-07-29 15:24 UTC (permalink / raw)
  To: Arek Kusztal
  Cc: dev, Akhil Goyal, Trahe, Fiona, Anoob Joseph, shallyv,
	Declan Doherty, Fan Zhang, Ananyev, Konstantin

On Wed, Jul 29, 2020 at 4:47 PM Arek Kusztal
<arkadiuszx.kusztal@intel.com> wrote:
>
> Enumerators RTE_CRYPTO_CIPHER_LIST_END, RTE_CRYPTO_AUTH_LIST_END,
> RTE_CRYPTO_AEAD_LIST_END will be removed to prevent some problems
> that may arise when adding new algorithms.

How does it cohabitate with Ferruh notice?
Is it a complement? or redundant?

https://git.dpdk.org/dpdk/commit/doc/guides/rel_notes/deprecation.rst?id=58245da070289f4b248f15970c3f8af73ea601ed


>
> Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@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 a58a179..34b0e3c 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -142,3 +142,9 @@ Deprecation Notices
>    Python 2 support will be completely removed in 20.11.
>    In 20.08, explicit deprecation warnings will be displayed when running
>    scripts with Python 2.
> +
> +* cryptodev: ``RTE_CRYPTO_AEAD_LIST_END`` from ``enum rte_crypto_aead_algorithm``,
> +  ``RTE_CRYPTO_CIPHER_LIST_END`` from ``enum rte_crypto_cipher_algorithm`` and
> +  ``RTE_CRYPTO_AUTH_LIST_END`` from ``enum rte_crypto_auth_algorithm`` will be
> +  removed.
> +



-- 
David Marchand


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

* Re: [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators
  2020-07-29 15:24 ` David Marchand
@ 2020-07-29 15:32   ` Trahe, Fiona
  2020-07-31  9:38     ` Akhil Goyal
  0 siblings, 1 reply; 8+ messages in thread
From: Trahe, Fiona @ 2020-07-29 15:32 UTC (permalink / raw)
  To: David Marchand, Kusztal, ArkadiuszX
  Cc: dev, Akhil Goyal, Anoob Joseph, shallyv, Doherty, Declan, Zhang,
	Roy Fan, Ananyev, Konstantin

Hi David,

> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Wednesday, July 29, 2020 4:24 PM
> To: Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com>
> Cc: dev <dev@dpdk.org>; Akhil Goyal <akhil.goyal@nxp.com>; Trahe, Fiona <fiona.trahe@intel.com>;
> Anoob Joseph <anoobj@marvell.com>; shallyv@marvell.com; Doherty, Declan
> <declan.doherty@intel.com>; Zhang, Roy Fan <roy.fan.zhang@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators
> 
> On Wed, Jul 29, 2020 at 4:47 PM Arek Kusztal
> <arkadiuszx.kusztal@intel.com> wrote:
> >
> > Enumerators RTE_CRYPTO_CIPHER_LIST_END, RTE_CRYPTO_AUTH_LIST_END,
> > RTE_CRYPTO_AEAD_LIST_END will be removed to prevent some problems
> > that may arise when adding new algorithms.
> 
> How does it cohabitate with Ferruh notice?
> Is it a complement? or redundant?
> 
> https://git.dpdk.org/dpdk/commit/doc/guides/rel_notes/deprecation.rst?id=58245da070289f4b248f15
> 970c3f8af73ea601ed
> 
Thanks for drawing attention to this.
It's complementary as far as I can see.
Ferruh's doesn't list all enums - but doesn’t appear to cover these crypto ones.

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

* Re: [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators
  2020-07-29 15:32   ` Trahe, Fiona
@ 2020-07-31  9:38     ` Akhil Goyal
  2020-08-06 16:09       ` Thomas Monjalon
  0 siblings, 1 reply; 8+ messages in thread
From: Akhil Goyal @ 2020-07-31  9:38 UTC (permalink / raw)
  To: Trahe, Fiona, David Marchand, Kusztal, ArkadiuszX, Thomas Monjalon
  Cc: dev, Anoob Joseph, shallyv, Doherty, Declan, Zhang, Roy Fan,
	Ananyev, Konstantin

Hi David,

> Hi David,
> 
> >
> > On Wed, Jul 29, 2020 at 4:47 PM Arek Kusztal
> > <arkadiuszx.kusztal@intel.com> wrote:
> > >
> > > Enumerators RTE_CRYPTO_CIPHER_LIST_END,
> RTE_CRYPTO_AUTH_LIST_END,
> > > RTE_CRYPTO_AEAD_LIST_END will be removed to prevent some problems
> > > that may arise when adding new algorithms.
> >
> > How does it cohabitate with Ferruh notice?
> > Is it a complement? or redundant?
> >
> >
> https://git.dpdk.org/dpdk/commit/doc/guides/rel_notes/deprecation.rst?id=58245da070289f4b248f15
> >
> Thanks for drawing attention to this.
> It's complementary as far as I can see.
> Ferruh's doesn't list all enums - but doesn’t appear to cover these crypto ones.

I feel we can have this deprecation notice as well for crypto specific enums.
Though I don’t want to push it if somebody has objections on this.
I am OK if Ferruh's notice looks enough to cover crypto enums as well.

Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
If no objections, Thomas or you can pick this directly on master.

Regards,
Akhil

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

* Re: [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators
  2020-07-31  9:38     ` Akhil Goyal
@ 2020-08-06 16:09       ` Thomas Monjalon
  0 siblings, 0 replies; 8+ messages in thread
From: Thomas Monjalon @ 2020-08-06 16:09 UTC (permalink / raw)
  To: Trahe, Fiona, David Marchand, Kusztal, ArkadiuszX, Anoob Joseph,
	Akhil Goyal
  Cc: dev, shallyv, Doherty, Declan, Zhang, Roy Fan, Ananyev, Konstantin

31/07/2020 11:38, Akhil Goyal:
> Hi David,
> 
> > Hi David,
> > 
> > >
> > > On Wed, Jul 29, 2020 at 4:47 PM Arek Kusztal
> > > <arkadiuszx.kusztal@intel.com> wrote:
> > > >
> > > > Enumerators RTE_CRYPTO_CIPHER_LIST_END,
> > RTE_CRYPTO_AUTH_LIST_END,
> > > > RTE_CRYPTO_AEAD_LIST_END will be removed to prevent some problems
> > > > that may arise when adding new algorithms.
> > >
> > > How does it cohabitate with Ferruh notice?
> > > Is it a complement? or redundant?
> > >
> > >
> > https://git.dpdk.org/dpdk/commit/doc/guides/rel_notes/deprecation.rst?id=58245da070289f4b248f15
> > >
> > Thanks for drawing attention to this.
> > It's complementary as far as I can see.
> > Ferruh's doesn't list all enums - but doesn’t appear to cover these crypto ones.
> 
> I feel we can have this deprecation notice as well for crypto specific enums.
> Though I don’t want to push it if somebody has objections on this.
> I am OK if Ferruh's notice looks enough to cover crypto enums as well.
> 
> Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
> If no objections, Thomas or you can pick this directly on master.

I think Ferruh notice would be enough, except it focus on "MAX" values,
not including the similar ones with "END" in their names.

Anyway, too much notices don't hurt.

Applied



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

end of thread, other threads:[~2020-08-06 16:09 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-29 14:46 [dpdk-dev] [PATCH] [RFC] doc: announce removal of crypto list end enumerators Arek Kusztal
2020-07-29 14:54 ` Kusztal, ArkadiuszX
2020-07-29 15:18 ` Bruce Richardson
2020-07-29 15:22 ` Trahe, Fiona
2020-07-29 15:24 ` David Marchand
2020-07-29 15:32   ` Trahe, Fiona
2020-07-31  9:38     ` Akhil Goyal
2020-08-06 16:09       ` 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).