Re: [PATCH v5 1/3] eal: deprecate old coremask-based EAL parameters

[Bruce Richardson <bruce.richardson@intel.com>]

On Mon, Jun 23, 2025 at 06:01:00PM +0200, David Marchand wrote:
> On Tue, May 27, 2025 at 5:29 PM Bruce Richardson
> <bruce.richardson@intel.com> wrote:
> >
> > As the number of cores/cpus on platforms has increased over the years,
> > the use of coremasks rather than core-lists for identifying DPDK cores
> > has become more and more unwieldy. At this point, let's deprecate the
> > coremask-based EAL parameters for future removal, and point users to the
> > core-list based versions instead.
> >
> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> > ---
> >  doc/guides/eventdevs/dlb2.rst                |  6 +++---
> >  doc/guides/faq/faq.rst                       |  8 +++-----
> >  doc/guides/linux_gsg/build_sample_apps.rst   |  7 +++----
> >  doc/guides/linux_gsg/eal_args.include.rst    |  8 ++------
> >  doc/guides/prog_guide/meson_ut.rst           |  2 +-
> >  doc/guides/prog_guide/multi_proc_support.rst |  2 +-
> >  doc/guides/prog_guide/service_cores.rst      |  8 ++++----
> >  doc/guides/rel_notes/deprecation.rst         | 10 ++++++++++
> >  doc/guides/sample_app_ug/ip_frag.rst         |  7 +------
> >  doc/guides/sample_app_ug/ip_reassembly.rst   |  7 +------
> >  doc/guides/sample_app_ug/multi_process.rst   | 14 +++++---------
> >  doc/guides/sample_app_ug/qos_scheduler.rst   |  2 +-
> >  doc/guides/sample_app_ug/test_pipeline.rst   |  2 +-
> >  doc/guides/tools/testbbdev.rst               |  2 +-
> >  lib/eal/common/eal_common_options.c          |  6 ++++++
> >  15 files changed, 43 insertions(+), 48 deletions(-)
> 
> This patch will need some rebase.
> 
> I see you fixed a few docs, but there are quite some misses in
> copy/pasted blocks:
> 
I'll take a look and respin this.

I'm think of separating it into two patches: first patch with the doc
updates to remove use of -c flag. This should happen independent of the
deprecation, I think, and second patch just the deprecation notice. Does
that sound reasonable?

/Bruce