From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3AB6B466C2; Sun, 4 May 2025 23:22:25 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 2A82B402A8; Sun, 4 May 2025 23:22:25 +0200 (CEST) Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) by mails.dpdk.org (Postfix) with ESMTP id 686AE40156 for ; Sun, 4 May 2025 23:22:23 +0200 (CEST) Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-2295d78b433so36071265ad.2 for ; Sun, 04 May 2025 14:22:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1746393742; x=1746998542; darn=dpdk.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=uDBK6KDEOW1SL9TGKZp2Jr5owXTfEEtFCn2X9gE0dsY=; b=aSFvZb+lg6iZD22jS0l9hcrbGl0F3t1Hdmud9bEjLJz9KmZs9w52Hv2flM2n4334q5 rgo8hE8e8K2PM3vhNAZQIyL6I5XStTKsCeS2dSXqfUSLmvOTJdhpaLjUOGrbFx/QSje6 qOmiyEVSNghkK0W/FQNmHYHoLBv+N5CRWQvfY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1746393742; x=1746998542; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=uDBK6KDEOW1SL9TGKZp2Jr5owXTfEEtFCn2X9gE0dsY=; b=IPhn/vbSM31Se4rCJNPqBnzub/440Aovdi13EojXf+wUuSy7KcPJDyyw93rieMCMLe SjCV9qgRJKJxfVp/ZTd62u4lJvJBBE7MMp+ghEIXdLFnosk8z6nGiNOoJM2dP3sb0ujN WbuUIOQ7NnpCmJNaglv4+knpgW+B9nnltvl6eNbynjwUsoAa0AdVWXLaSla6f6mujMrV W/n50yehYejLdHNFLXkguD1/2yqHZFwmEYKzDgqRn1gxmTgQgOVRs8L0//91OQDgHWeU TSe9ZVkx3rccePJx5qOMOnazrr8e2T100Hin5FhOpYsWWvRjczJJM8WYSDZ1NEarV5Xq JG2g== X-Gm-Message-State: AOJu0Yx748MIvsgAgX0sYiNfswA2hc4p4+O9sEVXC3EtVivmkZL220Aw iuUnWHgaymAzIvs3TZ+r8XVZqeIYcJVK3qLW1iqZ7AjDJ+H5xyYaWioEdxW37BZIx2Y4TiyeU3P xWN68W95QXF7ZsuTin6EAhCBx3qbpgq69h6O3LA== X-Gm-Gg: ASbGnctxcfyb40odN7YIxTdgkhl5eFUVCzkvw+NKH8xjlWj5fwqLv/MYgaQ1dHI9Hg4 CqxDFlAptxFajOGNCDVb+2RCFttTg2eEdUs14cnfrFTcRSn768zjceR+G7wTC128jiI5rpZ+Lw3 TI1f797IRGTcXcVDpyZU4VCjqLywtKbthdhryVP5HNO0ha7xxwq+nbqsA= X-Google-Smtp-Source: AGHT+IH9I0eqW85i0Ut5W9aCZVyUMcalsIuHGQ3qn1QKp9SRDC1bUc1ls4hOgO+kn5PrfJITLVggsSOk01WmcvwZSD4= X-Received: by 2002:a17:902:e750:b0:224:1221:1ab4 with SMTP id d9443c01a7336-22e1e8e686bmr82391045ad.22.1746393742538; Sun, 04 May 2025 14:22:22 -0700 (PDT) MIME-Version: 1.0 References: <20250313113829.1480907-1-bruce.richardson@intel.com> <20250502151134.536799-1-bruce.richardson@intel.com> In-Reply-To: <20250502151134.536799-1-bruce.richardson@intel.com> From: Patrick Robb Date: Sun, 4 May 2025 17:17:49 -0400 X-Gm-Features: ATxdqUEB-kKclZvcFyNGEE8Mr3MpM__gVP9Z_4pvfYWIaTgvLr-B84DQvkEyYEg Message-ID: Subject: Re: [PATCH v3 0/4] rework and expand EAL lcore options To: Bruce Richardson Cc: dev@dpdk.org, david.marchand@redhat.com, mb@smartsharesystems.com, stephen@networkplumber.org Content-Type: multipart/alternative; boundary="000000000000a98b13063455fa89" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org --000000000000a98b13063455fa89 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Please disregard the Community Lab DTS failure just reported on this patchseries. I need to reconfigure the testbed and rerun the test. On Fri, May 2, 2025 at 11:11=E2=80=AFAM Bruce Richardson wrote: > Following discussion earlier on this thread on the previous RFCs and > patches submitted [1], this series is a slightly different take on > solving the same fundamental problem - making it easy to use CPU cores > with ids >=3D RTE_MAX_LCORE, without having massive amounts of code > complexity internally. As previously discussed[2] this patchset > implements proposal "2a" and does the following: > > * marks the old coremask parameter (-c) and service core mask (-s) as > deprecated, so they can be dropped in future. Everything should now be > done using core lists where possible. > * consolidates the, previously separate but similar, "-l" and "--lcores" > flags into just short and long versions of the same thing. This does > not affect the "--lcores" flag at all, but expands the scope of what > was possible with "-l". > * a new flag, with long/short versions, is added as "-L" to allow the > user to just specify a range or set of physical CPU core ids to be > used by the app, and have them automatically mapped internally to a > set of lcore ids starting at 0. This provides the "easy solution" to > the original problem stated above. > > For completeness, since it's in the same area, the previously separate > documentation patch, adding examples for the "--lcores" parameter, is > rolled into this series as patch 3. > > [1] https://patches.dpdk.org/project/dpdk/list/?series=3D34911&state=3D* > [2] > https://inbox.dpdk.org/dev/Z_PIBLQIT3eu6Ab3@bricha3-mobl1.ger.corp.intel.= com/#t > > Bruce Richardson (4): > eal: deprecate old coremask-based EAL parameters > eal: merge corelist and core mapping options > doc: provide examples of using lcores EAL parameter > eal: add option for auto-mapping cpu ids to low lcore ids > > 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 | 61 ++++-- > .../prog_guide/env_abstraction_layer.rst | 3 +- > 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 | 202 +++++++----------- > lib/eal/common/eal_options.h | 6 +- > lib/eal/windows/include/sched.h | 1 + > 18 files changed, 168 insertions(+), 182 deletions(-) > > -- > 2.45.2 > > --000000000000a98b13063455fa89 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Please disregard the Community Lab DTS failure just=C2=A0r= eported on this patchseries. I need to reconfigure the testbed and rerun th= e test.

