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 5C337A0543; Wed, 15 Jul 2020 12:36:06 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 338FF1BED7; Wed, 15 Jul 2020 12:36:06 +0200 (CEST) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 3198C1BEBC for ; Wed, 15 Jul 2020 12:35:58 +0200 (CEST) IronPort-SDR: q2ym0iqWuXxAsg9SrFlnzZVy/3csRInUI7Rxo5yWqFkoEG47Xxw60Q+mCeJ8YC6JtM7jWZtpkI IQeR8kfi8pcQ== X-IronPort-AV: E=McAfee;i="6000,8403,9682"; a="150529988" X-IronPort-AV: E=Sophos;i="5.75,355,1589266800"; d="scan'208";a="150529988" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Jul 2020 03:35:57 -0700 IronPort-SDR: vIUHiIcAs9q5HFjuIy5j+bXmy9zoayicZb6ZE18WdQ9jkySJmr/9grB7x7aITPSmklKfwvsRYJ Q2wzQe4nbowQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,355,1589266800"; d="scan'208";a="326128061" Received: from vmedvedk-mobl.ger.corp.intel.com (HELO [10.251.186.44]) ([10.251.186.44]) by orsmga007.jf.intel.com with ESMTP; 15 Jul 2020 03:35:54 -0700 To: Thomas Monjalon , "Kinsella, Ray" , Stephen Hemminger Cc: dev@dpdk.org, david.marchand@redhat.com, jerinj@marvell.com, konstantin.ananyev@intel.com, bruce.richardson@intel.com References: <98d800b4-41f8-751d-c1a5-5365135c225e@ashroe.eu> <20200714073828.7cdd7355@hermes.lan> <1758703.2oNzlfH3Ak@thomas> From: "Medvedkin, Vladimir" Message-ID: <07b5fd0a-25b7-3bb3-f268-458b2266c93e@intel.com> Date: Wed, 15 Jul 2020 11:35:53 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <1758703.2oNzlfH3Ak@thomas> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v6 0/8] fib: implement AVX512 vector lookup 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" On 15/07/2020 10:47, Thomas Monjalon wrote: > 14/07/2020 16:38, Stephen Hemminger: >> "Kinsella, Ray" wrote: >>> On 13/07/2020 23:19, Stephen Hemminger wrote: >>>> Did anyone else see the recent AVX512 discussion from Linus: >>>> "I hope AVX512 dies a painful death, and that Intel starts fixing real problems >>>> instead of trying to create magic instructions to then create benchmarks that they can look good on. >>> >>> Yup - I saw this one. >>> Sweeping statements like these are good to provoke debate, the truth is generally more nuanced. >>> If you continue to read the post, Linus appears to be mostly questioning microprocessor design decisions. >>> >>> That is an interesting discussion, however the reality is that the technology does exists and may be beneficial for Packet Processing. >>> >>> I would suggest, we continue to apply the same logic governing adoption of any technology by DPDK. >>> When the technology is present and a clear benefit is shown, we use it with caution. >>> >>> In the case of Vladimir's patch, >>> the user has to explicitly switch on the AVX512 lookup with RTE_FIB_DIR24_8_VECTOR_AVX512. >> >> Using what is available makes sense in DPDK. > > Why does it require explicit enabling in application? > AVX512 is not reliable enough to be automatically used when available? > It is reliable enough. User have to explicitly trigger to avx512 lookup because using avx512 instructions can reduce the frequency of your cores. The user knows their environment better. So the scalar version is used so as not to affect the frequency. > > -- Regards, Vladimir