From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <danielx.t.mrzyglod@intel.com>
Received: from mga01.intel.com (mga01.intel.com [192.55.52.88])
 by dpdk.org (Postfix) with ESMTP id 946F39A8E
 for <dev@dpdk.org>; Wed, 22 Jun 2016 10:45:12 +0200 (CEST)
Received: from fmsmga001.fm.intel.com ([10.253.24.23])
 by fmsmga101.fm.intel.com with ESMTP; 22 Jun 2016 01:45:11 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.26,509,1459839600"; d="scan'208";a="992740934"
Received: from irsmsx101.ger.corp.intel.com ([163.33.3.153])
 by fmsmga001.fm.intel.com with ESMTP; 22 Jun 2016 01:45:10 -0700
Received: from irsmsx111.ger.corp.intel.com (10.108.20.4) by
 IRSMSX101.ger.corp.intel.com (163.33.3.153) with Microsoft SMTP Server (TLS)
 id 14.3.248.2; Wed, 22 Jun 2016 09:45:07 +0100
Received: from irsmsx107.ger.corp.intel.com ([169.254.10.96]) by
 irsmsx111.ger.corp.intel.com ([169.254.2.182]) with mapi id 14.03.0248.002;
 Wed, 22 Jun 2016 09:45:07 +0100
From: "Mrzyglod, DanielX T" <danielx.t.mrzyglod@intel.com>
To: "Singh, Jasvinder" <jasvinder.singh@intel.com>, "Dumitrescu, Cristian"
 <cristian.dumitrescu@intel.com>
CC: "dev@dpdk.org" <dev@dpdk.org>
Thread-Topic: [PATCH v2] examples/ip_pipeline: fix build error for gcc 4.8
Thread-Index: AQHRy5d7uYH/RAFa1UqKOhjL8KhH2J/1LWCg
Date: Wed, 22 Jun 2016 08:45:06 +0000
Message-ID: <7ADD74816B4C8A45B56203CBA65FE5A6377962B4@IRSMSX107.ger.corp.intel.com>
References: <1465472328-226671-1-git-send-email-danielx.t.mrzyglod@intel.com>
 <1466501740-28237-1-git-send-email-danielx.t.mrzyglod@intel.com>
In-Reply-To: <1466501740-28237-1-git-send-email-danielx.t.mrzyglod@intel.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-originating-ip: [163.33.239.180]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Subject: Re: [dpdk-dev] [PATCH v2] examples/ip_pipeline: fix build error for
	gcc 4.8
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: patches and discussions about DPDK <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: Wed, 22 Jun 2016 08:45:13 -0000

>From: Mrzyglod, DanielX T
>Sent: Tuesday, June 21, 2016 11:36 AM
>To: Singh, Jasvinder <jasvinder.singh@intel.com>; Dumitrescu, Cristian
><cristian.dumitrescu@intel.com>
>Cc: dev@dpdk.org; Mrzyglod, DanielX T <danielx.t.mrzyglod@intel.com>
>Subject: [PATCH v2] examples/ip_pipeline: fix build error for gcc 4.8
>
>This patch fixes a maybe-uninitialized warning when compiling DPDK with GC=
C 4.8
>
>examples/ip_pipeline/pipeline/pipeline_common_fe.c: In function
>'app_pipeline_track_pktq_out_to_link':
>examples/ip_pipeline/pipeline/pipeline_common_fe.c:66:31: error:
>'reader' may be used uninitialized in this function [-Werror=3Dmaybe-unini=
tialized]
>
>   struct app_pktq_out_params *pktq_out =3D
>
>Fixes: 760064838ec0 ("examples/ip_pipeline: link routing output ports to
>devices")
>
>Signed-off-by: Daniel Mrzyglod <danielx.t.mrzyglod@intel.com>
>Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
>---
> examples/ip_pipeline/app.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/examples/ip_pipeline/app.h b/examples/ip_pipeline/app.h
>index 7611341..242dae8 100644
>--- a/examples/ip_pipeline/app.h
>+++ b/examples/ip_pipeline/app.h
>@@ -667,11 +667,11 @@ app_swq_get_reader(struct app_params *app,
> 	struct app_pktq_swq_params *swq,
> 	uint32_t *pktq_in_id)
> {
>-	struct app_pipeline_params *reader;
>+	struct app_pipeline_params *reader =3D NULL;
> 	uint32_t pos =3D swq - app->swq_params;
> 	uint32_t n_pipelines =3D RTE_MIN(app->n_pipelines,
> 		RTE_DIM(app->pipeline_params));
>-	uint32_t n_readers =3D 0, id, i;
>+	uint32_t n_readers =3D 0, id =3D 0, i;
>
> 	for (i =3D 0; i < n_pipelines; i++) {
> 		struct app_pipeline_params *p =3D &app->pipeline_params[i];
>@@ -727,11 +727,11 @@ app_tm_get_reader(struct app_params *app,
> 	struct app_pktq_tm_params *tm,
> 	uint32_t *pktq_in_id)
> {
>-	struct app_pipeline_params *reader;
>+	struct app_pipeline_params *reader =3D NULL;
> 	uint32_t pos =3D tm - app->tm_params;
> 	uint32_t n_pipelines =3D RTE_MIN(app->n_pipelines,
> 		RTE_DIM(app->pipeline_params));
>-	uint32_t n_readers =3D 0, id, i;
>+	uint32_t n_readers =3D 0, id =3D 0, i;
>
> 	for (i =3D 0; i < n_pipelines; i++) {
> 		struct app_pipeline_params *p =3D &app->pipeline_params[i];
>--
>2.5.5

Self-NACK already applied