On Fri, May 2, 2025 at 11:11=E2=80=AFAM Bruce= Richardson <bruce.richard= son@intel.com> wrote:
Following discussion earlier on this thread on the previous RF= Cs and
patches submitted [1], this series is a slightly different take on
solving the same fundamental problem - making it easy to use CPU cores
with ids >=3D RTE_MAX_LCORE, without having massive amounts of code
complexity internally. As previously discussed[2] this patchset
implements proposal "2a" and does the following:

* marks the old coremask parameter (-c) and service core mask (-s) as
=C2=A0 deprecated, so they can be dropped in future. Everything should now = be
=C2=A0 done using core lists where possible.
* consolidates the, previously separate but similar, "-l" and &qu= ot;--lcores"
=C2=A0 flags into just short and long versions of the same thing. This does=
=C2=A0 not affect the "--lcores" flag at all, but expands the sco= pe of what
=C2=A0 was possible with "-l".
* a new flag, with long/short versions, is added as "-L" to allow= the
=C2=A0 user to just specify a range or set of physical CPU core ids to be =C2=A0 used by the app, and have them automatically mapped internally to a<= br> =C2=A0 set of lcore ids starting at 0. This provides the "easy solutio= n" to
=C2=A0 the original problem stated above.

For completeness, since it's in the same area, the previously separate<= br> documentation patch, adding examples for the "--lcores" parameter= , is
rolled into this series as patch 3.

[1] https://patches.dpdk.org= /project/dpdk/list/?series=3D34911&state=3D*
[2] https://inbox.dpd= k.org/dev/Z_PIBLQIT3eu6Ab3@bricha3-mobl1.ger.corp.intel.com/#t

Bruce Richardson (4):
=C2=A0 eal: deprecate old coremask-based EAL parameters
=C2=A0 eal: merge corelist and core mapping options
=C2=A0 doc: provide examples of using lcores EAL parameter
=C2=A0 eal: add option for auto-mapping cpu ids to low lcore ids

=C2=A0doc/guides/eventdevs/dlb2.rst=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 =C2=A06 +-
=C2=A0doc/guides/faq/faq.rst=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 =C2=A08 +-
=C2=A0doc/guides/linux_gsg/build_sample_apps.rst=C2=A0 =C2=A0 |=C2=A0 =C2= =A07 +-
=C2=A0doc/guides/linux_gsg/eal_args.include.rst=C2=A0 =C2=A0 =C2=A0|=C2=A0 = 61 ++++--
=C2=A0.../prog_guide/env_abstraction_layer.rst=C2=A0 =C2=A0 =C2=A0 |=C2=A0 = =C2=A03 +-
=C2=A0doc/guides/prog_guide/meson_ut.rst=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 |=C2=A0 =C2=A02 +-
=C2=A0doc/guides/prog_guide/multi_proc_support.rst=C2=A0 |=C2=A0 =C2=A02 +-=
=C2=A0doc/guides/prog_guide/service_cores.rst=C2=A0 =C2=A0 =C2=A0 =C2=A0|= =C2=A0 =C2=A08 +-
=C2=A0doc/guides/rel_notes/deprecation.rst=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 |=C2=A0 10 +
=C2=A0doc/guides/sample_app_ug/ip_frag.rst=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 |=C2=A0 =C2=A07 +-
=C2=A0doc/guides/sample_app_ug/ip_reassembly.rst=C2=A0 =C2=A0 |=C2=A0 =C2= =A07 +-
=C2=A0doc/guides/sample_app_ug/multi_process.rst=C2=A0 =C2=A0 |=C2=A0 14 +-=
=C2=A0doc/guides/sample_app_ug/qos_scheduler.rst=C2=A0 =C2=A0 |=C2=A0 =C2= =A02 +-
=C2=A0doc/guides/sample_app_ug/test_pipeline.rst=C2=A0 =C2=A0 |=C2=A0 =C2= =A02 +-
=C2=A0doc/guides/tools/testbbdev.rst=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 |=C2=A0 =C2=A02 +-
=C2=A0lib/eal/common/eal_common_options.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0| 202 +++++++-----------
=C2=A0lib/eal/common/eal_options.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 =C2=A0 =C2=A0 |=C2=A0 =C2=A06 +-
=C2=A0lib/eal/windows/include/sched.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0|=C2=A0 =C2=A01 +
=C2=A018 files changed, 168 insertions(+), 182 deletions(-)

--
2.45.2

--000000000000a98b13063455fa89--