From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] config: cleanup existing RTE_CACHE_LINE_SIZE selection scheme Date: Mon, 07 Dec 2015 15:45:10 +0100 Message-ID: <1750446.FDPWrLgo55@xps13> References: <1449498170-19872-1-git-send-email-jerin.jacob@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: Jerin Jacob Return-path: Received: from mail-wm0-f54.google.com (mail-wm0-f54.google.com [74.125.82.54]) by dpdk.org (Postfix) with ESMTP id CF81E95D6 for ; Mon, 7 Dec 2015 15:46:23 +0100 (CET) Received: by wmvv187 with SMTP id v187so169901957wmv.1 for ; Mon, 07 Dec 2015 06:46:23 -0800 (PST) In-Reply-To: <1449498170-19872-1-git-send-email-jerin.jacob@caviumnetworks.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2015-12-07 19:52, Jerin Jacob: > by default, all the targets will be configured with the 64-byte cache line > size, targets which have different cache line size can be overridden > through target specific config file. > > Selected ThunderX and power8 as CONFIG_RTE_CACHE_LINE_SIZE=128 targets > based on existing configuration. > > Signed-off-by: Jerin Jacob It looks good. We are moving from a constant defined on CC command line in most cases to a constant included in rte_config.h only. Have you checked that the generated libraries are identical? It would be good to have a check for POWER8. Chao? Thanks