From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr1-f68.google.com (mail-wr1-f68.google.com [209.85.221.68]) by dpdk.org (Postfix) with ESMTP id 591EF1B42E for ; Wed, 6 Feb 2019 18:10:09 +0100 (CET) Received: by mail-wr1-f68.google.com with SMTP id z15so6395883wrn.1 for ; Wed, 06 Feb 2019 09:10:09 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:content-transfer-encoding:mime-version; bh=3zr4FsA7B5DwsycRrv+ngqIR6KPfdY8FAavntW8iAF4=; b=J2wmyls4Sg6oUko1pkAQxbfwNQ+NBX/LC/vE8URxHwdKXuk6YkA+lZjSTkkj49lspw YSV3fI6Gk56BjUJ+vXTVRkXlxLmxYzvlFOVg5IGLt+/Ba1IhXTdVJJKJTgSMUvsR+jNY E1qB5mTPnuXR4qGnsG/sIFhGoSdX0vcVwHT41VpgMRLhiTFdLtds/6DedzmH0S1HPXVz HuVeq+/7y/ysxsDOLAvBKo1IbPD4eECpkpM1GvzPet8H3Qe9cy/AAsQFggMj4A4pxVKA Gr6rVeZppYSL6uZ5S/E8kpjqEObc2oEzlr/ql6CoiGGD7CFklWH6jDWoV3pVuvXawuNI 6Zow== X-Gm-Message-State: AHQUAuasC+dUQpfZW86WSlmaBrv3AiKci0jVtNoDYfw1Mp6o8jefohVX nlPlvQSYLZWmm63oQ1ZhUnQ= X-Google-Smtp-Source: AHgI3IbnfcqXHrEpjtQN8jl+SWSQq0aMP5TOCZHsV7i6THZzvHbOHd8CPO3c8XxgW5xwC7ILg9bZow== X-Received: by 2002:adf:ae1a:: with SMTP id x26mr8535119wrc.0.1549473008923; Wed, 06 Feb 2019 09:10:08 -0800 (PST) Received: from localhost ([2a01:4b00:f419:6f00:b833:9472:a517:a18f]) by smtp.gmail.com with ESMTPSA id 2sm6629276wrj.27.2019.02.06.09.10.07 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 06 Feb 2019 09:10:07 -0800 (PST) Message-ID: <1549473006.22781.2.camel@debian.org> From: Luca Boccassi To: Bruce Richardson , Chas Williams <3chas3@gmail.com> Cc: dev@dpdk.org Date: Wed, 06 Feb 2019 17:10:06 +0000 In-Reply-To: <20190131093203.GD122080@bricha3-MOBL.ger.corp.intel.com> References: <20190131021858.5832-1-3chas3@gmail.com> <20190131093203.GD122080@bricha3-MOBL.ger.corp.intel.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Mailer: Evolution 3.22.6-1+deb9u1 Mime-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH] build: add option to override max ethports X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Feb 2019 17:10:09 -0000 On Thu, 2019-01-31 at 09:32 +0000, Bruce Richardson wrote: > On Wed, Jan 30, 2019 at 09:18:58PM -0500, Chas Williams wrote: > > Allow users and packagers to override the default RTE_MAX_ETHPORTS. > > This adds a new meson option, max_ethports which defaults to the > > current value. > >=20 > > Signed-off-by: Chas Williams <3chas3@gmail.com> >=20 > Agree with the idea. Just one small comment below re alphabetical > order. >=20 > Otherwise: > Acked-by: Bruce Richardson >=20 > > --- > > =C2=A0config/meson.build=C2=A0=C2=A0| 1 + > > =C2=A0config/rte_config.h | 1 - > > =C2=A0meson_options.txt=C2=A0=C2=A0=C2=A0| 2 ++ > > =C2=A03 files changed, 3 insertions(+), 1 deletion(-) > >=20 > > diff --git a/config/meson.build b/config/meson.build > > index db32499b3..e59507065 100644 > > --- a/config/meson.build > > +++ b/config/meson.build > > @@ -103,6 +103,7 @@ endforeach > > =C2=A0# set other values pulled from the build options > > =C2=A0dpdk_conf.set('RTE_MAX_LCORE', get_option('max_lcores')) > > =C2=A0dpdk_conf.set('RTE_MAX_NUMA_NODES', get_option('max_numa_nodes')) > > +dpdk_conf.set('RTE_MAX_ETHPORTS', get_option('max_ethports')) > > =C2=A0dpdk_conf.set('RTE_LIBEAL_USE_HPET', get_option('use_hpet')) > > =C2=A0dpdk_conf.set('RTE_EAL_ALLOW_INV_SOCKET_ID', > > get_option('allow_invalid_socket_id')) > > =C2=A0# values which have defaults which may be overridden > > diff --git a/config/rte_config.h b/config/rte_config.h > > index d3732e953..7606f5d7b 100644 > > --- a/config/rte_config.h > > +++ b/config/rte_config.h > > @@ -51,7 +51,6 @@ > > =C2=A0#define RTE_PKTMBUF_HEADROOM 128 > > =C2=A0 > > =C2=A0/* ether defines */ > > -#define RTE_MAX_ETHPORTS 32 > > =C2=A0#define RTE_MAX_QUEUES_PER_PORT 1024 > > =C2=A0#define RTE_ETHDEV_QUEUE_STAT_CNTRS 16 > > =C2=A0#define RTE_ETHDEV_RXTX_CALLBACKS 1 > > diff --git a/meson_options.txt b/meson_options.txt > > index 574054597..e54eedc66 100644 > > --- a/meson_options.txt > > +++ b/meson_options.txt > > @@ -28,3 +28,5 @@ option('use_hpet', type: 'boolean', value: false, > > =C2=A0 description: 'use HPET timer in EAL') > > =C2=A0option('tests', type: 'boolean', value: true, > > =C2=A0 description: 'build unit tests') > > +option('max_ethports', type: 'string', value: '32', > > + description: 'maximum number of Ethernet devices in EAL') > > --=C2=A0 >=20 > We are trying to keep the options in alphabetical order, so this > setting > should be just before "max_lcores" rather than at the end. >=20 > Interestingly, I see that since 0.45 there is now the option of an > "integer" option type. We should probably switch to use that once we > bump > our minimum meson version. [Luca, perhaps another change for your > set?] > Ref: http://mesonbuild.com/Build-options.html#integers Sounds good, I've added this change in v6 of the series --=20 Kind regards, Luca Boccassi