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 26539A00C2; Wed, 8 Dec 2021 19:29:48 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id ADD8A4111B; Wed, 8 Dec 2021 19:29:47 +0100 (CET) Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mails.dpdk.org (Postfix) with ESMTP id B3DBD410F3 for ; Wed, 8 Dec 2021 19:29:45 +0100 (CET) Received: by linux.microsoft.com (Postfix, from userid 1061) id F3A2B20B7179; Wed, 8 Dec 2021 10:29:44 -0800 (PST) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com F3A2B20B7179 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1638988185; bh=91ee6hkzsR33qy48JB/YUNQcJGbMV4uAejSCgPm9wUc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=MJWGOI7ts0Lb6fv9cgrkv5Uwsrai7K0n5XIAjsjpdlxK5Lq70nEOaOTXN7tuB+3Vb DcFAe7kho25YgE4WysSjid5cfTkL2U7ELBbA6mxL1vn04eBbF58se9SWjcEMBeiqIY OUD7nH0PAQ1su0sNvL3I7cjNHtuh7eaeh5uSek/w= Date: Wed, 8 Dec 2021 10:29:44 -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: <20211208182944.GB28299@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> <20211208181927.GA28299@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> 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 06:23:30PM +0000, Bruce Richardson wrote: > On Wed, Dec 08, 2021 at 10:19:27AM -0800, Jie Zhou wrote: > > 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 > > A line to two added to the commit log is probably sufficient. > > Acked-by: Bruce Richardson Thanks Bruce. Will add the info into commit message in V14.