From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v4 0/5] net/mlx5: add vectorized Rx/Tx burst for x86 Date: Fri, 7 Jul 2017 10:58:59 +0100 Message-ID: <8a4b6ea3-4cf5-723f-f6e5-8a1d740513e1@intel.com> References: <20170628230403.10142-1-yskoh@mellanox.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org, adrien.mazarguil@6wind.com, nelio.laranjeiro@6wind.com To: Yongseok Koh Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 8E27B2C8 for ; Fri, 7 Jul 2017 11:59:02 +0200 (CEST) In-Reply-To: Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 7/6/2017 7:41 PM, Yongseok Koh wrote: > This is to introduce more efficient Rx/Tx burst functions using SIMD > instructions. Currently it is only supported by 64bit x86 having SSE4.1. > > From functional perspective, Rx burst function is equivalent to the > existing mlx5_rx_burst() except for scatter support, which will be added > soon. Tx burst function supports multi-segment packets and offload flags > unless it is disabled by txq_flags. However, disabling those features will > bring a little higher performance. > > v4: > * Restore 'pragma' for PEDANTIC. > * Resolve compile error with clang. > > v3: > * Remove requirement of SSE4.1 as DPDK now mandates SSE4.2 support at least. > * Bug fix in "net/mlx5: select Rx/Tx callbacks when starting device" > - Need to re-select Rx burst func when chaning MTU size. > * Resolved an optimization issue of gcc-6 in rxq_burst_v() > - Bit shift (<<) for 128b vector type is compiled differently. 'psllq' is > needed instead of 'sal'. > * Minor changes to address what is mentioned by review. > - Remove 'pragma' for PEDANTIC > - Make mlx5_ptype_table global. > - Change name of some inline funcs which also exist in mlx4 by the same name. > - Fix comments and indentation/spacing. > > v2: > * Streamline redundant conditional clauses in txq_complete(). > * Remove the mempool pointer in txq->mp2mr structure. > * Fix indentation and spacing. > > Yongseok Koh (5): > net/mlx5: change indexing for Tx SW ring > net/mlx5: free buffers in bulk on Tx completion > net/mlx5: use buffer address for LKEY search > net/mlx5: select Rx/Tx callbacks when starting device > net/mlx5: add vectorized Rx/Tx burst for x86 Series applied to dpdk-next-net/master, thanks.