From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 2D8211B1EE for ; Wed, 10 Jan 2018 14:49:07 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 10 Jan 2018 05:49:07 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,340,1511856000"; d="scan'208";a="18149583" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by FMSMGA003.fm.intel.com with ESMTP; 10 Jan 2018 05:49:07 -0800 Received: from shsmsx102.ccr.corp.intel.com (10.239.4.154) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.319.2; Wed, 10 Jan 2018 05:49:07 -0800 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.213]) by shsmsx102.ccr.corp.intel.com ([169.254.2.189]) with mapi id 14.03.0319.002; Wed, 10 Jan 2018 21:49:05 +0800 From: "Zhang, Helin" To: "Zhang, Qi Z" , "Richardson, Bruce" , "Xing, Beilei" CC: "dev@dpdk.org" , "Yigit, Ferruh" Thread-Topic: [PATCH v2 0/2] AVX2 Vectorized Rx/Tx functions for i40e Thread-Index: AQHTiVbf/zO+FXx21keWF3/N/ru1iaNsLKUAgAD0FxA= Date: Wed, 10 Jan 2018 13:49:04 +0000 Message-ID: References: <20171123165314.168786-1-bruce.richardson@intel.com> <20180109143254.234428-1-bruce.richardson@intel.com> <039ED4275CED7440929022BC67E706115312A5AA@SHSMSX103.ccr.corp.intel.com> In-Reply-To: <039ED4275CED7440929022BC67E706115312A5AA@SHSMSX103.ccr.corp.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v2 0/2] AVX2 Vectorized Rx/Tx functions for i40e X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 10 Jan 2018 13:49:08 -0000 > -----Original Message----- > From: Zhang, Qi Z > Sent: Wednesday, January 10, 2018 3:15 PM > To: Richardson, Bruce; Xing, Beilei > Cc: dev@dpdk.org; Zhang, Helin; Yigit, Ferruh > Subject: RE: [PATCH v2 0/2] AVX2 Vectorized Rx/Tx functions for i40e >=20 >=20 >=20 > > -----Original Message----- > > From: Richardson, Bruce > > Sent: Tuesday, January 9, 2018 10:33 PM > > To: Zhang, Qi Z ; Xing, Beilei > > > > Cc: dev@dpdk.org; Zhang, Helin ; Yigit, Ferruh > > ; Richardson, Bruce > > > > Subject: [PATCH v2 0/2] AVX2 Vectorized Rx/Tx functions for i40e > > > > This patch adds an AVX2 vectorized path to the i40e driver, based on > > the existing SSE4.2 version. Using AVX2 instructions gives better > > performance than the SSE version, though the percentage increase > > depends on the exact settings used. For example: > > > > * Using 16B rather than 32B descriptors gives the biggest benefit since > > 2 descriptors at a time can be read, rather than just 1 when 32B ones > > are used. > > * Bigger burst sizes for RX gives improved performance - while we see a= n > > improvement with testpmd with the default burst size of 32, burst siz= es > > of up to 128 give further improvements > > * In my testing, most of the improvement comes from faster processing o= n > > the RX path, though the improved TX also gives benefit. > > > > This has been tested on a system with CPU: "Intel(R) Xeon(R) Gold 6154 > > CPU @ 3.00GHz", and I've focused on testing with Rx ring sizes of > > approx 1k - generally > > --rxd=3D1024 and --txd=3D512, rather than the defaults which tend to gi= ve > > poorer zero-loss performance due to the smaller amount of buffering. > > > > V2: > > * Fixed incorrect config variable reference in makefile > > * Added missing stub function for when vector drivers are disabled > > * Added missing references to the new functions when checking for vecto= r > > code paths, e.g. for ring tear-down > > > > Bruce Richardson (2): > > net/i40e: add AVX2 Tx function > > net/i40e: add AVX2 Rx function > > > > drivers/net/i40e/Makefile | 19 + > > drivers/net/i40e/i40e_rxtx.c | 66 ++- > > drivers/net/i40e/i40e_rxtx.h | 6 + > > drivers/net/i40e/i40e_rxtx_vec_avx2.c | 792 > > ++++++++++++++++++++++++++++++++++ > > 4 files changed, 880 insertions(+), 3 deletions(-) create mode > > 100644 drivers/net/i40e/i40e_rxtx_vec_avx2.c > > > > -- > > 2.14.3 >=20 > Acked-by: Qi Zhang Applied to dpdk-next-net-intel, with minor commit log changes. Thanks! /Helin