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 A55AB41EB1 for ; Thu, 16 Mar 2023 16:52:31 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 84C3940DF6; Thu, 16 Mar 2023 16:52:31 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mails.dpdk.org (Postfix) with ESMTP id 7F0F840DDC for ; Thu, 16 Mar 2023 16:52:30 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1678981950; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=q8uke88wcMb3BMx2ShiPM8wlwzM4/a5O1Ph2f/iAcYo=; b=VT/WrqjXhXqZRy0RVyWjAZfyNXpsRRzfAwhpEYQuGPKPpNxL4QxwNTHhAg+BL/2N13Iqxj x+iLf8R2FaU64MGiq3KIz1t3Pi7G4pD3Dp3ypT/F9sF1MWiRCpfu50NWiaIYZ6iL6P7q35 lsVgxMGOC6H2T1KFDQwDqWfMl+g2Z9s= Received: from mail-pf1-f200.google.com (mail-pf1-f200.google.com [209.85.210.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-526-pIyDqmjbN_m4-rSiO4yVXA-1; Thu, 16 Mar 2023 11:52:28 -0400 X-MC-Unique: pIyDqmjbN_m4-rSiO4yVXA-1 Received: by mail-pf1-f200.google.com with SMTP id b8-20020aa78708000000b005eaa50faa35so1339491pfo.20 for ; Thu, 16 Mar 2023 08:52:28 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678981947; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=q8uke88wcMb3BMx2ShiPM8wlwzM4/a5O1Ph2f/iAcYo=; b=e1+5OI65nVJ47tSAwGyOQJo9Qe+gmbndi32cOaRSG57DMC0II4gFooAVo6eZZZ9uhP Lossfa0qb8twHtkHfFbGZIqfaup/vG6JCP67GnAs6x5Uhd29IAxEKnG8OCrh+HB99Sov zqifW2zTZaiZfXY64mB+DlD8EW7P4nyUKHuEhTu/Tnp1UcYbAcG19yfNnMo1shgrBPk0 whFWGzgQ0GMz9AheI97aURXf1nWloKyl8jqTHbsCSukXBdMHUnevYtVbjQyBHRNjAsxS BS3832bluivG6UbfN1q6y0WHzdAbTnRiGI4X5J9Bw769GcE2IJNNVWb/HE6Ck1tD0pFL bYyg== X-Gm-Message-State: AO0yUKVACzTYxmo4pJYc0c8W+CApd4ag54apTez0rAVGKMQLRAdMqchh AUyRgqxo/zycJ0VmwtJs1DqGVyoxwJKbCniNKZ0NehOA2ZkorZvCoUdyqY2uV7Dymt5m3FV+Fyg WpTCdvcsgGPwysMEkAxb3jeg= X-Received: by 2002:a05:6a00:1506:b0:625:ce0d:e69b with SMTP id q6-20020a056a00150600b00625ce0de69bmr1782215pfu.5.1678981947744; Thu, 16 Mar 2023 08:52:27 -0700 (PDT) X-Google-Smtp-Source: AK7set//vHdsNlqDGBD/1J8yn3lcCQQkcDKSRi92+F0C9fDxgoWgug9panEMsjM+EmfeMrmMEo+sWWMEMwU8+PDIH/o= X-Received: by 2002:a05:6a00:1506:b0:625:ce0d:e69b with SMTP id q6-20020a056a00150600b00625ce0de69bmr1782193pfu.5.1678981947287; Thu, 16 Mar 2023 08:52:27 -0700 (PDT) MIME-Version: 1.0 References: <20221220123754.239802-1-ktraynor@redhat.com> In-Reply-To: From: David Marchand Date: Thu, 16 Mar 2023 16:52:16 +0100 Message-ID: Subject: Re: [PATCH 21.11] ring: squash gcc 12.2.1 warnings To: dev , Honnappa Nagarahalli , Konstantin Ananyev , Thomas Monjalon Cc: stable@dpdk.org, alialnu@nvidia.com, Bruce Richardson , Luca Boccassi , "Xueming(Steven) Li" , Patrick Robb , Kevin Traynor X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org On Thu, Mar 16, 2023 at 12:00=E2=80=AFPM David Marchand wrote: > > On Tue, Dec 20, 2022 at 1:38=E2=80=AFPM Kevin Traynor wrote: > > > > gcc 12.2.1 on Fedora 37 is giving stringop-overread and > > stringop-overflow warnings when compiled with --buildtype=3Ddebug > > e.g. [1]. > > > > These are not yet fixed on main branch. They look similar > > to the overflow issues previously squashed in rte_memcpy with > > commit b5b3ea803e47 ("eal/x86: ignore gcc 10 stringop-overflow warnings= ") > > > > In order to ensure DPDK 21.11.3 compiles on Fedora 37, squash these > > warnings. If a subsequent cleaner fix becomes available on from main > > branch it can be backported to later DPDK 21.11 LTS release. > > > > [1] > > lib/ring/rte_ring_elem_pvt.h:100:25: error: > > =E2=80=98memcpy=E2=80=99 reading 32 bytes from a region of size 4 > > [-Werror=3Dstringop-overread] > > 100 | memcpy((void *)(ring + idx), > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > 101 | (const void *)(obj + i), 32); > > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > > lib/ring/rte_ring_elem_pvt.h:234:25: error: > > =E2=80=98memcpy=E2=80=99 writing 32 bytes into a region of size 4 overf= lows the destination > > [-Werror=3Dstringop-overflow=3D] > > 234 | memcpy((void *)(obj + i), (void *)(ring + idx), 32)= ; > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > > Signed-off-by: Kevin Traynor I see Konstantin is ok with it and I think Honnappa is off for some days. So I adjusted the commitlog for the main branch, and applied. For the record, there is a bugzilla opened for this issue: https://bugs.dpdk.org/show_bug.cgi?id=3D1062 --=20 David Marchand