From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 62537A00C5; Sun, 26 Apr 2020 17:38:28 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id AB7BB1BF9D; Sun, 26 Apr 2020 17:38:27 +0200 (CEST) Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id 155681BF8E for ; Sun, 26 Apr 2020 17:38:26 +0200 (CEST) Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id A22275C0106; Sun, 26 Apr 2020 11:38:25 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute7.internal (MEProxy); Sun, 26 Apr 2020 11:38:25 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm1; bh= KKLOx8j9ahbTEREEbFgR0jIVZ/FPRDphxqljUyIFnMc=; b=tZaOzQCFxGZ1CbCD OmGyJwsjTEsqA7gxmxK8v11JOa1CgZNQRsLydzfAVsFzMxuDB7tJ5CH4kr1TJpAL LgAsVnUpmB2u7C5vCiV6M67QZnudY6hB+SwHPH11aLrN7RT8Mtp7VVWnrK11UVGt g5kb/+XkalLOJIUZqRxNPpv8aaGhjqYxqvJqwPahvvr8Tn+KrqwLi4IRhOGopqeP b9DdriJDorg4Dg6Ox2Yh1OtP5y9xDKW7y0FqyDw6SC1Q1pc1ddo2YlgSYV6vuDBv a0046AzBbR5F0gWorPdTi64cMearn/mkPovybrbj/gtv286AC3/U55RrhRlygJ2N P93XCQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; bh=KKLOx8j9ahbTEREEbFgR0jIVZ/FPRDphxqljUyIFn Mc=; b=TylG7C86E59swcT02FmZvT6GUdg/igcLyvbGPugK9yeFT9Iaceu0G9kUO dPVN/fu5u70C0hhbdJrR8sIRZSy6XShMpY+gj50+wT3lHX0wcYjdWUsttQ0nuCAA rT/RIKy2sbN/u4hkWLrUhWLhHDWHSgtyAWYS3DJf6LN3OuEiuuxmjIuJDlOQZkKP tVnWzHcLGPiXSFY9BFon6tWrn1tscQaTE9HpvfBxmmYkNlZv2gGDZkKHa7x5kh95 R2rKNhJylpxunmyzMXvhCYL/IXidf+8BUWTIvjeKhh0x6huFzy4iNVJzlePz1ZNC Hn7MW7qmmv0yttKf8pPU4HeS6/+0g== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduhedrheejgdelvdcutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefhvffufffkjghfggfgtgesthfuredttddtvdenucfhrhhomhepvfhhohhmrghs ucfoohhnjhgrlhhonhcuoehthhhomhgrshesmhhonhhjrghlohhnrdhnvghtqeenucfkph epjeejrddufeegrddvtdefrddukeegnecuvehluhhsthgvrhfuihiivgeptdenucfrrghr rghmpehmrghilhhfrhhomhepthhhohhmrghssehmohhnjhgrlhhonhdrnhgvth X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id EAF00328005E; Sun, 26 Apr 2020 11:38:23 -0400 (EDT) From: Thomas Monjalon To: Dmitry Kozlyuk Cc: dev@dpdk.org, Pallavi Kadam , Narcisa Ana Maria Vasile , Ranjit Menon , Dmitry Kozlyuk , Jerin Jacob , Sunil Kumar Kori , Harini Ramakrishnan , Omar Cardona , David Marchand Date: Sun, 26 Apr 2020 17:38:22 +0200 Message-ID: <17842859.gIjEHFZm8d@thomas> In-Reply-To: <20200426152819.2496610-4-dmitry.kozliuk@gmail.com> References: <20200426032245.2437733-1-dmitry.kozliuk@gmail.com> <20200426152819.2496610-1-dmitry.kozliuk@gmail.com> <20200426152819.2496610-4-dmitry.kozliuk@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v2 3/3] eal/windows: fix build by enabling trace compilation 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" I think this patch is doing too many things at once. Why not just disabling tracing on Windows for now, and apply proper patches for memory management, timer, endianness, etc in 20.08? Some cosmetic comments below, 26/04/2020 17:28, Dmitry Kozlyuk: > Replace clock_gettime(CLOCK_REALTIME) with C11 timespec_get(). > Implementation is provided for MinGW-w64 that misses this function. > > Provide minimum viable implementations of malloc and timer functions > used by tracing. Regex stubs are already present in Windows EAL. > > Fixes: 185b7dc1d467 ("trace: save bootup timestamp") > Fixes: 321dd5f8fa62 ("trace: add internal init and fini interface") > Reported-by: Pallavi Kadam > Signed-off-by: Dmitry Kozlyuk Blank line is required between "Fixes" block and " Reported/SoB". [...] > --- a/lib/librte_eal/include/generic/rte_byteorder.h > +++ b/lib/librte_eal/include/generic/rte_byteorder.h > -#ifdef RTE_EXEC_ENV_FREEBSD > +#if defined(RTE_EXEC_ENV_FREEBSD) No need to change above line. > #include > -#else > +#elif defined(RTE_EXEC_ENV_LINUX) Parenthesis are useless. > #include > #endif