Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756102AbdGCOZr (ORCPT ); Mon, 3 Jul 2017 10:25:47 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:44584 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756043AbdGCOZE (ORCPT ); Mon, 3 Jul 2017 10:25:04 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org F04B46087F Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=shankerd@codeaurora.org Reply-To: shankerd@codeaurora.org Subject: Re: [PATCH] irqchip: gicv3-its: Use NUMA aware memory allocation for ITS tables To: Marc Zyngier , Ganapatrao Kulkarni Cc: linux-kernel , linux-arm-kernel , Thomas Gleixner , Jason Cooper , Vikram Sethi , Jayachandran C , "ganapatrao.kulkarni@cavium.com" References: <1498405569-463-1-git-send-email-shankerd@codeaurora.org> From: Shanker Donthineni Message-ID: Date: Mon, 3 Jul 2017 09:24:59 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1710 Lines: 44 Hi Marc, On 06/30/2017 03:51 AM, Marc Zyngier wrote: > On 30/06/17 04:01, Ganapatrao Kulkarni wrote: >> On Fri, Jun 30, 2017 at 8:04 AM, Ganapatrao Kulkarni >> wrote: >>> Hi Shanker, >>> >>> On Sun, Jun 25, 2017 at 9:16 PM, Shanker Donthineni >>> wrote: >>>> The NUMA node information is visible to ITS driver but not being used >>>> other than handling errata. This patch allocates the memory for ITS >>>> tables from the corresponding NUMA node using the appropriate NUMA >>>> aware functions. >> >> IMHO, the description would have been more constructive? >> >> "All ITS tables are mapped by default to NODE 0 memory. >> Adding changes to allocate memory from respective NUMA NODES of ITS devices. >> This will optimize tables access and avoids unnecessary inter-node traffic." > > But more importantly, I'd like to see figures showing the actual benefit > of this per-node allocation. Given that both of you guys have access to > such platforms, please show me the numbers! > I'll share the actual results which shows the improvement whenever available on our next chips. Current version of Qualcomm qdf2400 doesn't support multi socket configuration to capture results and share with you. Do you see any other issues with this patch apart from the performance improvements. I strongly believe this brings the noticeable improvement in numbers on systems where it has multi node memory/CPU configuration. > Thanks, > > M. > -- Shanker Donthineni Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.