From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] test/test: allow taking extra arguments from environment Date: Tue, 06 Nov 2018 02:59:33 +0100 Message-ID: <3203192.f4LorbPn4K@xps> References: <20181012153404.46098-1-bruce.richardson@intel.com> <1539366133.8721.47.camel@debian.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, Luca Boccassi To: Bruce Richardson Return-path: Received: from out5-smtp.messagingengine.com (out5-smtp.messagingengine.com [66.111.4.29]) by dpdk.org (Postfix) with ESMTP id 5C0B758C4 for ; Tue, 6 Nov 2018 02:59:36 +0100 (CET) In-Reply-To: <1539366133.8721.47.camel@debian.org> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 12/10/2018 19:42, Luca Boccassi: > On Fri, 2018-10-12 at 16:34 +0100, Bruce Richardson wrote: > > When running unit tests automatically, either via script, from meson, > > or otherwise, the same set of options may be used for each run, for > > example to set a standard coremask to be used for all tests. > > > > To facilitate this, this patch adds support for the test binary > > taking > > additional EAL parameters from the environment and appending them to > > the > > argc/argv list passed to eal init. This allows parameter modification > > without having to edit test scripts etc. > > > > There are now two environment variables which can be used for running > > tests: > > * DPDK_TEST - (added previously) passes the test name to be run > > automatically rather than running the app > > interactively. > > Used by "meson test" when running tests individually > > or > > as part of a suite. > > > > * DPDK_TEST_PARAMS - new parameter to specify the commandline > > arguments > > to use with the test binary. For example to run a > > test, > > or tests, on only 16 lcores, and to skip pci scan we > > can > > set this to "-l 0-15 --no-pci". > > > > Signed-off-by: Bruce Richardson > > --- > > doc/guides/rel_notes/release_18_11.rst | 8 ++++++ > > test/test/test.c | 34 > > +++++++++++++++++++++++++- > > 2 files changed, 41 insertions(+), 1 deletion(-) > > Was just thinking that something like this would be useful! > > Tested-by: Luca Boccassi Applied, thanks