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 80137A00C2; Wed, 8 Dec 2021 19:19:30 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 12CED4111B; Wed, 8 Dec 2021 19:19:30 +0100 (CET) Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mails.dpdk.org (Postfix) with ESMTP id CE6C3410F3 for ; Wed, 8 Dec 2021 19:19:27 +0100 (CET) Received: by linux.microsoft.com (Postfix, from userid 1061) id 2194920B7179; Wed, 8 Dec 2021 10:19:27 -0800 (PST) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 2194920B7179 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1638987567; bh=lE7BSFULmpiO992ExwNvVPENtfxNFvsOo0SWAwoaYdY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=JXDENaVfpF71moF2pxP9WFCu2prKKyE98x8WdgPifMfWcxEnMLJ+lsql1xbxVaVGf aRzjCf1Vv4IEORa/g/FIZdAkRpCN3kgrpKvsSx/8TyTNiXAzpkAlhiaxA+kXVBTxFx vDFGDehxzOyc2e+uW4mhXu8ebwEvt34Zu8a1wuIM= Date: Wed, 8 Dec 2021 10:19:27 -0800 From: Jie Zhou To: Bruce Richardson Cc: dev@dpdk.org, dmitry.kozliuk@gmail.com, roretzla@microsoft.com, navasile@linux.microsoft.com, dmitrym@microsoft.com, pallavi.kadam@intel.com, talshn@nvidia.com, thomas@monjalon.net, aconole@redhat.com Subject: Re: [PATCH v13 11/11] app/test: enable unit test on Windows Message-ID: <20211208181927.GA28299@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> References: <1638912263-7054-1-git-send-email-jizh@linux.microsoft.com> <1638928262-13177-1-git-send-email-jizh@linux.microsoft.com> <1638928262-13177-12-git-send-email-jizh@linux.microsoft.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) 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 On Wed, Dec 08, 2021 at 04:57:52PM +0000, Bruce Richardson wrote: > On Tue, Dec 07, 2021 at 05:51:02PM -0800, Jie Zhou wrote: > > Enable a subset of unit tests for Windows CI > > > > - For driver tests, driver owners should enable corresponding tests when > > enabling driver for Windows. For example, the cryptodev tests will be > > enabled by "patch-18949: app/test: enable crypto unit tests on Windows" > > (which depends on this patchset to be merged). > > - For dump tests, currently the tests hang on Windows which require > > further investigation. > > > > Signed-off-by: Jie Zhou > > > > Mostly these changes look ok to me. One small query below. > > /Bruce > > --- > > app/test/meson.build | 111 ++++++++++++++++++++++--------------------- > > 1 file changed, 58 insertions(+), 53 deletions(-) > > > > diff --git a/app/test/meson.build b/app/test/meson.build > > index 97ee83029e..fcf38729e7 100644 > > --- a/app/test/meson.build > > +++ b/app/test/meson.build > > @@ -1,12 +1,6 @@ > > # SPDX-License-Identifier: BSD-3-Clause > > # Copyright(c) 2017 Intel Corporation > > > > -if is_windows > > - build = false > > > > > # The following linkages are an exception to allow running the > > # unit tests without requiring that the developer install the > > # DPDK libraries. Explicit linkage of drivers (plugin libraries) > > @@ -385,7 +390,7 @@ if dpdk_conf.has('RTE_LIB_METRICS') > > test_sources += ['test_metrics.c'] > > fast_tests += [['metrics_autotest', true]] > > endif > > -if dpdk_conf.has('RTE_LIB_TELEMETRY') > > +if not is_windows and dpdk_conf.has('RTE_LIB_TELEMETRY') > > test_sources += ['test_telemetry_json.c', 'test_telemetry_data.c'] > > fast_tests += [['telemetry_json_autotest', true], ['telemetry_data_autotest', true]] > > If telemetry is enabled, is there a reason these won't run on Windows? Hi Bruce, the telemetry test has many POSIX socket specific codes which requires replacement for Windows. We will work on investigation of enabling more tests (including the telemetry tests) after this patchset. Sorry I forgot to mention that in the commit message, but it is on my list. The current goal is to get this patchset merged asap before another upstream churn to rebase, rework, etc. If you prefer me to send out a V14 to add this info into commit message or remove this change from meson.build but add test stub into the specific test files, please just let me know. Thanks for your understanding. -- Jie