diff mbox series

[v2,20.08,1/6] doc: announce deprecation of master lcore

Message ID 20200730005804.19251-2-stephen@networkplumber.org (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series inclusive language fixes and deprecation notices |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/Intel-compilation success Compilation OK
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS

Commit Message

Stephen Hemminger July 30, 2020, 12:57 a.m. UTC 
  Announce upcoming changes related to master/slave in reference
to lcore.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 doc/guides/rel_notes/deprecation.rst | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

Comments

Bruce Richardson July 30, 2020, 8:42 a.m. UTC | #1 
On Wed, Jul 29, 2020 at 05:57:59PM -0700, Stephen Hemminger wrote:
> Announce upcoming changes related to master/slave in reference
> to lcore.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  doc/guides/rel_notes/deprecation.rst | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 99c98062ffc2..7c60779f3e68 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -148,3 +148,20 @@ 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.
> +
> +* eal: To be more inclusive in choice of naming, the DPDK project
> +  will replace uses of master/slave in  the API's and command line arguments.
> +
> +  References to master/slave in relation to lcore will be renamed
> +  to initial/worker.  The function ``rte_get_master_lcore()``
> +  will be renamed to ``rte_get_initial_lcore()``.
> +  For the 20.11, release both names will be present and the
> +  old function will be marked with the deprecated tag.
> +  The old function will be removed in 21.11 version.

Since 20.11 is going to be ABI incompatible with previous versions anyway,
can we not have the function as a macro alias for the new, allowing us to
remove the old sooner without breaking any apps? Even if this is not the
case, I think it might be better to change "21.11 version" to "a future
release" to allow us some flexibility on when to remove them in case we can
remove them sooner.

> +
> +  The iterator for worker lcores will also change:
> +  ``RTE_LCORE_FOREACH_SLAVE`` will be replaced with
> +  ``RTE_LCORE_FOREACH_WORKER``.
> +
> +  The ``master-lcore`` argument to testpmd will be replaced
> +  with ``initial-lcore``.
> -- 

With the above comment.

Acked-by: Bruce Richardson <bruce.richardson@intel.com>
Thomas Monjalon Aug. 6, 2020, 4:49 p.m. UTC | #2 
30/07/2020 02:57, Stephen Hemminger:
> Announce upcoming changes related to master/slave in reference
> to lcore.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  doc/guides/rel_notes/deprecation.rst | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 99c98062ffc2..7c60779f3e68 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -148,3 +148,20 @@ 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.
> +
> +* eal: To be more inclusive in choice of naming, the DPDK project
> +  will replace uses of master/slave in  the API's and command line arguments.

nit: double space
> +
> +  References to master/slave in relation to lcore will be renamed
> +  to initial/worker.  The function ``rte_get_master_lcore()``
> +  will be renamed to ``rte_get_initial_lcore()``.
> +  For the 20.11, release both names will be present and the

nit: comma misplaced

> +  old function will be marked with the deprecated tag.
> +  The old function will be removed in 21.11 version.
> +
> +  The iterator for worker lcores will also change:
> +  ``RTE_LCORE_FOREACH_SLAVE`` will be replaced with
> +  ``RTE_LCORE_FOREACH_WORKER``.
> +
> +  The ``master-lcore`` argument to testpmd will be replaced
> +  with ``initial-lcore``.

It doesn't say that the old argument will be kept for compatibility in 20.11.
Stephen Hemminger Aug. 6, 2020, 5 p.m. UTC | #3 
On Thu, 06 Aug 2020 18:49:28 +0200
Thomas Monjalon <thomas@monjalon.net> wrote:

> 30/07/2020 02:57, Stephen Hemminger:
> > Announce upcoming changes related to master/slave in reference
> > to lcore.
> > 
> > Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 17 +++++++++++++++++
> >  1 file changed, 17 insertions(+)
> > 
> > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> > index 99c98062ffc2..7c60779f3e68 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -148,3 +148,20 @@ 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.
> > +
> > +* eal: To be more inclusive in choice of naming, the DPDK project
> > +  will replace uses of master/slave in  the API's and command line arguments.  
> 
> nit: double space
> > +
> > +  References to master/slave in relation to lcore will be renamed
> > +  to initial/worker.  The function ``rte_get_master_lcore()``
> > +  will be renamed to ``rte_get_initial_lcore()``.
> > +  For the 20.11, release both names will be present and the  
> 
> nit: comma misplaced
> 
> > +  old function will be marked with the deprecated tag.
> > +  The old function will be removed in 21.11 version.
> > +
> > +  The iterator for worker lcores will also change:
> > +  ``RTE_LCORE_FOREACH_SLAVE`` will be replaced with
> > +  ``RTE_LCORE_FOREACH_WORKER``.
> > +
> > +  The ``master-lcore`` argument to testpmd will be replaced
> > +  with ``initial-lcore``.  
> 
> It doesn't say that the old argument will be kept for compatibility in 20.11.

Testpmd is not part of API/ABI compatibility area so it is not a hard
requirement.
Thomas Monjalon Aug. 6, 2020, 5:14 p.m. UTC | #4 
06/08/2020 19:00, Stephen Hemminger:
> On Thu, 06 Aug 2020 18:49:28 +0200
> Thomas Monjalon <thomas@monjalon.net> wrote:
> 
> > 30/07/2020 02:57, Stephen Hemminger:
> > > Announce upcoming changes related to master/slave in reference
> > > to lcore.
> > > 
> > > Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> > > ---
> > >  doc/guides/rel_notes/deprecation.rst | 17 +++++++++++++++++
> > >  1 file changed, 17 insertions(+)
> > > 
> > > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> > > index 99c98062ffc2..7c60779f3e68 100644
> > > --- a/doc/guides/rel_notes/deprecation.rst
> > > +++ b/doc/guides/rel_notes/deprecation.rst
> > > @@ -148,3 +148,20 @@ 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.
> > > +
> > > +* eal: To be more inclusive in choice of naming, the DPDK project
> > > +  will replace uses of master/slave in  the API's and command line arguments.  
> > 
> > nit: double space
> > > +
> > > +  References to master/slave in relation to lcore will be renamed
> > > +  to initial/worker.  The function ``rte_get_master_lcore()``
> > > +  will be renamed to ``rte_get_initial_lcore()``.
> > > +  For the 20.11, release both names will be present and the  
> > 
> > nit: comma misplaced
> > 
> > > +  old function will be marked with the deprecated tag.
> > > +  The old function will be removed in 21.11 version.
> > > +
> > > +  The iterator for worker lcores will also change:
> > > +  ``RTE_LCORE_FOREACH_SLAVE`` will be replaced with
> > > +  ``RTE_LCORE_FOREACH_WORKER``.
> > > +
> > > +  The ``master-lcore`` argument to testpmd will be replaced
> > > +  with ``initial-lcore``.  
> > 
> > It doesn't say that the old argument will be kept for compatibility in 20.11.
> 
> Testpmd is not part of API/ABI compatibility area so it is not a hard
> requirement.

Not really hard, but I would say it is annoying enough to consider
maintaining compatibility in testpmd for some time.
diff mbox series

Patch

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 99c98062ffc2..7c60779f3e68 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -148,3 +148,20 @@  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.
+
+* eal: To be more inclusive in choice of naming, the DPDK project
+  will replace uses of master/slave in  the API's and command line arguments.
+
+  References to master/slave in relation to lcore will be renamed
+  to initial/worker.  The function ``rte_get_master_lcore()``
+  will be renamed to ``rte_get_initial_lcore()``.
+  For the 20.11, release both names will be present and the
+  old function will be marked with the deprecated tag.
+  The old function will be removed in 21.11 version.
+
+  The iterator for worker lcores will also change:
+  ``RTE_LCORE_FOREACH_SLAVE`` will be replaced with
+  ``RTE_LCORE_FOREACH_WORKER``.
+
+  The ``master-lcore`` argument to testpmd will be replaced
+  with ``initial-lcore``.