From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0DA19C433FE for ; Thu, 6 Oct 2022 10:27:09 +0000 (UTC) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 303A942BF0; Thu, 6 Oct 2022 12:27:08 +0200 (CEST) Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) by mails.dpdk.org (Postfix) with ESMTP id E8E2F42B70; Thu, 6 Oct 2022 12:27:05 +0200 (CEST) Received: from compute3.internal (compute3.nyi.internal [10.202.2.43]) by mailout.west.internal (Postfix) with ESMTP id A5AD23200936; Thu, 6 Oct 2022 06:27:04 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute3.internal (MEProxy); Thu, 06 Oct 2022 06:27:05 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:cc:content-transfer-encoding:content-type:date:date:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to; s=fm2; t=1665052024; x= 1665138424; bh=akMys4QdtGcafXuj/LRu4bl7E79ksiYnahfi3sZlwp0=; b=J WFNlkla+fem1RxfwgYdmGkXlACJoAszs4eClir+4tGdNbtk686JITnTDMw/9zOaf K3SjfJvsH1Fxgnwu6W0fEzclIjCyPXmkqzBWyWSLA0XWhSze7OpYY0Dxlo7FIcvK pEVsDGMPvlulO18e4GV37FIAtiinusn+8T6BXkipJoDEhV46LBDJtenQsFN0bO5S NAXr7Wnr3aqFtG7IvITshq1o40YPEnd1+ua/56dVASy7P8Qlk0by8GLH1p3a2DPx Z5vi6dMERARtKYPzYs1/wjzRKw+OmJz7Ay1V0W4CY8mc3W0hancyhJ/TMtsAUbfE XlZ0bucfx8SSzQNsPTjdA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:date:date:feedback-id:feedback-id:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm2; t=1665052024; x= 1665138424; bh=akMys4QdtGcafXuj/LRu4bl7E79ksiYnahfi3sZlwp0=; b=i M+Ibzwc8VsskUFYSM0ArwM87klOppgJYeyRgS/a2MH6osKmf8sLO2g0L8Pp9vwu8 b2eqL7YDr42j9hvKK8LKoJbIkFjI9mX0LJPRuNuLMK5ZGaAAfgP37JBgQsr/C3K+ zvKjXNDNA3umCRKjeC4jcWIIkEPFKotNtEPfL7TsQtHeXJQ16Doel8woROnP1zKl 93LoH7p65tO2ovREno/cUEXNgFNybSViJCh8DbkdxyyJQgAHIw10nIRory5D2hfV pL5ljx77SvauJAxkXhS8w/uenGQ1/grhyPCZfkbjiAr428lMciIjwLIUPzeghTqf 02wjw1NHJOEJHIl/vYsLQ== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvfedrfeeihedgvdekucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvvefufffkjghfggfgtgesthhqredttddtjeenucfhrhhomhepvfhhohhm rghsucfoohhnjhgrlhhonhcuoehthhhomhgrshesmhhonhhjrghlohhnrdhnvghtqeenuc ggtffrrghtthgvrhhnpeegtddtleejjeegffekkeektdejvedtheevtdekiedvueeuvdei uddvleevjeeujeenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfh hrohhmpehthhhomhgrshesmhhonhhjrghlohhnrdhnvght X-ME-Proxy: Feedback-ID: i47234305:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Thu, 6 Oct 2022 06:27:03 -0400 (EDT) From: Thomas Monjalon To: Rasesh Mody , Devendra Singh Rawat , amitprakashs@marvell.com Cc: dev@dpdk.org, stable@dpdk.org Subject: Re: [PATCH] net/qede/base: fix 32-bit build with GCC 12 Date: Thu, 06 Oct 2022 12:27:01 +0200 Message-ID: <1803473.atdPhlSkOF@thomas> In-Reply-To: <20221004111811.1502920-1-thomas@monjalon.net> References: <20221004111811.1502920-1-thomas@monjalon.net> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" 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 04/10/2022 13:18, Thomas Monjalon: > A pointer is passed to a macro and it seems mistakenly referenced. > This issue is seen only when compiling with GCC 12 for 32-bit: >=20 > drivers/net/qede/base/ecore_init_fw_funcs.c:1418:25: > error: array subscript 1 is outside array bounds of =E2=80=98u32[= 1]=E2=80=99 > {aka =E2=80=98unsigned int[1]=E2=80=99} [-Werror=3Darray-bounds] > 1418 | ecore_wr(dev, ptt, ((addr) + (4 * i)), \ > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 1419 | ((u32 *)&(arr))[i]); \ > | ~~~~~~~~~~~~~~~~~~~ > drivers/net/qede/base/ecore_init_fw_funcs.c:1465:17: > note: in expansion of macro =E2=80=98ARR_REG_WR=E2=80=99 > 1465 | ARR_REG_WR(p_hwfn, p_ptt, addr, pData, len_in_dwords); > | ^~~~~~~~~~ > drivers/net/qede/base/ecore_init_fw_funcs.c:1439:35: > note: at offset 4 into object =E2=80=98pData=E2=80=99 of size 4 > 1439 | u32 *pData, > | ~~~~~^~~~~ >=20 > Fixes: 3b307c55f2ac ("net/qede/base: update FW to 8.40.25.0") > Cc: stable@dpdk.org >=20 > Signed-off-by: Thomas Monjalon A similar patch was sent 6 weeks ago by Amit Prakash Shukla.