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 34366A0532; Wed, 22 Jan 2020 17:07:49 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 032302BF1; Wed, 22 Jan 2020 17:07:49 +0100 (CET) Received: from mail-wr1-f66.google.com (mail-wr1-f66.google.com [209.85.221.66]) by dpdk.org (Postfix) with ESMTP id 827672A62; Wed, 22 Jan 2020 17:07:47 +0100 (CET) Received: by mail-wr1-f66.google.com with SMTP id q6so7862476wro.9; Wed, 22 Jan 2020 08:07:47 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:content-transfer-encoding:user-agent:mime-version; bh=Ycqw1Crzqa8XjOcUNyDW85Xzhi2M6peuiPO5vkhQpZQ=; b=IsdJFQclFrYhf2SkorfgcmKWH22/WtAc2o+K2Tgbq5Y9yfx1nf/KGUmlW5ly3IMN3P jaX+fnRyzaa/QQEZOe9DDqHU9hFTzcvbL6NLQPgAHPPQkTpkqx/CVpNEka5ld4KZ/D2C rvK3I/HRhoTyDkrk/D1HpMMKbO2/sjGApHuu7Wn0d0UrwYQN3wMLW68HVLpqvQLTd0bi zUhg0SPCy1JTTVDhBmziJXEVwH8zuNOwhSzMRStMz5Cfwbv64QSUYV7dhZemGAKypCFP oCS04QPyx3LwJPWJVF0SXWO739krG1LEEMnJF+bLiTTpulKDRHfRzA1dxWdC/VAd+KTy yjLQ== X-Gm-Message-State: APjAAAXUf5tsNAbDvZeETHDHzFZYl0X1eB/methc4vxFHxzrb2UVeWm3 E4Ax0vWymiyp9anlTaJ9crA= X-Google-Smtp-Source: APXvYqwPKxa8Ahk/3DhJWII8CeKit8ObttMTtxw6V1TYxI1jhWbKTrtt6SZ+yeFa0JUaFCnULjiboA== X-Received: by 2002:adf:eb89:: with SMTP id t9mr12156635wrn.5.1579709267110; Wed, 22 Jan 2020 08:07:47 -0800 (PST) Received: from localhost ([88.98.246.218]) by smtp.gmail.com with ESMTPSA id s15sm54147219wrp.4.2020.01.22.08.07.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 22 Jan 2020 08:07:46 -0800 (PST) Message-ID: <7b19038b20bb3c35ec6e49a5ef877d521fbad5c4.camel@debian.org> From: Luca Boccassi To: Kevin Traynor , "Yu, PingX" , dpdk stable Cc: "dev@dpdk.org" , "Walker, Benjamin" , "Xu, Qian Q" , Raslan Darawsheh , Thomas Monjalon , "Peng, Yuan" , "Chen, Zhaoyan" , Damian Nowak , Arek Kusztal , Fiona Trahe , "john.griffin@intel.com" , deepak.k.jain@intel.com Date: Wed, 22 Jan 2020 16:07:45 +0000 In-Reply-To: References: <630ab48b958f4bea1c31e83b8a200f236cd6fcff.camel@debian.org> <016faae92f76474aacfd55d4d927f649@intel.com> <4f77afa24b0979b75bea7671ffdda50173cf83b6.camel@debian.org> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.30.5-1.1 MIME-Version: 1.0 Subject: Re: [dpdk-dev] [dpdk-stable] 17.11.10 (LTS) patches review and test 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" On Wed, 2020-01-22 at 15:17 +0100, Kevin Traynor wrote: > On 22/01/2020 12:37, Luca Boccassi wrote: > > On Wed, 2020-01-22 at 05:27 +0000, Yu, PingX wrote: > > > Luca, > > > Update the regression test result of Intel part. See the details > > > as > > > below. > > >=20 > > > * Intel(R) Testing > > > # Basic Intel(R) NIC testing > > > * PF(i40e): one compile issue is found and we fix it with > > > attached > > > patch to go ahead the test. All passed.=20 > > > 1. Bugs:make -j 110 -C test Compile failed > > > * PF(ixgbe): the same as PF(i40e). > > > * VF: Pass > > > * Build or compile: Pass > > > * Intel NIC single core/NIC performance: Pass=20 > > > =20 > > > #Basic cryptodev and virtio testing > > > * vhost/virtio basic loopback, PVP and performance test: Pass. > > > * cryptodev: 1 bug is found. > > > 1. [dpdk-stable-17.11.10]Crypto:cryptodev_qat_autotest test > > > failed. > > > PS: issue passed on 17.11.9. > > > =20 > > > Regards, > > > Yu Ping > >=20 > > Hello Yu, > >=20 > > Thanks for the report. For the first issue, you mention the patch > > is > > attached, but there were no attachments in your mail? > >=20 >=20 > I got the attachment, pasting below as it's straightforward. Didn't > check but I wonder is that the right Fixes commit, or another commit > removed path[] being used. >=20 > From bfb6609bc7c8d9d19f74eff6798c9a6931fe0d75 Mon Sep 17 00:00:00 > 2001 > From: Jiang Mao < > maox.jiang@intel.com > > > Date: Fri, 17 Jan 2020 12:53:11 -0500 > Subject: [PATCH] test/test: fix test app defining unused variable >=20 > Delete the unused variable to solve compiling error. >=20 > Signed-off-by: Jiang Mao < > maox.jiang@intel.com > > >=20 > Fixes: af75078f ("first public release") > --- > Note: This patch just fixes v17.11.10-rc1, the other version have no > compiling error. > test/test/process.h | 1 - > 1 file changed, 1 deletion(-) >=20 > diff --git a/test/test/process.h b/test/test/process.h > index 505d2f9..1c369e8 100644 > --- a/test/test/process.h > +++ b/test/test/process.h > @@ -56,7 +56,6 @@ > int num; > char *argv_cpy[numargs + 1]; > int i, status; > - char path[32]; >=20 > pid_t pid =3D fork(); > if (pid < 0) >=20 Thanks - the culprit is this botched backport which was adjusted but not enough and the unused variable was left behind: Fixes: 0ecbf7984bb0 ("test: optimise fd closing in forks") Applied with adjusted fixes line --=20 Kind regards, Luca Boccassi