From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yongseok Koh Subject: Re: [PATCH v3 5/5] net/mlx5: add vectorized Rx/Tx burst for SSE4.1 Date: Wed, 5 Jul 2017 22:58:15 +0000 Message-ID: References: <20170628230403.10142-1-yskoh@mellanox.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Cc: "dev@dpdk.org" , Adrien Mazarguil , =?iso-8859-1?Q?N=E9lio_Laranjeiro?= To: "ferruh.yigit@intel.com" Return-path: Received: from EUR03-VE1-obe.outbound.protection.outlook.com (mail-eopbgr50054.outbound.protection.outlook.com [40.107.5.54]) by dpdk.org (Postfix) with ESMTP id 8A87E2FDD for ; Thu, 6 Jul 2017 00:58:17 +0200 (CEST) In-Reply-To: Content-Language: en-US Content-ID: <766FFFCF9D775840B96AE31963F80873@eurprd05.prod.outlook.com> 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 Jul 5, 2017, at 11:12 AM, Yongseok Koh wrote: >=20 > To make vectorized burst routines enabled, it is required to run on x86_6= 4 > architecture which can support at least SSE4.1. If all the conditions are > met, the vectorized burst functions are enabled automatically. The decisi= on > is made individually on RX and TX. There's no PMD option to make a > selection. >=20 > Signed-off-by: Yongseok Koh > --- Hi Ferruh, I forgot to amend subject/description of this patch. Shall I submit v4? Or,= can you change it when you merge it, if the whole series of v3 is acked (by Nel= io)? If this patch can be accepted without submitting v4, then please amend the commit message to: ------ net/mlx5: add vectorized Rx/Tx burst for x86 To make vectorized burst routines enabled, it is required to run on x86_64 architecture. If all the conditions are met, the vectorized burst functions are enabled automatically. The decision is made individually on RX and TX. There's no PMD option to make a selection Signed-off-by: Yongseok Koh ------ This is because DPDK now has supporting SSE4.2 as a minimum requirement for= x86 host and I also dropped the requirement in v3. If I need to submit v4 to make the change, please let me know. Thanks, Yongseok