From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <olivier.matz@6wind.com>
Received: from mail.droids-corp.org (zoll.droids-corp.org [94.23.50.67])
 by dpdk.org (Postfix) with ESMTP id B11D01B254
 for <dev@dpdk.org>; Fri,  6 Oct 2017 09:31:12 +0200 (CEST)
Received: from lfbn-lil-1-182-75.w90-45.abo.wanadoo.fr ([90.45.31.75]
 helo=droids-corp.org)
 by mail.droids-corp.org with esmtpsa (TLS1.0:RSA_AES_256_CBC_SHA1:256)
 (Exim 4.84_2) (envelope-from <olivier.matz@6wind.com>)
 id 1e0NBq-0005Ld-4B; Fri, 06 Oct 2017 09:36:59 +0200
Received: by droids-corp.org (sSMTP sendmail emulation);
 Fri, 06 Oct 2017 09:31:04 +0200
Date: Fri, 6 Oct 2017 09:31:04 +0200
From: Olivier MATZ <olivier.matz@6wind.com>
To: Ferruh Yigit <ferruh.yigit@intel.com>
Cc: dev@dpdk.org
Message-ID: <20171006073102.lshs2mb4fdh7jxvz@platinum>
References: <20170911151333.5727-1-olivier.matz@6wind.com>
 <0def7ae9-ad8e-1df8-18cd-ee4e9515d621@intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <0def7ae9-ad8e-1df8-18cd-ee4e9515d621@intel.com>
User-Agent: NeoMutt/20170113 (1.7.2)
Subject: Re: [dpdk-dev] [PATCH 00/10] fix compilation with -Og
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Fri, 06 Oct 2017 07:31:12 -0000

Hi Ferruh,

On Fri, Oct 06, 2017 at 01:26:17AM +0100, Ferruh Yigit wrote:
> On 9/11/2017 4:13 PM, Olivier Matz wrote:
> > In developer mode (RTE_DEVEL_BUILD=y) where -Werror is passed in the
> > CFLAGS, the compilation fails with gcc-6.3.0 and EXTRA_CFLAGS=-Og. Some
> > errors are real bugs (but not critical), while some are false positives
> > (gcc bugs?).
> > 
> > The solution often consists in initializing a local variable to
> > ensure the compiler won't complain.
> > 
> > The patchset contains all the fixes needed to properly compile with -Og.
> > Feedback is welcome to decide if:
> > 1/ we include all of them, even if some are workarounds for
> >    gcc bugs
> > 2/ we only include the real fixes, without fixing the compilation with
> >    -Og.
> > 
> > 
> > Olivier Matz (10):
> >   net/bnxt: fix compilation with -Og
> >   net/qede: fix compilation with -Og
> >   net/virtio: fix compilation with -Og
> >   net/i40e: fix compilation with -Og
> >   uio: fix compilation with -Og
> >   cmdline: fix compilation with -Og
> >   metrics: fix compilation with -Og
> >   lpm6: fix compilation with -Og
> >   app/test-crypto-perf: fix memory leak
> >   app/test-crypto-perf: fix compilation with -Og
> 
> Series applied to dpdk-next-net/master, thanks.
> 
> Except 3/10, 4/10, 8/10. Normally we don't get partial sets, that makes
> life harder for everyone, but since mentioned ones requires some more
> input I didn't want to make rest wait.

Thanks

> When those three concluded would you mind sending another set for them?

Sure, will do as soon as we have the feedback


Olivier