From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH v2 2/2] meson: add tests app to build Date: Wed, 20 Dec 2017 12:20:01 +0000 Message-ID: <20171220122001.GC5200@bricha3-MOBL3.ger.corp.intel.com> References: <1513598038-148115-1-git-send-email-harry.van.haaren@intel.com> <1513768592-112485-1-git-send-email-harry.van.haaren@intel.com> <1513768592-112485-3-git-send-email-harry.van.haaren@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org To: Harry van Haaren Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 1EBE61B052 for ; Wed, 20 Dec 2017 13:20:05 +0100 (CET) Content-Disposition: inline In-Reply-To: <1513768592-112485-3-git-send-email-harry.van.haaren@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Wed, Dec 20, 2017 at 11:16:32AM +0000, Harry van Haaren wrote: > This patch enables the test/test app to be built. It also adds > the test binary to be a meson-test, which allows the meson test > infrastructure to be used to run tests. > > Tests are listed using the same test binary, however each test > sets a different DPDK_TEST environment variable. The string contents > of this DPDK_TEST env var is entered in the command line interface. > As such, the familiar test names such as "ring_perf_autotest" etc > are valid tests to run using this meson test infrastructure. > > Note that the tests are run serially, given that we cannot run > multiple primary processes at a time. As each test must initialize > EAL this takes some time depending on the number of hugepages. > In future, we could improve this to run multiple tests from one > EAL init, but it is out of scope for this patchset. > > Finally, an option to build the tests is added to the meson build > options. When disabled, the unit test code in test/test is not > compiled. The default is set to 'true'. To disable, run: > > $ meson configure -Dtests=false > > Signed-off-by: Harry van Haaren > > --- > > v2: > - Updated for SPDX license headers (Bruce) Acked-by: Bruce Richardson