From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3BF2BA2EFC for ; Thu, 19 Sep 2019 14:55:38 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C750D1EFD0; Thu, 19 Sep 2019 14:55:36 +0200 (CEST) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id 5BCC31EFCF for ; Thu, 19 Sep 2019 14:55:35 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Sep 2019 05:55:34 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,523,1559545200"; d="scan'208";a="192036295" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga006.jf.intel.com with ESMTP; 19 Sep 2019 05:55:34 -0700 Received: from shsmsx154.ccr.corp.intel.com (10.239.6.54) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 19 Sep 2019 05:55:33 -0700 Received: from shsmsx105.ccr.corp.intel.com ([169.254.11.23]) by SHSMSX154.ccr.corp.intel.com ([169.254.7.195]) with mapi id 14.03.0439.000; Thu, 19 Sep 2019 20:55:32 +0800 From: "Zhang, Qi Z" To: "Rong, Leyi" , "Lu, Wenzhuo" , "Ye, Xiaolong" CC: "dev@dpdk.org" Thread-Topic: [PATCH v3] net/iavf: enable AVX2 for iavf Thread-Index: AQHVbsbtKggRdW0cikiNS2VoNkOWuKcy9IMQ Date: Thu, 19 Sep 2019 12:55:31 +0000 Message-ID: <039ED4275CED7440929022BC67E7061153D98E79@SHSMSX105.ccr.corp.intel.com> References: <20190904100617.110676-1-leyi.rong@intel.com> <20190919084455.80907-1-leyi.rong@intel.com> In-Reply-To: <20190919084455.80907-1-leyi.rong@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNWYwMjYxMjAtNjNmYi00ZjU4LThkNTktYTkyYmU0MDM1NmQxIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoibHhzcWlBWmh2ZnVmZTJwcmU1aUIwZzRlK0hMSTExQjlDeGowekhQamhxTk1uTGl0blVUamY5Z0ZLUWpiMkxFSSJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action 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 v3] net/iavf: enable AVX2 for iavf 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" minor capture inline > -----Original Message----- > From: Rong, Leyi > Sent: Thursday, September 19, 2019 4:45 PM > To: Lu, Wenzhuo ; Zhang, Qi Z > ; Ye, Xiaolong > Cc: dev@dpdk.org; Rong, Leyi > Subject: [PATCH v3] net/iavf: enable AVX2 for iavf >=20 > This patch enables AVX data path for iavf PMD. >=20 > Signed-off-by: Leyi Rong >=20 > --- > v3: > - Adds release note. >=20 > v2: > - fix build error on aarch64. > --- <...> > diff --git a/drivers/net/iavf/iavf_rxtx_vec_common.h > b/drivers/net/iavf/iavf_rxtx_vec_common.h > index db509d71f..fff0555d2 100644 > --- a/drivers/net/iavf/iavf_rxtx_vec_common.h > +++ b/drivers/net/iavf/iavf_rxtx_vec_common.h > @@ -207,4 +207,76 @@ iavf_rxq_vec_setup_default(struct iavf_rx_queue > *rxq) > rxq->mbuf_initializer =3D *(uint64_t *)p; > return 0; > } > + > +static inline int > +iavf_rx_vec_queue_default(struct iavf_rx_queue *rxq) { > + if (!rxq) > + return -1; > + > + if (!rte_is_power_of_2(rxq->nb_rx_desc)) > + return -1; > + > + if (rxq->rx_free_thresh < IAVF_VPMD_RX_MAX_BURST) > + return -1; > + > + if (rxq->nb_rx_desc % rxq->rx_free_thresh) > + return -1; > + > + return 0; > +} > + > +#define IAVF_NO_VECTOR_FLAGS ( \ > + DEV_TX_OFFLOAD_MULTI_SEGS | \ > + DEV_TX_OFFLOAD_VLAN_INSERT | \ > + DEV_TX_OFFLOAD_SCTP_CKSUM | \ > + DEV_TX_OFFLOAD_UDP_CKSUM | \ > + DEV_TX_OFFLOAD_TCP_CKSUM) This macro looks redundant, it is already defined in ice_rxtx.h <...>