Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-11-02 Thread Lorenzo Pieralisi
On Wed, Nov 01, 2017 at 02:29:26PM -0600, Al Stone wrote: > On 10/20/2017 03:22 AM, Lorenzo Pieralisi wrote: > > On Thu, Oct 19, 2017 at 11:54:22AM -0500, Jeremy Linton wrote: > > > > [...] > > > +cpu_topology[cpu].core_id = topology_id; > +

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-11-01 Thread Al Stone
On 10/20/2017 03:22 AM, Lorenzo Pieralisi wrote: > On Thu, Oct 19, 2017 at 11:54:22AM -0500, Jeremy Linton wrote: > > [...] > + cpu_topology[cpu].core_id = topology_id; + topology_id = setup_acpi_cpu_topology(cpu, 2); + cpu_topol

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-10-23 Thread Jeremy Linton
Hi, On 10/20/2017 02:55 PM, Jeffrey Hugo wrote: On 10/20/2017 10:14 AM, Jeremy Linton wrote: Hi, On 10/20/2017 04:14 AM, Lorenzo Pieralisi wrote: On Thu, Oct 19, 2017 at 11:13:27AM -0500, Jeremy Linton wrote: On 10/19/2017 10:56 AM, Lorenzo Pieralisi wrote: On Thu, Oct 12, 2017 at 02:48:55P

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-10-20 Thread Jeffrey Hugo
On 10/20/2017 10:14 AM, Jeremy Linton wrote: Hi, On 10/20/2017 04:14 AM, Lorenzo Pieralisi wrote: On Thu, Oct 19, 2017 at 11:13:27AM -0500, Jeremy Linton wrote: On 10/19/2017 10:56 AM, Lorenzo Pieralisi wrote: On Thu, Oct 12, 2017 at 02:48:55PM -0500, Jeremy Linton wrote: Propagate the topol

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-10-20 Thread Sudeep Holla
On 20/10/17 17:14, Jeremy Linton wrote: > Hi, > > On 10/20/2017 04:14 AM, Lorenzo Pieralisi wrote: >> On Thu, Oct 19, 2017 at 11:13:27AM -0500, Jeremy Linton wrote: >>> On 10/19/2017 10:56 AM, Lorenzo Pieralisi wrote: On Thu, Oct 12, 2017 at 02:48:55PM -0500, Jeremy Linton wrote: > Prop

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-10-20 Thread Jeremy Linton
Hi, On 10/20/2017 04:14 AM, Lorenzo Pieralisi wrote: On Thu, Oct 19, 2017 at 11:13:27AM -0500, Jeremy Linton wrote: On 10/19/2017 10:56 AM, Lorenzo Pieralisi wrote: On Thu, Oct 12, 2017 at 02:48:55PM -0500, Jeremy Linton wrote: Propagate the topology information from the PPTT tree to the cpu_

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-10-20 Thread Lorenzo Pieralisi
On Thu, Oct 19, 2017 at 11:54:22AM -0500, Jeremy Linton wrote: [...] > >>+ cpu_topology[cpu].core_id = topology_id; > >>+ topology_id = setup_acpi_cpu_topology(cpu, 2); > >>+ cpu_topology[cpu].cluster_id = topology_id; > >>+

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-10-20 Thread Lorenzo Pieralisi
On Thu, Oct 19, 2017 at 11:13:27AM -0500, Jeremy Linton wrote: > On 10/19/2017 10:56 AM, Lorenzo Pieralisi wrote: > >On Thu, Oct 12, 2017 at 02:48:55PM -0500, Jeremy Linton wrote: > >>Propagate the topology information from the PPTT tree to the > >>cpu_topology array. We can get the thread id, core

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-10-19 Thread Jeremy Linton
Hi, I missed the rest of the comment below.. On 10/19/2017 10:56 AM, Lorenzo Pieralisi wrote: On Thu, Oct 12, 2017 at 02:48:55PM -0500, Jeremy Linton wrote: Propagate the topology information from the PPTT tree to the cpu_topology array. We can get the thread id, core_id and cluster_id by ass

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-10-19 Thread Jeremy Linton
On 10/19/2017 10:56 AM, Lorenzo Pieralisi wrote: On Thu, Oct 12, 2017 at 02:48:55PM -0500, Jeremy Linton wrote: Propagate the topology information from the PPTT tree to the cpu_topology array. We can get the thread id, core_id and cluster_id by assuming certain levels of the PPTT tree correspond

Re: [PATCH v3 6/7] arm64: topology: Enable ACPI/PPTT based CPU topology.

2017-10-19 Thread Lorenzo Pieralisi
On Thu, Oct 12, 2017 at 02:48:55PM -0500, Jeremy Linton wrote: > Propagate the topology information from the PPTT tree to the > cpu_topology array. We can get the thread id, core_id and > cluster_id by assuming certain levels of the PPTT tree correspond > to those concepts. The package_id is flagge