Re: [PATCH v2] eal: deprecate old coremask-based EAL parameters

[David Marchand <david.marchand@redhat.com>]

On Tue, Jul 1, 2025 at 9:56 AM David Marchand <david.marchand@redhat.com> wrote:
>
> On Mon, Jun 30, 2025 at 5:38 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.
> >
> > Depends-on: series-35540 ("use core lists not masks in documentation")
> >
> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> > Acked-by: Vipin Varghese <vipin.varghese@amd.com>
> > Acked-By: Anatoly Burakov <anatoly.burakov@intel.com>
> > Acked-by: Konstantin Ananyev <konstantin.ananyev@huawei.com>
>
> Acked-by: David Marchand <david.marchand@redhat.com>

I realised the test-null.sh script needs tweaking.
I'll squash this:

diff --git a/devtools/test-null.sh b/devtools/test-null.sh
index 6cd34f64f1..e3ac168ce3 100755
--- a/devtools/test-null.sh
+++ b/devtools/test-null.sh
@@ -7,7 +7,7 @@

 build=${1:-build} # first argument can be the build directory
 testpmd=$1 # or first argument can be the testpmd path
-coremask=${2:-3} # default using cores 0 and 1
+corelist=${2:-0-1} # default using cores 0 and 1
 eal_options=$3
 testpmd_options=$4

@@ -28,6 +28,6 @@ fi

 (sleep 1 && echo stop) |
 # testpmd only needs 20M, make it x2 (default number of cores) for NUMA systems
-$testpmd -c $coremask --no-huge -m 40 \
+$testpmd -l $corelist --no-huge -m 40 \
        $libs -a 0:0.0 --vdev net_null1 --vdev net_null2 $eal_options -- \
        --no-mlockall --total-num-mbufs=2048 $testpmd_options -ia


-- 
David Marchand