From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ferruh.yigit@intel.com>
Received: from mga04.intel.com (mga04.intel.com [192.55.52.120])
 by dpdk.org (Postfix) with ESMTP id 9A4BACF15
 for <dev@dpdk.org>; Fri, 17 Jun 2016 18:09:45 +0200 (CEST)
Received: from fmsmga004.fm.intel.com ([10.253.24.48])
 by fmsmga104.fm.intel.com with ESMTP; 17 Jun 2016 09:09:44 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.26,484,1459839600"; d="scan'208";a="124041361"
Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.221.18])
 ([10.237.221.18])
 by fmsmga004.fm.intel.com with ESMTP; 17 Jun 2016 09:09:44 -0700
To: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>, dev@dpdk.org
References: <1465379291-25310-1-git-send-email-nelio.laranjeiro@6wind.com>
From: Ferruh Yigit <ferruh.yigit@intel.com>
Message-ID: <576420C7.6010301@intel.com>
Date: Fri, 17 Jun 2016 17:09:43 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101
 Thunderbird/38.7.2
MIME-Version: 1.0
In-Reply-To: <1465379291-25310-1-git-send-email-nelio.laranjeiro@6wind.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 8bit
Subject: Re: [dpdk-dev] [PATCH 00/24] Refactor mlx5 to improve performance
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: Fri, 17 Jun 2016 16:09:46 -0000

On 6/8/2016 10:47 AM, Nelio Laranjeiro wrote:
> Enhance mlx5 with a data path that bypasses Verbs.
> 
> The first half of this patchset removes support for functionality completely
> rewritten in the second half (scatter/gather, inline send), while the data
> path is refactored without Verbs.
> 
> The PMD remains usable during the transition.
> 
> This patchset must be applied after "Miscellaneous fixes for mlx4 and mlx5".
> 
> Adrien Mazarguil (8):
>   mlx5: replace countdown with threshold for TX completions
>   mlx5: add debugging information about TX queues capabilities
>   mlx5: check remaining space while processing TX burst
>   mlx5: resurrect TX gather support
>   mlx5: work around spurious compilation errors
>   mlx5: remove redundant RX queue initialization code
>   mlx5: make RX queue reinitialization safer
>   mlx5: resurrect RX scatter support
> 
> Nelio Laranjeiro (15):
>   mlx5: split memory registration function for better performance
>   mlx5: remove TX gather support
>   mlx5: remove RX scatter support
>   mlx5: remove configuration variable for maximum number of segments
>   mlx5: remove inline TX support
>   mlx5: split TX queue structure
>   mlx5: split RX queue structure
>   mlx5: update prerequisites for upcoming enhancements
>   mlx5: add definitions for data path without Verbs
>   mlx5: add support for configuration through kvargs
>   mlx5: add TX/RX burst function selection wrapper
>   mlx5: refactor RX data path
>   mlx5: refactor TX data path
>   mlx5: handle RX CQE compression
>   mlx5: add support for multi-packet send
> 
> Yaacov Hazan (1):
>   mlx5: add support for inline send
> 

I run basic checks to the patchset:

There are various checkpatch warnings, all are warning or check level

Patch 8 and 13 failed to apply with via git, -looks line line numbers
shifted a little, this is not a problem since eventually it applies but
just for your information.

check-git-log is giving following errors, it is mainly case issue in Rx/Tx:
Wrong headline lowercase:
        mlx5: resurrect RX scatter support
        mlx5: make RX queue reinitialization safer
        mlx5: remove redundant RX queue initialization code
        mlx5: resurrect TX gather support
        mlx5: check remaining space while processing TX burst
        mlx5: add debugging information about TX queues capabilities
        mlx5: replace countdown with threshold for TX completions
        mlx5: handle RX CQE compression
        mlx5: refactor RX data path
        mlx5: add TX/RX burst function selection wrapper
        mlx5: split RX queue structure
        mlx5: split TX queue structure
        mlx5: remove inline TX support
        mlx5: remove RX scatter support
        mlx5: remove TX gather support
Headline too long:
        mlx5: remove configuration variable for maximum number of segments
        mlx5: split memory registration function for better performance


It compiles fine.

Regards,
ferruh