<div dir="ltr">Please disregard the Community Lab DTS failure just reported on this patchseries. I need to reconfigure the testbed and rerun the test.</div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, May 2, 2025 at 11:11 AM Bruce Richardson <<a href="mailto:bruce.richardson@intel.com">bruce.richardson@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Following discussion earlier on this thread on the previous RFCs and<br>
patches submitted [1], this series is a slightly different take on<br>
solving the same fundamental problem - making it easy to use CPU cores<br>
with ids >= RTE_MAX_LCORE, without having massive amounts of code<br>
complexity internally. As previously discussed[2] this patchset<br>
implements proposal "2a" and does the following:<br>
<br>
* marks the old coremask parameter (-c) and service core mask (-s) as<br>
deprecated, so they can be dropped in future. Everything should now be<br>
done using core lists where possible.<br>
* consolidates the, previously separate but similar, "-l" and "--lcores"<br>
flags into just short and long versions of the same thing. This does<br>
not affect the "--lcores" flag at all, but expands the scope of what<br>
was possible with "-l".<br>
* a new flag, with long/short versions, is added as "-L" to allow the<br>
user to just specify a range or set of physical CPU core ids to be<br>
used by the app, and have them automatically mapped internally to a<br>
set of lcore ids starting at 0. This provides the "easy solution" to<br>
the original problem stated above.<br>
<br>
For completeness, since it's in the same area, the previously separate<br>
documentation patch, adding examples for the "--lcores" parameter, is<br>
rolled into this series as patch 3.<br>
<br>
[1] <a href="https://patches.dpdk.org/project/dpdk/list/?series=34911&state=*" rel="noreferrer" target="_blank">https://patches.dpdk.org/project/dpdk/list/?series=34911&state=*</a><br>
[2] <a href="https://inbox.dpdk.org/dev/Z_PIBLQIT3eu6Ab3@bricha3-mobl1.ger.corp.intel.com/#t" rel="noreferrer" target="_blank">https://inbox.dpdk.org/dev/Z_PIBLQIT3eu6Ab3@bricha3-mobl1.ger.corp.intel.com/#t</a><br>
<br>
Bruce Richardson (4):<br>
eal: deprecate old coremask-based EAL parameters<br>
eal: merge corelist and core mapping options<br>
doc: provide examples of using lcores EAL parameter<br>
eal: add option for auto-mapping cpu ids to low lcore ids<br>
<br>
doc/guides/eventdevs/dlb2.rst | 6 +-<br>
doc/guides/faq/faq.rst | 8 +-<br>
doc/guides/linux_gsg/build_sample_apps.rst | 7 +-<br>
doc/guides/linux_gsg/eal_args.include.rst | 61 ++++--<br>
.../prog_guide/env_abstraction_layer.rst | 3 +-<br>
doc/guides/prog_guide/meson_ut.rst | 2 +-<br>
doc/guides/prog_guide/multi_proc_support.rst | 2 +-<br>
doc/guides/prog_guide/service_cores.rst | 8 +-<br>
doc/guides/rel_notes/deprecation.rst | 10 +<br>
doc/guides/sample_app_ug/ip_frag.rst | 7 +-<br>
doc/guides/sample_app_ug/ip_reassembly.rst | 7 +-<br>
doc/guides/sample_app_ug/multi_process.rst | 14 +-<br>
doc/guides/sample_app_ug/qos_scheduler.rst | 2 +-<br>
doc/guides/sample_app_ug/test_pipeline.rst | 2 +-<br>
doc/guides/tools/testbbdev.rst | 2 +-<br>
lib/eal/common/eal_common_options.c | 202 +++++++-----------<br>
lib/eal/common/eal_options.h | 6 +-<br>
lib/eal/windows/include/sched.h | 1 +<br>
18 files changed, 168 insertions(+), 182 deletions(-)<br>
<br>
--<br>
2.45.2<br>
<br>
</blockquote></div>