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 653A9A0527; Mon, 9 Nov 2020 18:03:42 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 3AC4B697B; Mon, 9 Nov 2020 18:03:41 +0100 (CET) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 0C0D26947 for ; Mon, 9 Nov 2020 18:03:37 +0100 (CET) IronPort-SDR: AANgIMFDFg3aO5UTt27fuYgVSS06sqAaWUnAzPGPbKefPG/rRgDiyDlnk/BouOh8K7LBJV+YOq nXTFD4yFnt2A== X-IronPort-AV: E=McAfee;i="6000,8403,9800"; a="254541087" X-IronPort-AV: E=Sophos;i="5.77,463,1596524400"; d="scan'208";a="254541087" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Nov 2020 09:03:34 -0800 IronPort-SDR: tzYuM3OGbsPz1O3nMejGpcye9qhEQ77adumQip02gE1MZ8i2M+1mouLWWMYpFy6g4U2OwC7RTo 5y20wwmwWtRw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,463,1596524400"; d="scan'208";a="473048005" Received: from orsmsx606.amr.corp.intel.com ([10.22.229.19]) by orsmga004.jf.intel.com with ESMTP; 09 Nov 2020 09:03:23 -0800 Received: from orsmsx601.amr.corp.intel.com (10.22.229.14) by ORSMSX606.amr.corp.intel.com (10.22.229.19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Mon, 9 Nov 2020 09:03:23 -0800 Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) by orsmsx601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5 via Frontend Transport; Mon, 9 Nov 2020 09:03:23 -0800 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (104.47.56.174) by edgegateway.intel.com (134.134.137.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.1713.5; Mon, 9 Nov 2020 09:03:20 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=iACGKtlob5t73/x4CXcwOqyAC3cGPkqFrjWO0k1w2AXq6PBOYX1Jrie2MUoLkRRhnu8yObDPmiuO4XfkaU9XkWfWEahkLf7ofzJWUj+rf+z0iqfcCD5CAYPLPWfPH3zQprCNjFj78c/8nGAZMslMBYhPbXTSdxJwHh9PGW0mV2+QQJZHUxPczvned2VTym22185Gm6fJ++xTEUAP0PhuXJnO/I9Ji8bdzi2uKj1NeG2nC80q4bZBMvzEVUpVXjSr9jqqAtOzj1RMxF0EkYMX6+/VkQfJ+oJCuiZbw+F5OKDCvWixgltZESAD0M1HvgNyQLzEEjvQQ+xu9woFY0GAVA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=p2v3y07UJv724mLfC4MpVdn2AOKcpB6auSVlnlASOzQ=; b=a96IAziySr/B5EBQGW6wgo4s/4CyFDqhb4At9l9PpUL8hE9YU5q47nBkyMTjIMp7kmVMp98ftFYxzorAqlbo4755JHQBZkF6hBJjFfoKzdVL/wqoh/17l3ITrH0hnctRm8QfISXfcAswcCmVEW+jYk0WYOisXKJPuGYbhC7z4RpAb94Pjvzuc5hQNhl/wFjY3sVjvsA4ByOJiQlpjpWgOJNkKMhyx8PEIJj1mioAM+0pK6gdp1UoJIUT7OOM6PpVTNZPAPSVi0lhGgpwdngmLzjSY6s4txf+/ehx7fnKf0mS6CrokB20dfmFI6+hyisphtIT0EN8xH3rBpgis7qoAw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=p2v3y07UJv724mLfC4MpVdn2AOKcpB6auSVlnlASOzQ=; b=xYEzrawMjJH+6LLLs4j5l++G8hvI6yg4DjlDlrySCe86s1V8amNVCLvihROJhb+Lk9zeM0pK8LR/zl0r9b88YfFeytPBriurcU2Teyui4WHubLmXnyjRIAPHNzeJtfJRD/HKr3zUGGghW9LABW7aDY7xbBSNYKD9pCpwZ/KzZd8= Received: from DM5PR1101MB2121.namprd11.prod.outlook.com (2603:10b6:4:50::19) by DM5PR1101MB2124.namprd11.prod.outlook.com (2603:10b6:4:55::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3541.21; Mon, 9 Nov 2020 17:03:15 +0000 Received: from DM5PR1101MB2121.namprd11.prod.outlook.com ([fe80::283c:db65:84a4:b621]) by DM5PR1101MB2121.namprd11.prod.outlook.com ([fe80::283c:db65:84a4:b621%10]) with mapi id 15.20.3541.025; Mon, 9 Nov 2020 17:03:15 +0000 From: "Walsh, Conor" To: Honnappa Nagarahalli , "Ananyev, Konstantin" CC: "dev@dpdk.org" , "Chen, LingliX" , nd , nd Thread-Topic: [PATCH v1] app/test: zcd needs to be initialised to fix gcc compile error Thread-Index: AQHWssx7UeSjSO8VKEW0ixDwHHuez6m4UHwAgAevGSCAAAodgIAABIJQ Date: Mon, 9 Nov 2020 17:03:15 +0000 Message-ID: References: <20201104170306.1688771-1-conor.walsh@intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-reaction: no-action dlp-version: 11.5.1.3 dlp-product: dlpe-windows authentication-results: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=intel.com; x-originating-ip: [51.37.110.131] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 0ffe00e6-0381-4883-415e-08d884d15981 x-ms-traffictypediagnostic: DM5PR1101MB2124: x-ld-processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:9508; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: VwGWvU5+OAPnf6Wb6dilqA3qh6nNELkU/tFRHywRscIOMJZ6KKFCL/hyg5fTZmE6AgkURbMkxMr46XWXMkeKJ+tSkSEpwefraFDK7VpQQGhm4KDbdhAis15QZ2tJxXppQx6dFxE8GF7mvmTRD9O0Fxv9WdqvqFuQQqiPHq6xnJ1OwZKenpZauiiI3JTAi/GppEYteXAQuqZporoxFtMpcOFn1tUpS4LBIH4mVEIP0rrnd8njU/ZboO+PpmgqaZEyktSqY4nxskOAPxdLnUNa0dmObRGriuRw4Rt3NT+BoQaoIIQWYuOcaWK5JnQGaSU8hYWQKn8cxXFsflS13Iw5Cg== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DM5PR1101MB2121.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(136003)(396003)(366004)(346002)(376002)(39860400002)(71200400001)(4326008)(54906003)(7696005)(6636002)(53546011)(6506007)(26005)(9686003)(186003)(110136005)(5660300002)(8936002)(55016002)(52536014)(316002)(86362001)(478600001)(30864003)(66446008)(83380400001)(66476007)(64756008)(33656002)(76116006)(66946007)(66556008)(8676002)(2906002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata: ZjskUE9x7R/rzlAmD5M+PjyE/yM5qbGIZrw9Pc8HAtHZQpbFAphDD8mQltZfmw+dq5iSMPLABC7COWAW7hiudgpHZ7VQRZmcJaeZC6ZGsv3RsNB9Y6FNXMDnuMcVABV9+SRrFJ14UnCeY/w1rz48Qz+iEbuFte3ErH5yheKeXEbRKiPnjqFd0MFYKuHk17oz8hC9EVVssin8r4nF9aorUwvefZO0cYB5aUL3z9LV5XGjlhIuOZLxQpoum4xVLW45q599YfUv6rrJkgdbtMtj1zZMF9ZF6hBUpbit5el1C0AbrI410ffzRr8pEWbmrFq9fMsvLPExtSPCwhNgFw1MkLwVn/IpH4Uci0vuaFygkYRmGeoHK9JvFQ778rfDP0IpQOv4yKnDoi+RkQM5j0xLnTxfAx6jKB3Rn/80eA6znaJvJQCt6ee7d0jCtnQb3b427dEW4sQY4Wa2A3wbAvR1mZ/tY+0IMraqybu9jVk+IHT3UNohxZx6Yi7SZr8pTSi2qnKlNhxJZ1kxXDghBe2oN2akCcx7GdZkN0ShRo/C21+g3usNG3ZSJi/VR+pUkKeEN7pgMZjF2sErwiabhi4B+sqd0fgDTbQy8N7hrL2f7DYTx2j0mmO4Nz0x29TgBLCb2mllmA7bV6+xt61q2KTd8nDhkphLzsGCKr7iQVM4neHB7V6LAtiBf6BQXUiE2UITBOaUmTsuZuNJcfd8Y99qkzF87+1oyA99Qw02EClURpkcehEjksb5xwGeinVsebPYWJ6uUvuz4AvgUE+NeI6mSkFP3utG+lNDIFCX0Hwp1hhmZnDFYcgfGHsOKnKdFd0wl6sNvh+DEhc451SRj9npYY0EtSqqwCagD7UMu1ZzGlLscvO8NEoaGmlqw3IHRXe4WW1UV8gckmno6I62/oIpAA== Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM5PR1101MB2121.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0ffe00e6-0381-4883-415e-08d884d15981 X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Nov 2020 17:03:15.4490 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: dZYBOUsFyD8X+89l8vIPwsk9UeXz6M7XtGGhz5zCHzrEXnlIK0U2zFBMmT7CXrp9ND4G3j1VhUNz2jq61fp1nQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR1101MB2124 X-OriginatorOrg: intel.com Subject: Re: [dpdk-dev] [PATCH v1] app/test: zcd needs to be initialised to fix gcc compile error 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" Hi Honnappa, > From: Honnappa Nagarahalli > Sent: Monday 9 November 2020 16:45 > To: Walsh, Conor ; Ananyev, Konstantin > > Cc: dev@dpdk.org; Chen, LingliX ; nd > ; Honnappa Nagarahalli ; > nd > Subject: RE: [PATCH v1] app/test: zcd needs to be initialised to fix gcc = compile > error >=20 > >=20 > > > > Hi Honnappa, > > > > Sorry about the delayed reply. > No problem, thank you. >=20 > > The build error trace is below, the build command used was: CC=3Dgcc me= son > -- > > optimization=3D1 --werror build1; ninja -C build1 I am not sure how I c= ould fix > > this issue without initialising zcd. > > > > [2339/2407] Compiling C object app/test/dpdk- > > test.p/test_ring_mt_peek_stress_zc.c.o > > FAILED: app/test/dpdk-test.p/test_ring_mt_peek_stress_zc.c.o > > gcc -Iapp/test/dpdk-test.p -Iapp/test -I../app/test -Ilib/librte_acl - > > I../lib/librte_acl -I. -I.. -Iconfig -I../config -Ilib/librte_eal/inclu= de - > > I../lib/librte_eal/include -Ilib/librte_eal/linux/include - > > I../lib/librte_eal/linux/include -Ilib/librte_eal/x86/include - > > I../lib/librte_eal/x86/include -Ilib/librte_eal/common - > > I../lib/librte_eal/common -Ilib/librte_eal -I../lib/librte_eal - > Ilib/librte_kvargs - > > I../lib/librte_kvargs -Ilib/librte_metrics -I../lib/librte_metrics - > > Ilib/librte_telemetry -I../lib/librte_telemetry -Idrivers/bus/pci - > > I../drivers/bus/pci -I../drivers/bus/pci/linux -Ilib/librte_pci -I../li= b/librte_pci > - > > Idrivers/bus/vdev -I../drivers/bus/vdev -Ilib/librte_bitratestats - > > I../lib/librte_bitratestats -Ilib/librte_ethdev -I../lib/librte_ethdev = - > > Ilib/librte_net -I../lib/librte_net -Ilib/librte_mbuf -I../lib/librte_m= buf - > > Ilib/librte_mempool -I../lib/librte_mempool -Ilib/librte_ring - > I../lib/librte_ring > > -Ilib/librte_meter -I../lib/librte_meter -Ilib/librte_bpf -I../lib/libr= te_bpf - > > Ilib/librte_cfgfile -I../lib/librte_cfgfile -Ilib/librte_cmdline - > I../lib/librte_cmdline > > -Ilib/librte_cryptodev -I../lib/librte_cryptodev -Ilib/librte_distribut= or - > > I../lib/librte_distributor -Ilib/librte_efd -I../lib/librte_efd -Ilib/l= ibrte_hash - > > I../lib/librte_hash -Ilib/librte_rcu -I../lib/librte_rcu -Ilib/librte_e= ventdev - > > I../lib/librte_eventdev -Ilib/librte_timer -I../lib/librte_timer -Ilib/= librte_fib - > > I../lib/librte_fib -Ilib/librte_rib -I../lib/librte_rib -Ilib/librte_fl= ow_classify - > > I../lib/librte_flow_classify -Ilib/librte_table -I../lib/librte_table - > Ilib/librte_port > > -I../lib/librte_port -Ilib/librte_sched -I../lib/librte_sched -Ilib/lib= rte_ip_frag > - > > I../lib/librte_ip_frag -Ilib/librte_kni -I../lib/librte_kni -Ilib/librt= e_lpm - > > I../lib/librte_lpm -Ilib/librte_graph -I../lib/librte_graph -Ilib/librt= e_ipsec - > > I../lib/librte_ipsec -Ilib/librte_security -I../lib/librte_security - > > Ilib/librte_latencystats -I../lib/librte_latencystats -Ilib/librte_memb= er - > > I../lib/librte_member -Ilib/librte_node -I../lib/librte_node - > Ilib/librte_pipeline > > -I../lib/librte_pipeline -Ilib/librte_rawdev -I../lib/librte_rawdev - > > Ilib/librte_reorder -I../lib/librte_reorder -Ilib/librte_stack - > I../lib/librte_stack - > > Idrivers/mempool/ring -I../drivers/mempool/ring -Idrivers/mempool/stack > - > > I../drivers/mempool/stack -Idrivers/event/skeleton - > > I../drivers/event/skeleton -Idrivers/net/bonding -I../drivers/net/bondi= ng - > > Idrivers/net/ring -I../drivers/net/ring -Ilib/librte_power - > I../lib/librte_power - > > Ilib/librte_pdump -I../lib/librte_pdump -Idrivers/crypto/scheduler - > > I../drivers/crypto/scheduler -fdiagnostics-color=3Dalways -pipe - > > D_FILE_OFFSET_BITS=3D64 -Wall -Winvalid-pch -Werror -O1 -include > > rte_config.h -Wextra -Wcast-qual -Wdeprecated -Wformat-nonliteral - > > Wformat-security -Wmissing-declarations -Wmissing-prototypes - > Wnested- > > externs -Wold-style-definition -Wpointer-arith -Wsign-compare -Wstrict- > > prototypes -Wundef -Wwrite-strings -Wno-packed-not-aligned -Wno- > missing- > > field-initializers -march=3Dnative -mno-avx512f - > DALLOW_EXPERIMENTAL_API - > > Wno-format-truncation -D_GNU_SOURCE -fno-strict-aliasing - > > DALLOW_INTERNAL_API -MD -MQ app/test/dpdk- > > test.p/test_ring_mt_peek_stress_zc.c.o -MF app/test/dpdk- > > test.p/test_ring_mt_peek_stress_zc.c.o.d -o app/test/dpdk- > > test.p/test_ring_mt_peek_stress_zc.c.o -c > > ../app/test/test_ring_mt_peek_stress_zc.c > > In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: > > ../app/test/test_ring_stress_impl.h: In function 'test_worker': > > ../app/test/test_ring.h:82:3: error: 'zcd.n1' may be used uninitialized= in this > > function [-Werror=3Dmaybe-uninitialized] > > test_ring_mem_copy(zcd->ptr2, src, > > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > esize, num - zcd->n1); > > ~~~~~~~~~~~~~~~~~~~~~ > > ../app/test/test_ring_mt_peek_stress_zc.c:32:26: note: 'zcd.n1' was > declared > > here > > struct rte_ring_zc_data zcd; > > ^~~ > > In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: > > ../app/test/test_ring.h:67:2: error: '*((void *)&zcd+8)' may be used > > uninitialized in this function [-Werror=3Dmaybe-uninitialized] > > memcpy(dst, src, sz); > > ^~~~~~~~~~~~~~~~~~~~ > > ../app/test/test_ring_mt_peek_stress_zc.c:32:26: note: '*((void > *)&zcd+8)' > > was declared here > I remember seeing a similar error in test_ring_st_peek_stress_zc.c. > In the file test_ring_mt_peek_stress_zc.c, after the call to > enqueue/dequeue API calls, we have: > "n =3D (m =3D=3D n) ? n : 0;" >=20 > This is present as the code was copied from existing code. I do not see w= hy > this line is required. Can you please try to remove this line and see if = it > works? I commented out all occurrences of that line in test_ring_mt_peek_stess_zc.= c and the build is still failing. Trace at end of email. >=20 > > struct rte_ring_zc_data zcd; > > ^~~ > > In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: > > ../app/test/test_ring.h:67:2: error: 'zcd' may be used uninitialized in= this > > function [-Werror=3Dmaybe-uninitialized] > > memcpy(dst, src, sz); > > ^~~~~~~~~~~~~~~~~~~~ > > ../app/test/test_ring_mt_peek_stress_zc.c:32:26: note: 'zcd' was declar= ed > > here > > struct rte_ring_zc_data zcd; > > ^~~ > > In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: > > ../app/test/test_ring.h:63:11: error: 'zcd.n1' may be used uninitialize= d in > this > > function [-Werror=3Dmaybe-uninitialized] > > sz =3D num * sizeof(void *); > > ~~~~^~~~~~~~~~~~~~~~ > > ../app/test/test_ring_mt_peek_stress_zc.c:14:26: note: 'zcd.n1' was > declared > > here > > struct rte_ring_zc_data zcd; > > ^~~ > > In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: > > ../app/test/test_ring.h:67:2: error: '*((void *)&zcd+8)' may be used > > uninitialized in this function [-Werror=3Dmaybe-uninitialized] > > memcpy(dst, src, sz); > > ^~~~~~~~~~~~~~~~~~~~ > > ../app/test/test_ring_mt_peek_stress_zc.c:14:26: note: '*((void > *)&zcd+8)' > > was declared here > > struct rte_ring_zc_data zcd; > > ^~~ > > In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: > > ../app/test/test_ring.h:67:2: error: 'zcd' may be used uninitialized in= this > > function [-Werror=3Dmaybe-uninitialized] > > memcpy(dst, src, sz); > > ^~~~~~~~~~~~~~~~~~~~ > > ../app/test/test_ring_mt_peek_stress_zc.c:14:26: note: 'zcd' was declar= ed > > here > > struct rte_ring_zc_data zcd; > > ^~~ > > cc1: all warnings being treated as errors [2340/2407] Compiling C objec= t > > drivers/libtmp_...ontx2.a.p/event_octeontx2_otx2_worker_dual.c.o > > ninja: build stopped: subcommand failed. > > > > Thanks, > > Conor. > > > > > From: Honnappa Nagarahalli > > > Sent: Wednesday 4 November 2020 18:48 > > > To: Walsh, Conor ; Ananyev, Konstantin > > > > > > Cc: dev@dpdk.org; Chen, LingliX ; nd > > > ; Honnappa Nagarahalli > ; > > nd > > > > > > Subject: RE: [PATCH v1] app/test: zcd needs to be initialised to fix > > > gcc compile error > > > > > > Hi Conor, > > > Thanks for the patch. > > > > > > > > > > > > > > > > > When DPDK is compiled with gcc < 9 with the optimization level set > > > > to 1 gcc sees zcd in test_ring.h as possibly being uninitialised. T= o > > > > correct this error > > > zcd > > > > has been initialised to {0} in test_ring_mt_peek_stress_zc.c. > > > > > > > > Signed-off-by: Conor Walsh > > > > --- > > > > app/test/test_ring_mt_peek_stress_zc.c | 4 ++-- > > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/app/test/test_ring_mt_peek_stress_zc.c > > > > b/app/test/test_ring_mt_peek_stress_zc.c > > > > index 7e0bd511a7..ba0acf237c 100644 > > > > --- a/app/test/test_ring_mt_peek_stress_zc.c > > > > +++ b/app/test/test_ring_mt_peek_stress_zc.c > > > > @@ -11,7 +11,7 @@ _st_ring_dequeue_bulk(struct rte_ring *r, void > > > > **obj, uint32_t n, > > > > uint32_t *avail) > > > > { > > > > uint32_t m; > > > > - struct rte_ring_zc_data zcd; > > > > + struct rte_ring_zc_data zcd =3D {0}; > > > I faced similar problems when I created this test case. I refrained > > > from doing this initialization as it might mask the problems in the > > > library. Is it possible to identify which path in the library is > > > actually resulting in this error? I would prefer to fix that instead. > > > > > > > > > > > m =3D rte_ring_dequeue_zc_bulk_start(r, n, &zcd, avail); > > > > n =3D (m =3D=3D n) ? n : 0; > > > > @@ -29,7 +29,7 @@ _st_ring_enqueue_bulk(struct rte_ring *r, void * > > > const > > > > *obj, uint32_t n, > > > > uint32_t *free) > > > > { > > > > uint32_t m; > > > > - struct rte_ring_zc_data zcd; > > > > + struct rte_ring_zc_data zcd =3D {0}; > > > > > > > > m =3D rte_ring_enqueue_zc_bulk_start(r, n, &zcd, free); > > > > n =3D (m =3D=3D n) ? n : 0; > > > > -- > > > > 2.25.1 Thanks, Conor. [2339/2407] Compiling C object app/test/dpdk-test.p/test_ring_mt_peek_stres= s_zc.c.o FAILED: app/test/dpdk-test.p/test_ring_mt_peek_stress_zc.c.o gcc -Iapp/test/dpdk-test.p -Iapp/test -I../app/test -Ilib/librte_acl -I../l= ib/librte_acl -I. -I.. -Iconfig -I../config -Ilib/librte_eal/include -I../l= ib/librte_eal/include -Ilib/librte_eal/linux/include -I../lib/librte_eal/li= nux/include -Ilib/librte_eal/x86/include -I../lib/librte_eal/x86/include -I= lib/librte_eal/common -I../lib/librte_eal/common -Ilib/librte_eal -I../lib/= librte_eal -Ilib/librte_kvargs -I../lib/librte_kvargs -Ilib/librte_metrics = -I../lib/librte_metrics -Ilib/librte_telemetry -I../lib/librte_telemetry -I= drivers/bus/pci -I../drivers/bus/pci -I../drivers/bus/pci/linux -Ilib/librt= e_pci -I../lib/librte_pci -Idrivers/bus/vdev -I../drivers/bus/vdev -Ilib/li= brte_bitratestats -I../lib/librte_bitratestats -Ilib/librte_ethdev -I../lib= /librte_ethdev -Ilib/librte_net -I../lib/librte_net -Ilib/librte_mbuf -I../= lib/librte_mbuf -Ilib/librte_mempool -I../lib/librte_mempool -Ilib/librte_r= ing -I../lib/librte_ring -Ilib/librte_meter -I../lib/librte_meter -Ilib/lib= rte_bpf -I../lib/librte_bpf -Ilib/librte_cfgfile -I../lib/librte_cfgfile -I= lib/librte_cmdline -I../lib/librte_cmdline -Ilib/librte_cryptodev -I../lib/= librte_cryptodev -Ilib/librte_distributor -I../lib/librte_distributor -Ilib= /librte_efd -I../lib/librte_efd -Ilib/librte_hash -I../lib/librte_hash -Ili= b/librte_rcu -I../lib/librte_rcu -Ilib/librte_eventdev -I../lib/librte_even= tdev -Ilib/librte_timer -I../lib/librte_timer -Ilib/librte_fib -I../lib/lib= rte_fib -Ilib/librte_rib -I../lib/librte_rib -Ilib/librte_flow_classify -I.= ./lib/librte_flow_classify -Ilib/librte_table -I../lib/librte_table -Ilib/l= ibrte_port -I../lib/librte_port -Ilib/librte_sched -I../lib/librte_sched -I= lib/librte_ip_frag -I../lib/librte_ip_frag -Ilib/librte_kni -I../lib/librte= _kni -Ilib/librte_lpm -I../lib/librte_lpm -Ilib/librte_graph -I../lib/librt= e_graph -Ilib/librte_ipsec -I../lib/librte_ipsec -Ilib/librte_security -I..= /lib/librte_security -Ilib/librte_latencystats -I../lib/librte_latencystats= -Ilib/librte_member -I../lib/librte_member -Ilib/librte_node -I../lib/libr= te_node -Ilib/librte_pipeline -I../lib/librte_pipeline -Ilib/librte_rawdev = -I../lib/librte_rawdev -Ilib/librte_reorder -I../lib/librte_reorder -Ilib/l= ibrte_stack -I../lib/librte_stack -Idrivers/mempool/ring -I../drivers/mempo= ol/ring -Idrivers/mempool/stack -I../drivers/mempool/stack -Idrivers/event/= skeleton -I../drivers/event/skeleton -Idrivers/net/bonding -I../drivers/net= /bonding -Idrivers/net/ring -I../drivers/net/ring -Ilib/librte_power -I../l= ib/librte_power -Ilib/librte_pdump -I../lib/librte_pdump -Idrivers/crypto/s= cheduler -I../drivers/crypto/scheduler -fdiagnostics-color=3Dalways -pipe -= D_FILE_OFFSET_BITS=3D64 -Wall -Winvalid-pch -Werror -O1 -include rte_config= .h -Wextra -Wcast-qual -Wdeprecated -Wformat-nonliteral -Wformat-security -= Wmissing-declarations -Wmissing-prototypes -Wnested-externs -Wold-style-def= inition -Wpointer-arith -Wsign-compare -Wstrict-prototypes -Wundef -Wwrite-= strings -Wno-packed-not-aligned -Wno-missing-field-initializers -march=3Dna= tive -mno-avx512f -DALLOW_EXPERIMENTAL_API -Wno-format-truncation -D_GNU_SO= URCE -fno-strict-aliasing -DALLOW_INTERNAL_API -MD -MQ app/test/dpdk-test.p= /test_ring_mt_peek_stress_zc.c.o -MF app/test/dpdk-test.p/test_ring_mt_peek= _stress_zc.c.o.d -o app/test/dpdk-test.p/test_ring_mt_peek_stress_zc.c.o -c= ../app/test/test_ring_mt_peek_stress_zc.c ../app/test/test_ring_mt_peek_stress_zc.c: In function '_st_ring_dequeue_bu= lk': ../app/test/test_ring_mt_peek_stress_zc.c:13:11: error: variable 'm' set bu= t not used [-Werror=3Dunused-but-set-variable] uint32_t m; ^ ../app/test/test_ring_mt_peek_stress_zc.c: In function '_st_ring_enqueue_bu= lk': ../app/test/test_ring_mt_peek_stress_zc.c:31:11: error: variable 'm' set bu= t not used [-Werror=3Dunused-but-set-variable] uint32_t m; ^ In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: ../app/test/test_ring_stress_impl.h: In function 'test_mt1': ../app/test/test_ring.h:82:3: error: 'zcd.n1' may be used uninitialized in = this function [-Werror=3Dmaybe-uninitialized] test_ring_mem_copy(zcd->ptr2, src, ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ esize, num - zcd->n1); ~~~~~~~~~~~~~~~~~~~~~ ../app/test/test_ring_mt_peek_stress_zc.c:32:26: note: 'zcd.n1' was declare= d here struct rte_ring_zc_data zcd; ^~~ In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: ../app/test/test_ring.h:67:2: error: '*((void *)&zcd+8)' may be used uninit= ialized in this function [-Werror=3Dmaybe-uninitialized] memcpy(dst, src, sz); ^~~~~~~~~~~~~~~~~~~~ ../app/test/test_ring_mt_peek_stress_zc.c:32:26: note: '*((void *)&zcd+8)' = was declared here struct rte_ring_zc_data zcd; ^~~ In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: ../app/test/test_ring.h:67:2: error: 'zcd' may be used uninitialized in thi= s function [-Werror=3Dmaybe-uninitialized] memcpy(dst, src, sz); ^~~~~~~~~~~~~~~~~~~~ ../app/test/test_ring_mt_peek_stress_zc.c:32:26: note: 'zcd' was declared h= ere struct rte_ring_zc_data zcd; ^~~ In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: ../app/test/test_ring_stress_impl.h: In function 'test_worker': ../app/test/test_ring.h:63:11: error: 'zcd.n1' may be used uninitialized in= this function [-Werror=3Dmaybe-uninitialized] sz =3D num * sizeof(void *); ~~~~^~~~~~~~~~~~~~~~ ../app/test/test_ring_mt_peek_stress_zc.c:32:26: note: 'zcd.n1' was declare= d here struct rte_ring_zc_data zcd; ^~~ In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: ../app/test/test_ring.h:67:2: error: '*((void *)&zcd+8)' may be used uninit= ialized in this function [-Werror=3Dmaybe-uninitialized] memcpy(dst, src, sz); ^~~~~~~~~~~~~~~~~~~~ ../app/test/test_ring_mt_peek_stress_zc.c:32:26: note: '*((void *)&zcd+8)' = was declared here struct rte_ring_zc_data zcd; ^~~ In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: ../app/test/test_ring.h:67:2: error: 'zcd' may be used uninitialized in thi= s function [-Werror=3Dmaybe-uninitialized] memcpy(dst, src, sz); ^~~~~~~~~~~~~~~~~~~~ ../app/test/test_ring_mt_peek_stress_zc.c:32:26: note: 'zcd' was declared h= ere struct rte_ring_zc_data zcd; ^~~ In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: ../app/test/test_ring.h:63:11: error: 'zcd.n1' may be used uninitialized in= this function [-Werror=3Dmaybe-uninitialized] sz =3D num * sizeof(void *); ~~~~^~~~~~~~~~~~~~~~ ../app/test/test_ring_mt_peek_stress_zc.c:14:26: note: 'zcd.n1' was declare= d here struct rte_ring_zc_data zcd; ^~~ In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: ../app/test/test_ring.h:67:2: error: '*((void *)&zcd+8)' may be used uninit= ialized in this function [-Werror=3Dmaybe-uninitialized] memcpy(dst, src, sz); ^~~~~~~~~~~~~~~~~~~~ ../app/test/test_ring_mt_peek_stress_zc.c:14:26: note: '*((void *)&zcd+8)' = was declared here struct rte_ring_zc_data zcd; ^~~ In file included from ../app/test/test_ring_mt_peek_stress_zc.c:5: ../app/test/test_ring.h:67:2: error: 'zcd' may be used uninitialized in thi= s function [-Werror=3Dmaybe-uninitialized] memcpy(dst, src, sz); ^~~~~~~~~~~~~~~~~~~~ ../app/test/test_ring_mt_peek_stress_zc.c:14:26: note: 'zcd' was declared h= ere struct rte_ring_zc_data zcd; ^~~ cc1: all warnings being treated as errors [2340/2407] Compiling C object drivers/libtmp_rte_event_octeontx2.a.p/event= _octeontx2_otx2_worker_dual.c.o ninja: build stopped: subcommand failed.