Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753773AbdLMRiT (ORCPT ); Wed, 13 Dec 2017 12:38:19 -0500 Received: from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]:33064 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753739AbdLMRiP (ORCPT ); Wed, 13 Dec 2017 12:38:15 -0500 Date: Wed, 13 Dec 2017 17:38:51 +0000 From: Lorenzo Pieralisi To: Jeremy Linton Cc: "Rafael J. Wysocki" , linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, sudeep.holla@arm.com, hanjun.guo@linaro.org, will.deacon@arm.com, catalin.marinas@arm.com, gregkh@linuxfoundation.org, viresh.kumar@linaro.org, mark.rutland@arm.com, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, jhugo@codeaurora.org, wangxiongfeng2@huawei.com, Jonathan.Zhang@cavium.com, ahs3@redhat.com, Jayachandran.Nair@cavium.com, austinwc@codeaurora.org, lenb@kernel.org Subject: Re: [PATCH v5 6/9] ACPI/PPTT: Add topology parsing code Message-ID: <20171213173851.GA4060@red-moon> References: <20171201222330.18863-1-jeremy.linton@arm.com> <20171201222330.18863-7-jeremy.linton@arm.com> <9009154.M66RGdQJXA@aspire.rjw.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1080 Lines: 30 On Tue, Dec 12, 2017 at 10:13:08AM -0600, Jeremy Linton wrote: > Hi, > > First, thanks for taking a look at this. > > On 12/11/2017 07:12 PM, Rafael J. Wysocki wrote: > >On Friday, December 1, 2017 11:23:27 PM CET Jeremy Linton wrote: > >>The PPTT can be used to determine the groupings of CPU's at > >>given levels in the system. Lets add a few routines to the PPTT > >>parsing code to return a unique id for each unique level in the > >>processor hierarchy. This can then be matched to build > >>thread/core/cluster/die/package/etc mappings for each processing > >>element in the system. > >> > >>Signed-off-by: Jeremy Linton > > > >Why can't this be folded into patch [2/9]? > > It can, and I will be happy squash it. > > It was requested that the topology portion of the parser be split > out back in v3. > > https://www.spinics.net/lists/linux-acpi/msg78487.html I asked to split cache/topology since I am not familiar with cache code and Sudeep - who looks after the cache code - won't be able to review this series in time for v4.16. Lorenzo