From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f53.google.com (mail-wm0-f53.google.com [74.125.82.53]) by dpdk.org (Postfix) with ESMTP id 282B3378E for ; Tue, 20 Sep 2016 20:17:58 +0200 (CEST) Received: by mail-wm0-f53.google.com with SMTP id l132so49249978wmf.1 for ; Tue, 20 Sep 2016 11:17:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to; bh=yx7OpMr2YzXRcv9UDN8i9W2+x5BBi2rTE+/Ht4gKiSQ=; b=s1BXp2mMy5IHL4Ol68U6A00cCCXFai0ijxehgSr84IB4JmBWheF0Uag5aAXlsEAy4T M4krze7lTyX7yxIlHxeOm3ZnkFiIeWg3Dt4+27eseVm6iYGg12tskJSEcGIS+3SvO06f dieOy1KGJQVDYKWBOwbePMOihhgT2G7ae22PNuRHifffVatt5Sx5dXEe9A3CQC507EEZ 8V6qlcHUI43U8TcUCcJJwrZPdvx1HN7MLBaLpsMjSHGGVtMvgpgfBDPwxC4X/DCmOru1 EfES7WFzonal5LksBrJdijmZk9RRfy9yqolBNUHC9G4RDN3DHDJdmRF5LqYYI+V9zw15 MUjQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=yx7OpMr2YzXRcv9UDN8i9W2+x5BBi2rTE+/Ht4gKiSQ=; b=dDGIVmbWIoDL0wffkm0VKe7IR73rXh+UD7Y0COWWZgXKtfk066cCB2EkPoyN3cKLR6 m08sRhIlK9FWvYmWnazglud+lxr5MeAY7IhBHUMXmUV2mgt/p2t+kSB+ycpvPFo/t/ub q3gZHlto1Qz3FTS8xXBwK45y8QWoPOO+FlqkvUYdKohVxMc6CWP9n3SxejeUQX44qdSj VFLCwvmb9FIm8iFIBzKPHiyjPHW0ua8okLXUXSa+ZlT+y6/3mDUyfps+1UcLGX3V5+Zo Mr9WTEwaE+N9iPq4Dc3kUcywBeD461BUX4GhDc170HIn7JhQVHF5Y39/Y3Las6SrZj0O cl5w== X-Gm-Message-State: AE9vXwNhrj0nIfoKM1zYcLdp56Wa+yZCruPRbMMAhYGfbVdQ8ggNTTTZX0wj/iObyjPTPdyx X-Received: by 10.194.112.68 with SMTP id io4mr28929871wjb.1.1474395477881; Tue, 20 Sep 2016 11:17:57 -0700 (PDT) Received: from 6wind.com (guy78-3-82-239-227-177.fbx.proxad.net. [82.239.227.177]) by smtp.gmail.com with ESMTPSA id y2sm29615906wji.42.2016.09.20.11.17.56 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 20 Sep 2016 11:17:57 -0700 (PDT) Date: Tue, 20 Sep 2016 20:17:51 +0200 From: Adrien Mazarguil To: Bruce Richardson Cc: dev@dpdk.org Message-ID: <20160920181751.GJ17252@6wind.com> References: <1474295814-4879-1-git-send-email-bruce.richardson@intel.com> <20160919145959.GB17252@6wind.com> <20160919152605.GG45548@bricha3-MOBL3> <20160920125126.GG17252@6wind.com> <20160920155848.GA74952@bricha3-MOBL3> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160920155848.GA74952@bricha3-MOBL3> Subject: Re: [dpdk-dev] [PATCH] net/mlx: fix compile errors with ignore pedantic pragma X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 20 Sep 2016 18:17:58 -0000 On Tue, Sep 20, 2016 at 04:58:48PM +0100, Bruce Richardson wrote: > On Tue, Sep 20, 2016 at 02:51:27PM +0200, Adrien Mazarguil wrote: > > On Mon, Sep 19, 2016 at 04:26:05PM +0100, Bruce Richardson wrote: > > > On Mon, Sep 19, 2016 at 04:59:59PM +0200, Adrien Mazarguil wrote: > > > > Hi Bruce, > > > > > > > > On Mon, Sep 19, 2016 at 03:36:54PM +0100, Bruce Richardson wrote: > > > > > With recent gcc versions, e.g. gcc 6.1, compilation of mlx drivers with > > > > > debug enabled produces lots of errors complaining that "pedantic" is > > > > > not a warning level that can be ignored. > > > > > > > > > > error: ‘-pedantic’ is not an option that controls warnings [-Werror=pragmas] > > > > > #pragma GCC diagnostic ignored "-pedantic" > > > > > ^~~~~~~~~~~ > > > > > > > > > > These errors can be removed by changing the "-pedantic" to "-Wpedantic". > > > > > > > > Nice to have a workaround, I thought they did not keep the option at all. > > > > However after testing: > > > > > > > > - It does not seem to work with GCC 4.6 and older, they prefer -pedantic: > > > > "warning: unknown option after `#pragma GCC diagnostic' kind". > > > > > > > > - GCC 4.9 (possibly 5.x as well) does not care, can use either -pedantic or > > > > -Wpedantic. > > > > > > > > - GCC 6 can only supports -Wpedantic. > > > > > > > > Note we're working toward removing the need for these #pragma in the first > > > > place as soon as possible, however in the meantime I fear that checking the > > > > GCC version is necessary. > > > > > > > Depends on how old of GCC version we need to support. From the release notes > > > it appears that -Wpedantic was introduced in GCC 4.8 (3 1/2 years ago). > > > > > > https://gcc.gnu.org/gcc-4.8/changes.html > > > > > > Do we need to support compilation on gcc versions older than this? > > > > I'm all for upgrading so I do not really mind if we stop caring about older > > GCC versions (especially considering this problem only occurs in debugging > > mode which is seldom used by non-developers). The version check is necessary > > if we want to keep full compatibility with at least: > > > > - RHEL <= 6.x > > - Debian <= 7.x > > - Ubuntu <= 13.04 > > > > Works for me either way, thus: > > > > Acked-by: Adrien Mazarguil > > > Any objections to dropping of support for debug settings for these OS's? Not sure who should answer this. These #pragma are workarounds we should eventually get rid of anyway (DPDK headers are now clean enough, some work remains to be done on the Verbs side) so let's use your patch in the meantime. -- Adrien Mazarguil 6WIND