From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 2/3] hash: run-time function selection Date: Sat, 20 Jan 2018 15:36:24 +0100 Message-ID: <2479496.07huUYr6Q1@xps> References: <1509991489-26442-1-git-send-email-elza.mathew@intel.com> <20171211132612.GA9292@bricha3-MOBL3.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, Bruce Richardson , pablo.de.lara.guarch@intel.com To: Elza Mathew Return-path: Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 9DB35A49C for ; Sat, 20 Jan 2018 15:37:00 +0100 (CET) In-Reply-To: <20171211132612.GA9292@bricha3-MOBL3.ger.corp.intel.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" 11/12/2017 14:26, Bruce Richardson: > On Mon, Nov 06, 2017 at 10:04:49AM -0800, Elza Mathew wrote: > > Compile-time function selection can potentially lead to > > lower performance on generic builds done by distros. > > Replaced compile time flag checks with run-time function > > selection. > > > > Signed-off-by: Elza Mathew > > --- > > lib/librte_hash/rte_fbk_hash.c | 11 ++++++++++- > > lib/librte_hash/rte_fbk_hash.h | 8 -------- > > 2 files changed, 10 insertions(+), 9 deletions(-) > > > Title needs an update to indicate this change is for fbk-hash. I suspect > that can be fixed on apply. > > Acked-by: Bruce Richardson Applied, thanks