From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ferruh Yigit <ferruh.yigit@intel.com>
Subject: Re: [PATCH] ena: Update PMD to cooperate with latest ENA
 firmware
Date: Mon, 20 Jun 2016 17:16:33 +0100
Message-ID: <576816E1.6000302@intel.com>
References: <1466099690-28085-1-git-send-email-jan@semihalf.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: quoted-printable
Cc: Alexander Matushevsky <matua@amazon.com>, Jakub Palider <jpa@semihalf.com>
To: Jan Medala <jan@semihalf.com>, dev@dpdk.org
Return-path: <dev-bounces@dpdk.org>
Received: from mga14.intel.com (mga14.intel.com [192.55.52.115])
 by dpdk.org (Postfix) with ESMTP id 88AE8AD82
 for <dev@dpdk.org>; Mon, 20 Jun 2016 18:16:36 +0200 (CEST)
In-Reply-To: <1466099690-28085-1-git-send-email-jan@semihalf.com>
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>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On 6/16/2016 6:54 PM, Jan Medala wrote:
> This patch includes:
> * Update of ENA communication layer
>=20
> * Fixed memory management issue
> 	After allocating memzone it's required to zeroize it
> 	as well as freeing memzone with dedicated function.
>=20
> * Added debug area and host information
>=20
> * Disabling readless communication regarding to HW revision
>=20
> * Allocating coherent memory in node-aware way
>=20
> Signed-off-by: Alexander Matushevsky <matua@amazon.com>
> Signed-off-by: Jakub Palider <jpa@semihalf.com>
> Signed-off-by: Jan Medala <jan@semihalf.com>
> ---

Hi Jan,

This is a big patch and hard to review, is it possible to split patch
into more logical pieces? List in the commit log is potential for
splinting into.

Also there are small checkpatch [1] and check-git-log [2] warnings.


[1]
### [dpdk-dev] ena: Update PMD to cooperate with latest ENA firmware

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional
statements (16, 16)
#568: FILE: drivers/net/ena/base/ena_com.c:2549:
                if (unlikely(!host_attr->debug_area_virt_addr)) {
+               host_attr->debug_area_size =3D 0;

WARNING:LONG_LINE: line over 80 characters
#1026: FILE: drivers/net/ena/base/ena_defs/ena_admin_defs.h:1806:
+               (val <<
ENA_ADMIN_FEATURE_RSS_FLOW_HASH_INPUT_ENABLE_L3_SORT_SHIFT)

WARNING:LONG_LINE: line over 80 characters
#2229: FILE: drivers/net/ena/ena_ethdev.c:357:
 +               (DRV_MODULE_VER_SUBMINOR <<
ENA_ADMIN_HOST_INFO_SUB_MINOR_SHIFT);

 total: 0 errors, 3 warnings, 0 checks, 2291 lines checked

0/1 valid patch


[2]
Wrong headline uppercase:
        ena: Update PMD to cooperate with latest ENA firmware

Thanks,
ferruh