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 EE820A0C47 for ; Tue, 14 Sep 2021 12:21:32 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id DE3F0410E0; Tue, 14 Sep 2021 12:21:32 +0200 (CEST) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mails.dpdk.org (Postfix) with ESMTP id 13DE14003C; Tue, 14 Sep 2021 12:21:28 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10106"; a="209044448" X-IronPort-AV: E=Sophos;i="5.85,292,1624345200"; d="scan'208";a="209044448" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 14 Sep 2021 03:21:28 -0700 X-IronPort-AV: E=Sophos;i="5.85,292,1624345200"; d="scan'208";a="470078014" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.29.189]) by fmsmga007-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 14 Sep 2021 03:21:26 -0700 Date: Tue, 14 Sep 2021 11:21:23 +0100 From: Bruce Richardson To: David Marchand Cc: Ferruh Yigit , David Christensen , dev , dpdk stable Message-ID: References: <20210623161620.49916-1-drc@linux.vnet.ibm.com> <20210902235326.3236718-1-drc@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH v2] config/ppc: ignore gcc 11 psabi warnings 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 Sender: "stable" On Tue, Sep 14, 2021 at 12:16:36PM +0200, David Marchand wrote: > On Tue, Sep 14, 2021 at 11:18 AM Ferruh Yigit wrote: > > > > On 9/3/2021 12:53 AM, David Christensen wrote: > > > Suppress the gcc warning "note: the layout of aggregates containing > > > vectors with 4-byte alignment has changed in GCC 5" on POWER systems > > > by setting "-Wno-psabi". Warning was originally added to gcc in > > > commit https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=9832651 to warn > > > of the vector alignment changes introduced in GCC 5. Older gcc > > > versions forced vector alignment to 16 bytes due to requirements for > > > POWER 6 and earlier CPUs, but these restrictions don't apply to CPUs > > > supported by DPDK. > > > > > > Bugzilla ID: 739 > > > > > > Signed-off-by: David Christensen > > > --- > > > v2: > > > - update copyright year > > > - rebase for 21.11-rc0 > > > --- > > > config/ppc/meson.build | 7 +++++++ > > > 1 file changed, 7 insertions(+) > > > > > > diff --git a/config/ppc/meson.build b/config/ppc/meson.build > > > index adf49e1f42..5354db4e0a 100644 > > > --- a/config/ppc/meson.build > > > +++ b/config/ppc/meson.build > > > @@ -1,5 +1,6 @@ > > > # SPDX-License-Identifier: BSD-3-Clause > > > # Copyright(c) 2018 Luca Boccassi > > > +# Copyright(c) 2021 IBM Corporation > > > > > > if not dpdk_conf.get('RTE_ARCH_64') > > > error('Only 64-bit compiles are supported for this platform type') > > > @@ -17,6 +18,12 @@ if not power9_supported > > > dpdk_conf.set('RTE_MACHINE','power8') > > > endif > > > > > > +# Suppress the gcc warning "note: the layout of aggregates containing > > > +# vectors with 4-byte alignment has changed in GCC 5". > > > +if cc.get_id() == 'gcc' and cc.version().version_compare('>=10.0') and cc.version().version_compare('<12.0') and cc.has_argument('-Wno-psabi') > > > + add_project_arguments('-Wno-psabi', language: 'c') > > > +endif > > > + > > > # Certain POWER9 systems can scale as high as 1536 LCORES, but setting such a > > > # high value can waste memory, cause timeouts in time limited autotests, and is > > > # unlikely to be used in many production situations. Similarly, keeping the > > > -- > > > > I am getting following build error in my environment: > > "config/ppc/meson.build:23:6: ERROR: Unknown statement." > > > > The compiler I have is: > > powerpc64le-linux-gcc (gcc 10.2.0 "powerpc64le-linux-gcc.br_real (Buildroot > > 2020.08-14-ge5a2a90) 10.2.0") > > > > meson version: Version: 0.59.1 > > Good catch. > My fault, and I did not see it because I was still testing with gcc 9. > > > > > > Multi-line statements seems need to be merged with '\': > > > > diff --git a/config/ppc/meson.build b/config/ppc/meson.build > > index 0b1948fc7cb9..f95009c77e7a 100644 > > --- a/config/ppc/meson.build > > +++ b/config/ppc/meson.build > > @@ -20,7 +20,7 @@ endif > > > > # Suppress the gcc warning "note: the layout of aggregates containing > > # vectors with 4-byte alignment has changed in GCC 5". > > -if cc.get_id() == 'gcc' and cc.version().version_compare('>=10.0') and > > +if cc.get_id() == 'gcc' and cc.version().version_compare('>=10.0') and \ > > cc.version().version_compare('<12.0') and cc.has_argument('-Wno-psabi') > > I based this update of mine on other existing multiline statements in dpdk. > But I did not notice that all of them are within parens. > > So both \ and () are fine. > > Bruce, I did not see this described in our meson coding style. > Do you have an opinion for multiline statements in meson? > No, I don't have an opinion either way. If you want, we can pick one to use in the coding style, but if we do we should also match the style for python code, as I believe they both share this multi-line approach.