From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ferruh.yigit@intel.com>
Received: from mga09.intel.com (mga09.intel.com [134.134.136.24])
 by dpdk.org (Postfix) with ESMTP id 1B8EB1094
 for <dev@dpdk.org>; Mon, 16 Jan 2017 17:18:43 +0100 (CET)
Received: from fmsmga001.fm.intel.com ([10.253.24.23])
 by orsmga102.jf.intel.com with ESMTP; 16 Jan 2017 08:18:42 -0800
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.33,239,1477983600"; d="scan'208";a="1094788117"
Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.38])
 ([10.237.220.38])
 by fmsmga001.fm.intel.com with ESMTP; 16 Jan 2017 08:18:41 -0800
To: Shahaf Shuler <shahafs@mellanox.com>, adrien.mazarguil@6wind.com
References: <1484573439-10752-1-git-send-email-shahafs@mellanox.com>
Cc: dev@dpdk.org, Elad Persiko <eladpe@mellanox.com>
From: Ferruh Yigit <ferruh.yigit@intel.com>
Message-ID: <2a106ecd-6a38-59fa-3f7c-7cd9c357ff3e@intel.com>
Date: Mon, 16 Jan 2017 16:18:41 +0000
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.6.0
MIME-Version: 1.0
In-Reply-To: <1484573439-10752-1-git-send-email-shahafs@mellanox.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 8bit
Subject: Re: [dpdk-dev] [PATCH v3] net/mlx5: support extended statistics
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: Mon, 16 Jan 2017 16:18:44 -0000

Hi Shahaf,

On 1/16/2017 1:30 PM, Shahaf Shuler wrote:
> Implement extended statistics callbacks.
> 
> Suggested-by: Hanoch Haim <hhaim@cisco.com>
> Signed-off-by: Shahaf Shuler <shahafs@mellanox.com>
> Signed-off-by: Elad Persiko <eladpe@mellanox.com>

I am getting following build error [1] with clang when MLX5_DEBUG=y,
this seems because of pedandic.

Variable length array supported by language with c99 but mlx is forcing
to compile itself with gnu99 (-std=gnu99), not sure why, which supports
variable length array as an extension.


[1]
 .../drivers/net/mlx5/mlx5_stats.c:266:20: error: variable length array
folded to constant array as an extension [-Werror,-Wgnu-folding-constant]
        uint64_t counters[xstats_n];
                          ^~~~~~~~
 .../drivers/net/mlx5/mlx5_stats.c:289:20: error: variable length array
folded to constant array as an extension [-Werror,-Wgnu-folding-constant]
        uint64_t counters[xstats_n];
                          ^~~~~~~~